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

Removed dead plugins #702

Merged
merged 1 commit into from Mar 18, 2017
Merged

Removed dead plugins #702

merged 1 commit into from Mar 18, 2017

Conversation

back-to
Copy link
Collaborator

@back-to back-to commented Mar 13, 2017

FILES URL STATUS
bliptv.py http://blip.tv REDIRECT to different website
gaminglive.py http://gaminglive.tv DEAD ?? REDIRECT (401 Authorization Required)
letontv.py http://leton.tv DEAD
livestation.py http://www.livestation.com DEAD
meerkat.py http://meerkatapp.co REDIRECT to different website
veetle.py http://veetle.com DEAD
viagame.py http://www.viagame.com REDIRECT to different website

| FILES           | URL                        | STATUS |
| ---             | ---                        | ---    |
| bliptv.py       | http://blip.tv             | **REDIRECT** to different website |
| gaminglive.py   | http://gaminglive.tv       | **DEAD ?? REDIRECT** (401 Authorization Required) |
| letontv.py      | http://leton.tv            | **DEAD** |
| livestation.py  | http://www.livestation.com | **DEAD** |
| meerkat.py      | http://meerkatapp.co       | **REDIRECT** to different website |
| veetle.py       | http://veetle.com          | **DEAD** |
| viagame.py      | http://www.viagame.com     | **REDIRECT** to different website |
@codecov
Copy link

codecov bot commented Mar 13, 2017

Codecov Report

Merging #702 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #702      +/-   ##
==========================================
- Coverage   48.56%   48.54%   -0.02%     
==========================================
  Files          69       69              
  Lines        5846     5840       -6     
==========================================
- Hits         2839     2835       -4     
+ Misses       3007     3005       -2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 192571b...3fb7e1c. Read the comment docs.

@beardypig
Copy link
Member

Either than or mark them as broken.

@back-to
Copy link
Collaborator Author

back-to commented Mar 15, 2017

Didn't know that there was a broken method

But what would be the point to have broken plugins,
when they will never work again because there websites are down/closed.

The broken method would work great,
if the website would be still up but with DRM DASH or something that streamlink does not support.

@gravyboat
Copy link
Member

Yeah I agree with @back-to. In this event the sites are definitely down and not broken.

@beardypig
Copy link
Member

I dunno, it was more to inform the user. Like if they had URLs that they saved or whatever. But saying the URL is not supported is also reasonable.

@beardypig beardypig merged commit c8a4639 into streamlink:master Mar 18, 2017
@back-to back-to deleted the dead_plugins branch March 18, 2017 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants