Skip to content
This repository has been archived by the owner on Sep 15, 2020. It is now read-only.

liveleak: change regex to match changed html from liveleak #153

Merged
merged 1 commit into from
Mar 13, 2018
Merged

liveleak: change regex to match changed html from liveleak #153

merged 1 commit into from
Mar 13, 2018

Conversation

youvegottabecrazy
Copy link

  1. Match string changed from res= to label=

  2. I noticed one video with a label="360p". Existing code is matching only label="SD" and label="HD". I changed the SD regex to match either SD or 360p. I'd imagine it may be possible for videos to have both SD and 360p versions listed, and this may match the lesser quality one, but I'd think it'd be rare. (I didn't see any examples of that in a few hundred videos.) Doesn't seem worth the effort of coding another tier. You don't want to see the example video, trust me on this one.

  3. I noticed that it's possible for the video on a liveleak page to be an embedded youtube video. INCEPTION. I'm leaving that one alone. Here is an example, for the record: https://www.liveleak.com/view?i=196_1519606734&ajax=1

@youvegottabecrazy
Copy link
Author

Oh, it's also possible for a LL video page to contain no video. In this case the user gets the standard no compatible streams message. Example of video with no video: https://www.liveleak.com/view?i=785_1516784931&ajax=1

@Protuhj Protuhj merged commit 03d1c0f into Protuhj:master Mar 13, 2018
@Protuhj
Copy link
Owner

Protuhj commented Mar 13, 2018

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants