-
Notifications
You must be signed in to change notification settings - Fork 179
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
[full-ci] standalone graph service with LDAP #5199
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
Notes:
|
💥 Acceptance test localApiTests-apiGraph-ocis failed. Further test are cancelled... |
13a3ba6
to
5c89176
Compare
wow, CI ran into a panic https://drone.owncloud.com/owncloud/ocis/17385/51/4
|
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
5c89176
to
4126c7a
Compare
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
fixed the panic in cs3org/reva#3534 |
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
@butonic I was able to reproduce the webui test failure. And AFAICS it's caused by the changed status code in PostUser(). I have no idea why that is. But changing the status code back to I somehow suspect this to be a weird issue with test-middleware ... but that is just a wild guess. |
5c3ca10
to
8f00f63
Compare
@individual-it how do we bump the version of the test middleware? https://github.com/owncloud/owncloud-test-middleware/blob/a8233f8f5895a74d09e83b75199883c62a8bbf8a/src/helpers/graphHelper.js#L28 needs to check for 201 ... as well ... as the actual acceptance tests and the code itself. |
|
thx, we should get this part of the changes in ASAP |
to test set up an ldap server and set these environment variables:
|
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
8f00f63
to
b563c1f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Kudos, SonarCloud Quality Gate passed! |
TODO in a subsequent PR
|
Author: Jörn Friedrich Dreyer <jfd@owncloud.com> Date: Mon Dec 12 12:46:10 2022 +0100 [full-ci] standalone graph service with LDAP (#5199) * standalone graph service with LDAP Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de> * no panic on PATCH and DELETE Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de> * fix apitoken yaml key Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de> * update user, fix response codes Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de> * fix group creation return code Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de> * remove unknown user property Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de> * fix create return code checks in graph feature context Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de> * updating uses 200 OK when returning a body Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de> * revert user statusCreated change for now Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de> * revert return code changes Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de> Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
This PR allows setting an API Token for bearer based authentication. Setting a Token will disable the requireAdmin middleware and for now, also disable provisioning roles for new users.
Together with the LDAP backend this allows using the graph service for provisioning.
we will fix creation status codes in a subsequent PR