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

Feature request: support Syfy videos of the form http://www.syfy.com/wilwheaton #2827

Closed
jillh510 opened this issue Apr 28, 2014 · 2 comments
Closed

Comments

@jillh510
Copy link

@jillh510 jillh510 commented Apr 28, 2014

Looks like the id is available in the source:

<iframe frameborder="0" scrolling="no" width="688" height="387" class="video_iframe_page" src="/_utils/video/thP_video_controller.php?sect=wilwheaton&/video/medium/autoplay/_vid2771787"></iframe>

(the id is 2771787, which works fine if I make this fake URL:
http://www.syfy.com/videos/Blastr/Blastr%20TV/vid:2771787)

@pdball
Copy link

@pdball pdball commented Apr 29, 2014

That URL works with this change:

diff --git a/youtube_dl/extractor/theplatform.py b/youtube_dl/extractor/theplatform.py
index 91f2453..2d69513 100644
--- a/youtube_dl/extractor/theplatform.py
+++ b/youtube_dl/extractor/theplatform.py
@@ -53,6 +53,8 @@ class ThePlatformIE(InfoExtractor):
body = meta.find(_x('smil:body'))

     f4m_node = body.find(_x('smil:seq/smil:video'))
  •    if f4m_node is None:
    
  •        f4m_node = body.find(_x('smil:seq/smil:par/smil:video'))
     if f4m_node is not None:
         f4m_url = f4m_node.attrib['src']
         if 'manifest.f4m?' not in f4m_url:
    
@phihag
Copy link
Contributor

@phihag phihag commented Apr 29, 2014

Thanks for the report. Fixed in youtube-dl 2014.04.30.

@phihag phihag closed this in 22d99a8 Apr 30, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.