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

return proper errors when ocs/cloud/users is using the cs3 backend #3530

Merged
merged 1 commit into from
Apr 20, 2022

Conversation

rhafer
Copy link
Contributor

@rhafer rhafer commented Apr 13, 2022

The ocs API was just exiting with a fatal error on any update request,
when configured for the cs3 backend. Now it returns a proper error.

Closes: #3483

@rhafer rhafer requested a review from butonic April 13, 2022 15:05
@rhafer rhafer self-assigned this Apr 13, 2022
@rhafer rhafer force-pushed the cs3-no-fatal branch 2 times, most recently from fe53043 to 605f8a5 Compare April 20, 2022 09:23
The ocs API was just exiting with a fatal error on any update request,
when configured for the cs3 backend. Now it returns a proper error.

Closes: owncloud#3483
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
44.4% 44.4% Duplication

@rhafer rhafer merged commit fa1b6d5 into owncloud:master Apr 20, 2022
ownclouders pushed a commit that referenced this pull request Apr 20, 2022
Merge: 7266049 916b2bb
Author: Ralf Haferkamp <[email protected]>
Date:   Wed Apr 20 13:06:52 2022 +0200

    Merge pull request #3530 from rhafer/cs3-no-fatal

    return proper errors when ocs/cloud/users is using the cs3 backend
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.

updating graph users using ocs endpoint crashes the ocis service
2 participants