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

Dashboard messages should be more accurate when self-signed cert issue happens #15298

Closed
6 of 9 tasks
sleshchenko opened this issue Nov 25, 2019 · 2 comments
Closed
6 of 9 tasks
Assignees
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.

Comments

@sleshchenko
Copy link
Member

sleshchenko commented Nov 25, 2019

Describe the bug

When Che is deployed with self-signed certificate and the corresponding CA is not imported to browser - the only way to understand why Che fails to load in browser - use Developer Tools and analyze failed requests.
So, Dashboard messages should be more accurate when self-signed cert issue happens

Che version

  • latest
  • nightly
  • other: please specify

Steps to reproduce

  1. Deploy Che with self-signed certs
  2. Open Che in a browser without CA imported
  3. Add Che host as trusted.
  4. Check error message:
    Actual: The authorization token is missing
Screenshot

Screenshot_20191125_141727

Expected: Something like Keycloak host is signed with self-signed certificate. Add it to trusted hosts or import the corresponding CA into browser

Expected behavior

Error messages clearly describe an issue and ways how to fix it.

Runtime

  • kubernetes (include output of kubectl version)
  • Openshift (include output of oc version)
  • minikube (include output of minikube version and kubectl version)
  • minishift (include output of minishift version and oc version)
  • docker-desktop + K8S (include output of docker version and kubectl version)
  • other: (please specify)
@sleshchenko sleshchenko added kind/bug Outline of a bug - must adhere to the bug report template. team/ide2 labels Nov 25, 2019
@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 Nov 25, 2019
@ibuziuk ibuziuk 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 Nov 25, 2019
@sleshchenko sleshchenko added this to the Backlog - Controller milestone Jan 9, 2020
@sleshchenko
Copy link
Member Author

I believe it's really important since we have plans to enable TLS by default, and even without it I received a lot of complain that people do not understand what is wrong with their installation and consider such error as totally non-working Che, but in fact but more step is needed from user side and it's just not obvious.
cc @slemeur @l0rd

@sleshchenko sleshchenko added severity/P1 Has a major impact to usage or development of the system. and removed severity/P2 Has a minor but important impact to the usage or development of the system. labels Jan 11, 2020
@sleshchenko
Copy link
Member Author

The PR that fixes the case with Authorization token is missed is merged
Screenshot from 2020-02-12 09-34-18

@l0rd l0rd mentioned this issue Mar 5, 2020
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

4 participants