Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

temporarly fix for previously fixed bug #112

Merged
merged 1 commit into from
Jul 11, 2024

Conversation

khsrali
Copy link
Contributor

@khsrali khsrali commented Jul 2, 2024

FirecREST is used to raise a wrong exception from utilities/mkdir on existing folder.
This was fixed in #172, however, now instead of X-Exists, which is the expected response, it returns X-Invalid-Path (see the new issue #202 )

This PR will update the outdated wrong header exception, to the new wrong header exception 😆
The purpose is to make things function, until issue #202 is solved on FirecREST.

Once that issue is resolved, the header exception should change appropriately to X-Exists

@ekouts ekouts merged commit cfa46c5 into eth-cscs:main Jul 11, 2024
8 checks passed
@khsrali khsrali deleted the temp-fix-mkdir branch July 16, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants