-
Notifications
You must be signed in to change notification settings - Fork 314
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
How should authentication work? #692
Comments
@insanum @michaelmhoffman can I get your input on known quirks in the existing auth setup, any context on how any existing Google project is set up, and LMK any bad assumptions I'm making above? Do you know if the default auth is supposed to work somehow if you don't pass |
I checked and the similar gmailctl project that I use for Gmail integrations has the same manual setup process: https://github.com/mbrt/gmailctl/blob/69c24b795402ff895cc90262b3cbdebd501c8b53/cmd/gmailctl/localcred/local_provider.go#L21. I suspect that's the best option available. But still would be helpful to publish a doc about the howto & why for users. |
This is now much improved as of 4.4-pre, rolling over into 4.5 for the rest. I'm almost positive there's no workable alternative to the self-service auth setup, which is the same as most other open source projects seem to be doing. #572 mentioned a PKCE mechanism, but I suspect that wouldn't work for "Testing" auth clients either, and I haven't confirmed yet but I suspect getting a "Production" client and sharing it with all users entails paying Google to give it high enough limits for all the users. I created https://github.com/insanum/gcalcli/blob/HEAD/docs/api-auth.md with more explanations, moved some of the details from the README into there, and linked it from the app and README. |
K, overall status here is I'm pretty sure there's no good easy alternative to the auth setup we have now, but I forked off #759 to still keep an eye out for better options. Still don't understand how PKCE would fit into this equation, asked for clarification on #572 but didn't get a response. If anyone knows, please follow up there... And with that, I'm going to call this issue "Done". The current setup has some shortcomings, but have an overall sense for how it can/should work, and so far nobody's volunteered any categorically better alternatives. |
High-level tracking issue to collect together some context on auth issues from various issue reports and figure out the direction we want to go.
I've seen a few flavors of issues:
Overall I believe auth is generally functional again now that we've switched off of the deprecated oauth2client dep in #683, but only if you carefully follow the setup instructions from the README and don't trip over some huge gotchas.
The text was updated successfully, but these errors were encountered: