-
Notifications
You must be signed in to change notification settings - Fork 182
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
creating group give 200 status code #7678
Comments
I found a couple other inconsistencies between specification and actual behavior. creating users:https://owncloud.dev/libre-graph-api/#/users/CreateUser expects export GDPR:https://owncloud.dev/libre-graph-api/#/user/ExportPersonalData - |
Let us please align that. @kulmann @JammingBen can we change on the server side? Is that a breaking change for web? |
Web doesn't care about the status code. But it needs the id of the created user within the response body. I.e. could be changed in the backend without breaking web.
Web triggers the request and only reacts on error status codes. Immediately goes into a scheduled task which checks if the gdpr export file is already available / not in processing ( |
we need to adjust the status code in the test file after fix
ocis/tests/acceptance/features/apiGraph/createUser.feature Lines 24 to 37 in 2009114
|
The status code expected in the test will need to be adjusted in the PR that implements the fix into ocis. |
Describe the bug
according to the api specification on https://owncloud.dev/libre-graph-api/#/groups/CreateGroup
creating group gives
201
status code but it actually gives200
Steps to reproduce
create a group
response
Expected behavior
http status code should be
201
Actual behavior
http status code should is
200
Setup
Please describe how you started the server and provide a list of relevant environment variables or configuration files.
OCIS_XXX=master
The text was updated successfully, but these errors were encountered: