bump omniauth-github to stop sending access-token via query params

[grosser]

github is sending emails about it "Please use the Authorization HTTP header instead as using the access_token query parameter is deprecated.
"

confirmed working locally by printing in config/initializers/faraday.rb

@zendesk/compute

[patrobinson]

I don't think this works, because omniauth-github has not released a new gem version for this change. The current maintainers are waiting for someone to give them access omniauth/omniauth-github#87 so we may have to use the github source.

[grosser]

there might be more bugs in there, but I can see it clearly not working in 925b687 and being better after the update 🤷‍♂
PR welcome if you can find another area where it uses the query

[patrobinson]

The code to fix this was only committed yesterday omniauth/omniauth-github#84

Github only send you an email every 3 days it says so maybe it'll come through then?

[grosser]

yeah, could be that it just changed something and now it no longer logs ... we'll see when it comes back, if it does then it's time to use it github :(

…

On Thu, Feb 6, 2020 at 3:55 PM Patrick Robinson ***@***.***> wrote: The code to fix this was only committed yesterday omniauth/omniauth-github#84 <omniauth/omniauth-github#84> Github only send you an email every 3 days it says so maybe it'll come through then? — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#3713?email_source=notifications&email_token=AAACYZ6KPZ76T6WBBP4ZTATRBSPNVA5CNFSM4KQTLA22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELBHHLI#issuecomment-583168941>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAACYZYOVMBHIULSVXR5WLDRBSPNVANCNFSM4KQTLA2Q> .

[grosser]

yeah, when I add logging deeper down it shows that it still uses params ... time for git dependency ...