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

GitHub identity is persisted locally after log-out #13694

Closed
artaleks9 opened this issue Jul 4, 2019 · 8 comments
Closed

GitHub identity is persisted locally after log-out #13694

artaleks9 opened this issue Jul 4, 2019 · 8 comments
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.
Milestone

Comments

@artaleks9
Copy link
Contributor

artaleks9 commented Jul 4, 2019

Description

It seems logout from CHE doesn't include logout from GitHub.
The problem was detected in the beginning in the CRW (https://issues.jboss.org/browse/CRW-323)

Reproduction Steps

  • User1 logs into CHE, performs authorization on his GitHub account, gets list of his projects and logs out from CHE
  • User2 logs into CHE in the same browser window
  • User2 can see and use the projects from User1 GitHub account

OS and version:
Environment: Openshift
7.0.0-rc-3.0-SNAPSHOT

Additional information:

  • See video record:

che-eclipse-github-persist

@artaleks9 artaleks9 added 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. target/che7GA labels Jul 4, 2019
@tsmaeder tsmaeder added this to the 7.0.0 milestone Jul 5, 2019
@slemeur
Copy link
Contributor

slemeur commented Jul 16, 2019

cc @vparfonov

@l0rd l0rd mentioned this issue Jul 16, 2019
85 tasks
@skabashnyuk
Copy link
Contributor

@artaleks9 can you update the issue with the information about the environment that was used?

@skabashnyuk
Copy link
Contributor

@ashumilova can you remind me what service dashboard is using to get a token? is this something from Che master side or it's communicating with Github directly?

@ashumilova
Copy link
Contributor

@skabashnyuk It uses our API method /api/oauth/token?oauth_provider=github

@skabashnyuk
Copy link
Contributor

@ashumilova As I can see GitHub token is cached in LocalStorage.
Знімок екрана  о 15 41 40
From your POV how this usecase should be handled?

@skabashnyuk
Copy link
Contributor

BTW any manipulation of user with github link to his kyecloak account will not be visible until this item exists in cache I think.

@gorkem
Copy link
Contributor

gorkem commented Jul 19, 2019

@ashumilova shouldn't logout clear all the LocalStorage?

@ashumilova
Copy link
Contributor

It's whether not storing token in local storage(trying session storage), that is shared between user sessions, or adding additional data to differ user tokens. Not sure logout could be tracked in all use cases (expiration for example).

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

7 participants