Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix 404 error on twitch oauth authorize url#1856

Merged
gravyboat merged 1 commit into
streamlink:masterfrom
NyanKiyoshi:fix/twitch/oauth-authorize-url
Jun 27, 2018
Merged

Fix 404 error on twitch oauth authorize url#1856
gravyboat merged 1 commit into
streamlink:masterfrom
NyanKiyoshi:fix/twitch/oauth-authorize-url

Conversation

@NyanKiyoshi

Copy link
Copy Markdown
Contributor

This PR fixes the 404 error when user request a oauth token for Twitch by removing the trailing slash from the endpoint.

How to reproduce

  1. streamlink --twitch-oauth-authenticate
  2. Twitch should answer with a HTTP 404 error.

@codecov

codecov Bot commented Jun 26, 2018

Copy link
Copy Markdown

Codecov Report

Merging #1856 into master will not change coverage.
The diff coverage is 0%.

@@          Coverage Diff           @@
##           master   #1856   +/-   ##
======================================
  Coverage    50.5%   50.5%           
======================================
  Files         239     239           
  Lines       14130   14130           
======================================
  Hits         7136    7136           
  Misses       6994    6994

@beardypig beardypig left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, thanks @NyanKiyoshi :)

@gravyboat

Copy link
Copy Markdown
Member

Thanks @NyanKiyoshi!

@gravyboat gravyboat merged commit f8f2c9c into streamlink:master Jun 27, 2018
@gravyboat gravyboat added the bug label Jun 27, 2018
@NyanKiyoshi NyanKiyoshi deleted the fix/twitch/oauth-authorize-url branch June 27, 2018 01:30
mkbloke pushed a commit to mkbloke/streamlink that referenced this pull request Aug 18, 2020
…authorize-url

Fix 404 error on twitch oauth authorize url
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants