Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

Todo: Update Redirect URLs (external) #6

Open
sorenjohanson opened this issue Jan 2, 2020 · 5 comments
Open

Todo: Update Redirect URLs (external) #6

sorenjohanson opened this issue Jan 2, 2020 · 5 comments
Assignees

Comments

@sorenjohanson
Copy link
Member

Reminder to update external redirect URLs on release.

@sorenjohanson
Copy link
Member Author

sorenjohanson commented Apr 28, 2020

So the following URLs need to be updated:

  • Dropbox
  • pCloud
  • OneDrive/Azure
  • Google Drive

Dropbox and pCloud work without issues on both Chrome (which uses https://.chromiumapps.org/) and Firefox (which uses https://.allizom.org/).

OneDrive refuses to do different root URLs, so either Chrome or Firefox works, but not both.

Google Drive is pending approval, but I think it's misconfigured. There's a Redirect URL complaint on Firefox and on Chrome there's only an [Object] object error. Maybe the Client ID is set up wrong.

@sorenjohanson
Copy link
Member Author

Switched reject(err) to reject(err.message) in the GDrive handler. Turns out the client ID given in the oauth2 manifest is wrong.

@sorenjohanson
Copy link
Member Author

Google Drive works on Firefox, but not on Chrome:

OAuth2 request failed: Service responded with error: 'bad client id: 212480071870-s2opocg9q728787qph4eogsko25gsnt9.apps.googleusercontent.com'

@sorenjohanson
Copy link
Member Author

Chrome should work now, once the version has been published.

@sorenjohanson
Copy link
Member Author

Update:

Dropbox works on both Firefox and Chrome 83+ ✅
pCloud works on both Firefox and Chrome ✅
OneDrive still has issues, since I can only specify one single Redirect URI in the Developer Console. ❌
Google Drive works on Firefox... ✅
... but not on Chrome, most likely since the OAuth2 screen has not been approved yet ❌

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

No branches or pull requests

1 participant