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

plugins.vaughnlive: Update for API changes #1376

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from

Conversation

intact
Copy link
Contributor

@intact intact commented Jul 2, 2016

No description provided.

@intact intact changed the title plugins.vaughnlive: Fix INFO_URL plugins.vaughnlive: Update for API changes Jul 28, 2016
@msikma
Copy link

msikma commented Dec 17, 2016

Unfortunately this is still broken. It claims the following:

[cli][info] Found matching plugin vaughnlive for URL <url>
error: Unable to validate response text: Minimum length is 3 but value is 1

@msikma
Copy link

msikma commented Dec 17, 2016

Seems the URL base has changed again: #1495 (comment)

@intact
Copy link
Contributor Author

intact commented Dec 17, 2016

Latest version works for me (https://github.com/intact/livestreamer/blob/fix-vaughn/src/livestreamer/plugins/vaughnlive.py)

(they added new data into url, but streams work even with old INFO_URL)

@msikma
Copy link

msikma commented Dec 17, 2016

That is weird, I tried your version but for some reason it didn't work. Maybe I messed up pulling your branch somehow.

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.

2 participants