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

Pre-emptively trigger login after google token expired #200

Open
tyler-dane opened this issue Dec 24, 2024 · 0 comments · May be fixed by #212
Open

Pre-emptively trigger login after google token expired #200

tyler-dane opened this issue Dec 24, 2024 · 0 comments · May be fixed by #212
Labels
enhancement New feature or request

Comments

@tyler-dane
Copy link
Contributor

tyler-dane commented Dec 24, 2024

Feature Title

Pre-emptively trigger login after google token expired

Feature Description

Currently, the user's session might be active, but their google access token has expired. Compass doesn't know that the google access token expired, though, because it doesn't proactively check for it. Instead, it tries to make a normal event change request. Then google responds letting us know that the token expired, which we use to log the user out and redirect them to the sign in page.

The down side of this is that the user thought they were all good and spent time making/updating an event, only to be promptly redirected to login without any warning.

Instead, we should redirect the user before they're able to make any changes, providing a nice UX/message about why this is important.

Use Case

No response

Additional Context

implementation idea(s):

  • use the WebSocket's connection as a trigger to run a preliminary check
  • if a user returns the compass after a long time, the session with be refreshed. before/after that happens, do a preemptive GET request on a Google calendar resource. This will let us know if their access token is still valid. There's probably a better way to
@tyler-dane tyler-dane added the enhancement New feature or request label Dec 24, 2024
@tyler-dane tyler-dane moved this to Backlog in 🏗 Compass Roadmap Dec 24, 2024
@tyler-dane tyler-dane moved this from Backlog to In review in 🏗 Compass Roadmap Jan 3, 2025
@tyler-dane tyler-dane moved this from In review to In progress in 🏗 Compass Roadmap Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In progress
Development

Successfully merging a pull request may close this issue.

1 participant