-
Notifications
You must be signed in to change notification settings - Fork 57
CHE-283 : Add a property to disable keycloak authentication in rh-che assembly #174
CHE-283 : Add a property to disable keycloak authentication in rh-che assembly #174
Conversation
7c236f9
to
37f1862
Compare
@davidfestal Does this PR also includes the work that is done in - #173 (there are common parts in both but some functionality is missing) ? |
No. I'm testing #173 currently (with a small modification to allow testing Keycloak on Minishift). Then when we merge #173 to But you have the opportunity to review the main changes of this PR in the meantime :-) |
@davidfestal ok, thanks for clarifications.
Could you please explain how to test #173 on Minishift ? |
It's necessary to add it on `master` and on `openshift-connector-branch` to avoid a broken build when the related `fabric8-online` changes are be merged.
It's necessary to add it on `master` and on `openshift-connector-branch` to avoid a broken build when the related `fabric8-online` changes are be merged.
Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
Signed-off-by: David Festal <dfestal@redhat.com>
37f1862
to
748a479
Compare
Signed-off-by: David Festal <dfestal@redhat.com>
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
… assembly (#174) * First sketch of CHE-283 * Add the customized valve to enable bypassing Keycloak authentication * Disable Keycloak by default (as requested by @l0rd) in the RH assembly * Fix a bad bug bringing CORS errors due to duplicate headers in response * Clean formatting-only changes * Fix for #181 Signed-off-by: David Festal <dfestal@redhat.com>
This PR introduces the new
che.keycloak.disabled
property in therh-che.properties
file.When this property is set to
true
, all the related Keycloak actions, both client side and server-side are bypassed, and no authentication is required.