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

search not working after login to different account without refresh #881

Closed
FSM1 opened this issue Apr 7, 2021 · 2 comments · Fixed by #907
Closed

search not working after login to different account without refresh #881

FSM1 opened this issue Apr 7, 2021 · 2 comments · Fixed by #907
Assignees
Labels
Priority: P0 Critical Added to issues relating to a critical severity bugs.

Comments

@FSM1
Copy link
Contributor

FSM1 commented Apr 7, 2021

Investigate whether the correct account reflects on the API Client when logging out and back in, without refreshing

@edwardmack commented on Tue Apr 06 2021

Steps to reproduce:

  • Using cs files, logged in with metamask account ...9869D7, Search works as expected. Sign out.
  • Sign in, using different account, continue with Github. I see the files in connected to my github login.
  • Try searching for known file name, no results.
  • Refresh browser (F5),
  • Try searching for known file name, now it shows the expected results.
@FSM1 FSM1 self-assigned this Apr 7, 2021
@FSM1 FSM1 added the Priority: P0 Critical Added to issues relating to a critical severity bugs. label Apr 7, 2021
@RyRy79261
Copy link
Contributor

Reproduced

Github login files:
image

Github login search:
image

@RyRy79261
Copy link
Contributor

Noted in the console, that the query does receive something back:
image
Though not the second time.

Having refreshed the authorization bearer tokens had changed
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: P0 Critical Added to issues relating to a critical severity bugs.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants