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.

Sign up for GitHub

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

Jump to bottom

Allow localhost as redirect URL for keycloak authentication #19158

Closed
ibuziuk opened this issue Feb 25, 2021 · 2 comments
Closed

Allow localhost as redirect URL for keycloak authentication #19158

ibuziuk opened this issue Feb 25, 2021 · 2 comments
Assignees
@sleshchenko
Labels
area/install Issues related to installation, including offline/air gap and initial setup kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system.
Milestone
7.34

Comments

@ibuziuk
Copy link
Member

ibuziuk commented Feb 25, 2021 •
edited
Loading

Is your task related to a problem? Please describe.

Currently, localhost redirect URL should be configured manually by admin
Would be great to have an env var (probably disabled by default) to automatically add the keycloak / SSO redirect web origin for localhost development. This will allow having workspaces.openshift.com a default option for local run of Eclipse Che Dashboard without any manual steps

Describe the solution you'd like

property e.g. che.keycloak.allow_localhost, if set to true adds the localhost:3000 redirect / web-origin, which would allow local run of Eclipse Che Dashboard

Describe alternatives you've considered

Manually configured by admin

Additional context

Downstream issue https://issues.redhat.com/browse/CRT-955
@ibuziuk ibuziuk added the kind/task Internal things, technical debt, and to-do tasks to be performed. label Feb 25, 2021
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Feb 25, 2021
@ibuziuk ibuziuk added area/dashboard area/install Issues related to installation, including offline/air gap and initial setup labels Feb 25, 2021
@ibuziuk
Copy link
Member Author

ibuziuk commented Feb 25, 2021

cc: @sleshchenko @tolusha

@Katka92 Katka92 added severity/P2 Has a minor but important impact to the usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Mar 1, 2021
@tolusha tolusha added this to the 7.29 milestone Mar 18, 2021
@tolusha tolusha mentioned this issue Mar 23, 2021
Closed
38 tasks
@tolusha tolusha removed this from the 7.29 milestone Apr 8, 2021
This was referenced Jul 16, 2021
Merged
Merged
@sleshchenko sleshchenko self-assigned this Jul 16, 2021
@sleshchenko sleshchenko added this to the 7.34 milestone Jul 16, 2021
@tolusha tolusha mentioned this issue Jul 20, 2021
Closed
24 tasks
@tolusha tolusha mentioned this issue Jul 21, 2021
Merged
6 tasks
@sleshchenko
Copy link
Member

Implemented in eclipse-che/che-operator#937.
eclipse-che/che-dashboard#300 adds instructions on how to enable localhost on the cluster.

@tolusha tolusha mentioned this issue Jul 26, 2021
Closed
31 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sleshchenko sleshchenko

Labels
area/install Issues related to installation, including offline/air gap and initial setup kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Milestone
7.34
Development

No branches or pull requests
5 participants
@ibuziuk @tolusha @sleshchenko @Katka92 @che-bot