Skip to content
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

Fix omniauth-oauth2 v1.4 compatibility #2

Merged
merged 2 commits into from Mar 16, 2016
Merged

Fix omniauth-oauth2 v1.4 compatibility #2

merged 2 commits into from Mar 16, 2016

Conversation

tsubery
Copy link
Contributor

@tsubery tsubery commented Feb 13, 2016

Version 1.4 sends the full url (including query string) when validating tokens. This is incompatible with beeminder.com api.

Version 1.4 sends the full url (including query string) when validating
tokens. This incompatible with beeminder.com api.
@tsubery tsubery changed the title Add version limit to omniauth-oauth2 gem omniauth-oauth2 gem Feb 27, 2016
@tsubery tsubery changed the title omniauth-oauth2 gem Fix omniauth-oauth2 v1.4 compatibility Feb 27, 2016
@tsubery
Copy link
Contributor Author

tsubery commented Mar 16, 2016

Hi @dreeves, can you merge this?
I want to remove my local fork

dreeves added a commit that referenced this pull request Mar 16, 2016
Fix omniauth-oauth2 v1.4 compatibility
@dreeves dreeves merged commit e77759c into beeminder:master Mar 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants