Skip to content
This repository has been archived by the owner on Jan 31, 2022. It is now read-only.

Move all authentication to use the "fallback" method. #77

Open
DominicMaas opened this issue Aug 24, 2018 · 2 comments
Open

Move all authentication to use the "fallback" method. #77

DominicMaas opened this issue Aug 24, 2018 · 2 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@DominicMaas
Copy link
Owner

There seems to be some issues when using chrome.identity.launchWebAuthFlow() with the popup box just not appearing. The fallback method creates a new tab, redirects to a blank page where we then collect the token. Works much faster and helps condense code.

It should be noted that users who are already logged in need to keep using the old client id otherwise refresh tokens will not work. New users can use the "fallback" client id. Perform some kind of check (like storing a variable on login in storage) that separates old and new users.

@DominicMaas DominicMaas added the enhancement New feature or request label Aug 24, 2018
@DominicMaas DominicMaas added this to the 1.0.6 milestone Aug 24, 2018
@DominicMaas DominicMaas self-assigned this Aug 24, 2018
@DominicMaas
Copy link
Owner Author

In the latest commit I added changes to use the new auth system for new users. I did perform some brief testing for existing users, but if someone watching could give it a test and see if everything still works, that would be great!

The aim is to not break existing logged in accounts (including refresh tokens), while implementing an easier system for new users.

@DominicMaas
Copy link
Owner Author

Firefox has issues signing in with the new code, see #61

Error: Opener tab must be in the same window as the tab being created => undefined
tab is undefined => background.js:217
Unchecked lastError value: Error: Invalid tab ID: 254 => background.js:216

Seems having multiple windows open while signing in does not work.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant