-
Notifications
You must be signed in to change notification settings - Fork 135
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
Support verification_url
mispelling in the device auth response
#565
Comments
Hi, no I have nothing against it. As the oauth2 implementation was released shortly after ours, I was actually thinking if we can somehow reuse some of their definitions and cleanup our code. Perhaps the response object would be a good start. |
Indeed the |
I can create a PR just reusing the |
Hi, sorry for the late reply. I gave it another thought: Changing the response type is a breaking change, and we want to prevent that on We are open for a PR. |
Thanks for the answer, I had the same feeling. I'll do a PR tomorrow ;) |
🎉 This issue has been resolved in version 3.20.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Preflight Checklist
Describe your problem
Hello, any chance that this lib would implement this kind of workaround implemented by golang/oauth2:
https://github.com/golang/oauth2/blob/master/deviceauth.go#L58-L78
I am implementing a client for google API using zitadel/oidc lib but they are using verification_url instead of uri ...
https://stackoverflow.com/q/76696956/5690223
https://developers.google.com/identity/protocols/oauth2/limited-input-device?hl=fr#success-response
Describe your ideal solution
I have no better solution in head than the workaround described
Version
3.14.0
Additional Context
No response
The text was updated successfully, but these errors were encountered: