-
Notifications
You must be signed in to change notification settings - Fork 189
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
add enable basic auth option and check permissions #627
Conversation
b2d605f
to
c5cfc10
Compare
default is disabled, code looks good. maybe add a docs page for testing?? |
c5cfc10
to
f8bc7cf
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.
Maybe we should add a log message with Level WARN that says that the configuration is insecure? Other than that
yep, I am annoying admins on every request made with basic auth: https://github.com/owncloud/ocis/pull/627/files#diff-f41fd6020fdfa56d329e7af72f4aab2cR94 |
f8bc7cf
to
9f4bcf9
Compare
a lot of the background steps are now failing because the ocs api now dies with a 500 when basic auth is enabled:
|
it tries to contact 9152 ... the old root storage that should no longer be in use ...
|
this is what is going on. the testsuite uses the ocs api to provision accounts. the requests were not authenticated, yet. now the proxy tries to authenticate 'admin', the default admin account, which does not exist, so the proxy returned a 500 because it was trying to create a new account ... so we now end the request when basic auth fails. We also now use the default admin account now, which is |
8d187a8
to
64f9db7
Compare
34bc326
to
807c042
Compare
Rebased and force pushed... |
807c042
to
3447d98
Compare
I introduced an |
478b66b
to
77793d0
Compare
924b3d9
to
7ff8da0
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>
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>
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>
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>
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>
793c322
to
fdfd404
Compare
Kudos, SonarCloud Quality Gate passed!
|
We added a new
enable-basic-auth
option andPROXY_ENABLE_BASIC_AUTH
environment variable that can be set totrue
to make the proxy verify the basic auth header with the accounts service. This should only be used for testing and development and is disabled by default.first step for owncloud/product#198
I am not using a new middleware, because I want to save an additional trip to the accounts service.