-
Notifications
You must be signed in to change notification settings - Fork 0
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
Google data not loading from public document link without login #3
Comments
Yes, that is expected because an API key is required even for accessing public documents. We could instead display a better error message indicating that this API key is missing, or simply remove the Google Data extension from the snapshot releases by default. |
@wetneb If the Google drive extension was external and installed manually by users, how would things become easier as far as the API key requirement. I am trying to envision how making the extension manually installed would make it easier for Google drive connections for our users. Can you elaborate with a step by step in your reply? |
If the Google drive extension was external, it could be installed in any version of OpenRefine (built manually from the source, snapshot release or official release) and would come with its own Google API credentials, so OpenRefine itself would not need to store those credentials. |
I'm seeing this with the official 3.7.6 release as well. |
My best guess is that this is a symptom of the application being built/packaged with a valid OAuth clientID & secret, but an invalid API key. The API key isn't needed for OAuth authentication. If the user is authenticated, that authentication token takes priority over the API key. The API key is only needed for unauthenticated access, e.g. access to a public spreadsheet without having the user authenticate. |
Adding a public document via the Google data tab produces error.
To Reproduce
Steps to reproduce the behavior:
Try loading data from a public google document in the latest 3.7 snapshot release.
Current Results
403 Error (see screenshot below)
Expected Behavior
Data should be possible to load even without logging in.
Screenshots
Versions
Additional context
Error first encountered during user testing session on Sept. 30th
The text was updated successfully, but these errors were encountered: