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

[brightcove] http://www.eitb.tv/es/video/60-minutos-60-minutos-2013-2014/2677100210001/2743577154001/lasa-y-zabala-30-anos/ #1608

Closed
JuanCab opened this issue Oct 16, 2013 · 5 comments
Assignees

Comments

@JuanCab
Copy link

@JuanCab JuanCab commented Oct 16, 2013

Getting an XML parsing error on this one:

youtube-dl --verbose http://www.eitb.tv/es/video/60-minutos-60-minutos-2013-2014/2677100210001/2743577154001/lasa-y-zabala-30-anos/
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', 'http://www.eitb.tv/es/video/60-minutos-60-minutos-2013-2014/2677100210001/2743577154001/lasa-y-zabala-30-anos/']
[debug] youtube-dl version 2013.10.15
[debug] Python version 2.7.5 - Darwin-12.5.0-x86_64-i386-64bit
[debug] Proxy map: {}
WARNING: Falling back on generic information extractor.
[generic] : Downloading webpage
[generic] : Extracting information
[generic] Brightcove video detected.
Traceback (most recent call last):
File "/Users/xoan/anaconda/lib/python2.7/runpy.py", line 162, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/Users/xoan/anaconda/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/usr/local/bin/youtube-dl/main.py", line 18, in
File "/usr/local/bin/youtube-dl/youtube_dl/init.py", line 756, in main

File "/usr/local/bin/youtube-dl/youtube_dl/init.py", line 709, in _real_main

File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 621, in download
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 353, in extract_info
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 117, in extract
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 140, in _real_extract
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/brightcove.py", line 57, in _build_brighcove_url
File "/Users/xoan/anaconda/lib/python2.7/xml/etree/ElementTree.py", line 1300, in XML
parser.feed(text)
File "/Users/xoan/anaconda/lib/python2.7/xml/etree/ElementTree.py", line 1642, in feed
self._raiseerror(v)
File "/Users/xoan/anaconda/lib/python2.7/xml/etree/ElementTree.py", line 1506, in _raiseerror
raise err
xml.etree.ElementTree.ParseError: not well-formed (invalid token): line 15, column 5

phihag added a commit that referenced this issue Oct 16, 2013
@phihag
Copy link
Contributor

@phihag phihag commented Oct 16, 2013

Ok, we're now correcting the invalid XML, but the video is still not playable.

@phihag
Copy link
Contributor

@phihag phihag commented Oct 16, 2013

Looks like they are communicating via AMF (a Flash data interchange protocol). I see multiple requests to http://c.brightcove.com/services/messagebroker/amf?playerKey=AQ~~,AAAAF8Q-iyk~,FDoJSqZe3TTE7xe4JBg40_oRG4rE0toX .

@JuanCab
Copy link
Author

@JuanCab JuanCab commented Oct 21, 2013

It looks like there is enough documentation for AMF for an open implementation to exist (I found http://pyamf.org/ after quick search, although I am not sure if it is limited to the server-side of the equation). Would it be possible to implement support for AMF?

@ghost ghost assigned jaimeMF Nov 6, 2013
@jaimeMF jaimeMF closed this in 5d7b253 Nov 6, 2013
@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Nov 6, 2013

It's fixed now. Thanks for the report!
There's no need to use the AMF calls, we just have to build the correct brightcove url and that's all. I actually think brightcove allways use the AMF protocol when using the flash player.

@phihag
Copy link
Contributor

@phihag phihag commented Nov 6, 2013

This error has been fixed in youtube-dl v2013.11.06.1, which we are releasing at the moment. Type sudo youtube-dl -U to update.

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.