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

Afreecatv has failed #12179

Closed
itttttta opened this issue Feb 19, 2017 · 7 comments
Closed

Afreecatv has failed #12179

itttttta opened this issue Feb 19, 2017 · 7 comments
Assignees
Labels
bug

Comments

@itttttta
Copy link

@itttttta itttttta commented Feb 19, 2017

C:\Users\wd\AppData\Local\Programs\Python\Python35-32\python.exe C:/Users/wd/Desktop/youtube-dl-master/youtube_dl/main.py http://vod.afreecatv.com/PLAYER/STATION/18650793
[afreecatv] 18650793: Downloading XML
ERROR: Specified AfreecaTV video does not exist

Process finished with exit code 1

@yan12125 yan12125 added the bug label Feb 19, 2017
@mcerdeira
Copy link

@mcerdeira mcerdeira commented Feb 19, 2017

Seems like it is expecting the video url in the path "track/video/file" from the downloaded xml, but it is does not exists.
This is the XML it parses and gets now:
http://afbbs.afreecatv.com:8080/api/video/get_video_info.php?nTitleNo=18650793
link xml
get_video_info.php.xml.zip

If I have to guess, the xml has changed (compared to the version it was developed for) and now the link is here:
<video duration="107">rtmp://flv10.afreecatv.com/afflv10/mp4:AFFLV/00/7/29/10_00_1487406867466338_2147483647.mp4

@mcerdeira
Copy link

@mcerdeira mcerdeira commented Feb 20, 2017

@itttttta
Copy link
Author

@itttttta itttttta commented Feb 20, 2017

Yeah , you are right .

Right xml:

`This XML file does not appear to have any style information associated with it. The document tree is shown below.

tjdbs0274 SUCCEED http://stimg.afreecatv.com/LOGO/tj/tjdbs0274/tjdbs0274.jpg <title> </title> 149 http://videoimg.afreecatv.com/php/SnapshotLoad.php?rowKey=20170218_F8599C69_188956549_1_r http://ad.afreecatv.com/RealMedia/ads/adstream_sx.ads/www.afreeca.com/VOD_21c@x67 http://afbbs.afreecatv.com:8080/api/video/set_video_log.php http://222.122.117.204/mvod/20170218/549/F8599C69_188956549_1.mp4 false http://vod.afreecatv.com/PLAYER/STATION/18650795 <iframe> </iframe> ]]> </iframe> `

Faild Xml:

`This XML file does not appear to have any style information associated with it. The document tree is shown below.

badkids SUCCEED http://stimg.afreecatv.com/LOGO/ba/badkids/badkids.jpg <title> </title> 107 http://iflv10.afreecatv.com/AFFLV/00/7/29/10_00_1487406867466338_2147483647.jpg http://ad.afreecatv.com/RealMedia/ads/adstream_sx.ads/www.afreeca.com/VOD_8c@x67 rtmp://flv10.afreecatv.com/afflv10/mp4:AFFLV/00/7/29/10_00_1487406867466338_2147483647.mp4 http://afbbs.afreecatv.com:8080/api/video/set_video_log.php false false http://vod.afreecatv.com/PLAYER/STATION/18650793 <iframe> </iframe> ]]> </iframe> `

However,there are a lot of faild xml files.

@mcerdeira
Copy link

@mcerdeira mcerdeira commented Feb 20, 2017

Yeah, I don´t know if the xml is failed, seems to be just another format... (also, the "new" xml contains a video link but rtmp protocol) may be this is silly to ask but: is there any specification from afreecatv?

@itttttta
Copy link
Author

@itttttta itttttta commented Feb 20, 2017

Sorry,i don't know.I test a few urls .There are just two xml styles above .One is rtmp video ,i can not download this video by youtube-dl.

@mcerdeira
Copy link

@mcerdeira mcerdeira commented Feb 20, 2017

Ok, so I think that we need to test for method 1, and then, if the tag is not present, go for method 2, similar to my proposed fix (dirty hack) here using a flag (or not): https://gist.github.com/mcerdeira/cf83dedf6ba553b5e34ad5b5e8371f31#file-gistfile1-py-L115

        if xpath_element(video_xml, './track/video/file') is None:
            method_b = True
            #raise ExtractorError('Specified AfreecaTV video does not exist',
#                     expected=True)
@yan12125 yan12125 self-assigned this Mar 24, 2017
@yan12125 yan12125 closed this in 51ef491 Mar 25, 2017
@yan12125
Copy link
Collaborator

@yan12125 yan12125 commented Mar 25, 2017

Sorry for being late. I've pushed the fix to master. Thanks for identifying and diagnosing the problem, @mcerdeira and @ivanda! The fix will be included in the next version.

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.