- Notifications
You must be signed in to change notification settings - Fork928
Closed
Description
Turning this into an issue for documentation purposes. I'm dropping everything and working on this now, and gunning to have this done by end of day
Updates
- I think I got this figured out – when we started trying to embed more and more data into the HTML to speed up queries, we didn't update all the query logic to account for the fact that even when we log out, there's still going to be user data in the HTML
- None of the unit tests caught this because the metadata doesn't exist in the JS runtime – it only exists when we're actually serving files through the Go backend
Steps to reproduce
- Log into Coder via Google OAuth (I'm using Coder Dogfood)
- Click the user icon in the top-right corner
- Click "Sign out"
- Immediate "Unexpected application error" based on permissions being unavailable
Video
Screen.Recording.2024-05-02.at.9.17.04.AM.mov
Notes
- As far as I can tell, this is strictly an issue on the UI side. The log out request is correctly going through, but the UI isn't syncing correctly afterwards