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

More OAuth2 Providers #21

Closed
wants to merge 3 commits into from
Closed

More OAuth2 Providers #21

wants to merge 3 commits into from

Conversation

riptl
Copy link

@riptl riptl commented Feb 11, 2019

Twitch (#19)
OAuth documentation
Getting current user (Same for APIv5 and New API)

@riptl riptl changed the title Add Twitch OAuth2 Provider More OAuth2 Providers Feb 11, 2019
@riptl
Copy link
Author

riptl commented Feb 11, 2019

Twitter (#16)
OAuth documentation
Getting current user

@riptl
Copy link
Author

riptl commented Feb 12, 2019

rip indent f6c4a49 GitHub merge tool is useless

@riptl
Copy link
Author

riptl commented Feb 17, 2019

No need for PR anymore, this is best done manually.

Twitch

  • Authorize https://id.twitch.tv/oauth2/authorize
  • Token https://id.twitch.tv/oauth2/token
  • Scope user_read
  • Current User https://api.twitch.tv/kraken/user
  • Username Field name
  • Username Prefix: none

Twitter

  • Authorize https://api.twitter.com/oauth/authorize
  • Token https://api.twitter.com/oauth/access_token
  • Scope: none
  • Current User https://api.twitter.com/1.1/account/verify_credentials.json?skip_status=true&include_email=false
  • Username Field screen_name
  • Username Prefix: none

@riptl riptl closed this Feb 17, 2019
@riptl riptl deleted the patch-1 branch February 17, 2019 02:38
@nektro
Copy link
Owner

nektro commented Feb 17, 2019

The reason Twitch and Twitter have not been implemented is because of their non-standard OAuth2 flows., Although I really do appreciate you finding the proper endpoints, it really helped.

When using the Twitter endpoints, they request an extra header with a value that seems Twitter specific and I am still looking into it.

For Twitch the problem is 2 fold in both public versions of their API dubbed kracken and helix respectively. The kracken API requires a non-standard use of the Authorization header at the access_token endpoint. Usually this wouldn't be a problem, but a use of the Authorization header has already been added in the way that the header is used. And when using the helix API, the /user endpoint has been removed and only exposes /users which wraps the JSON output in an array.

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