search

cosmocode / dokuwiki-plugin-oauth

Generic oAuth1 and oAuth2 plugin for DokuWiki
http://www.dokuwiki.org/plugin:oauth
30 stars 78 forks source link

Update GitHub authorization to use token in header instead of query #90

Closed thess closed 4 years ago

thess commented 4 years ago

GitHub has deprecated the use of authorization via the use of a query parameter and instead wants the token passed in the header. The enclosed patch is an attempt to work-around this issue. It has been tested on our dokuwiki site for about 4 weeks now. Authorization seems to be working and we have had no observable problems. Perhaps someone else can give this a try and hopefully merge it or its equivalent into the upstream base.

splitbrain commented 4 years ago

I'm merging this to fix the immediate problem. In the long run we probably should upgrade the underlying library.