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
Upgrade omniauth-github to fix Github login deprecation notice #6022
Upgrade omniauth-github to fix Github login deprecation notice #6022
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for including the links in the description! LGTM 💯
Sorry @vaidehijoshi, I forgot about the issue :( Going to update the description so it automatically fixes it! |
@rhymes thanks so much for doing this! i just got that GitHub email again yesterday, so this is really great timing 🤗 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks okay to me! I didn't test it out locally but looking at the other changes in 1.4.0
, they appear relatively minor aside from the headers change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What type of PR is this? (check all applicable)
Description
Recently both @ludwiczakpawel and I received a deprecation email from GitHub as they are changing the way the HTTP clients are supposed to send authentication params (yes to headers, no to URL params).
This is the deprecation notice:
Tested this manually, it works
NOTE: Octokit, Github's client, has already been upgraded to a correct version: https://github.com/octokit/octokit.rb/releases/tag/v4.16.0
Related Tickets & Documents
Closes #5899