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

Validate locally stored tokens for Cloud Integrations #3619

Open
axosoft-ramint opened this issue Sep 26, 2024 · 0 comments
Open

Validate locally stored tokens for Cloud Integrations #3619

axosoft-ramint opened this issue Sep 26, 2024 · 0 comments
Labels
area-integrations Issues or features related to integrations
Milestone

Comments

@axosoft-ramint
Copy link
Contributor

axosoft-ramint commented Sep 26, 2024

Our "sync cloud integrations" process checks if integrations were disconnected in the backend and disconnects them locally if they were, but it doesn't check if the active token has changed on the backend. In this case, it should update its local token.

This can be done in several ways:

  1. During the passive sync that runs during checkin, grab the actual token for each connected one, then validate against what you have stored locally and updated if needed.
  2. (Preferred): When using a cloud token, if you get an error, or when it expires, try calling the /refresh backend route first to refresh the token, and give the call another try. If it still fails after refresh, show an error.

Note for 2 that when calling the refresh route, if the token on the backend differs than what you sent it in the body, it returns the new token instead of the current one.

@axosoft-ramint axosoft-ramint added this to the 16.0 milestone Sep 26, 2024
@axosoft-ramint axosoft-ramint added the area-integrations Issues or features related to integrations label Sep 26, 2024
@eamodio eamodio modified the milestones: 16.0, 16.1 Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-integrations Issues or features related to integrations
Projects
None yet
Development

No branches or pull requests

2 participants