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

[dailymotion] "unable to extract comment count" warning #12209

Closed
4 of 8 tasks
scfc opened this issue Feb 21, 2017 · 2 comments
Closed
4 of 8 tasks

[dailymotion] "unable to extract comment count" warning #12209

scfc opened this issue Feb 21, 2017 · 2 comments

Comments

@scfc
Copy link
Contributor

scfc commented Feb 21, 2017

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.02.21. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

  • I've verified and I assure that I'm running youtube-dl 2017.02.21

Before submitting an issue make sure you have:

  • At least skimmed through README and most notably FAQ and BUGS sections
  • Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

  • Bug report (encountered problems with youtube-dl)
  • Site support request (request for adding support for a new site)
  • Feature request (request for a new functionality)
  • Question
  • Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:

Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):

[tim@passepartout /tmp/tmp.yut8CyFERW]$ ./youtube-dl -v http://www.dailymotion.com/video/x5cnyfr
[debug] System config: [u'--prefer-free-formats']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'http://www.dailymotion.com/video/x5cnyfr']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.02.21
[debug] Python version 2.7.13 - Linux-4.9.9-200.fc25.x86_64-x86_64-with-fedora-25-Twenty_Five
[debug] exe versions: ffmpeg 3.1.7, ffprobe 3.1.7, rtmpdump 2.4
[debug] Proxy map: {}
[dailymotion] x5cnyfr: Downloading webpage
WARNING: unable to extract comment count; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[dailymotion] x5cnyfr: Downloading m3u8 information
[dailymotion] x5cnyfr: Downloading m3u8 information
[dailymotion] x5cnyfr: Downloading m3u8 information
[dailymotion] x5cnyfr: Downloading m3u8 information
[dailymotion] x5cnyfr: Downloading m3u8 information
[dailymotion] x5cnyfr: Downloading m3u8 information
[dailymotion] x5cnyfr: Downloading m3u8 information
[debug] Invoking downloader on u'https://www.dailymotion.com/cdn/H264-1920x1080/video/x5cnyfr.mp4?auth=1487877568-2562-7l2uz9qg-63451a3fef7415ac564a60576bf5b36b'
[download] Resuming download at byte 56119163
[download] Destination: Air Crash Investigation - Killer Attitude (S17E01)-x5cnyfr.mp4
[download]   2.8% of 1.90GiB at  1.33MiB/s ETA 23:37^C
ERROR: Interrupted by user
[tim@passepartout /tmp/tmp.yut8CyFERW]$

Description of your issue, suggested solution and other information

The [dailymotion] downloader warns: "unable to extract comment count".

@yan12125
Copy link
Collaborator

http://www.dailymotion.com/video/x5cnyfr has gone. Any new examples?

@scfc
Copy link
Contributor Author

scfc commented Feb 22, 2017

http://www.dailymotion.com/video/x2sq97f_air-disasters-mayday-munich-season-3-episode-2_tv? IIRC this happens for all Dailymotion videos.

@dstftw dstftw closed this as completed in 527ef85 Feb 22, 2017
Dedsec1 referenced this issue in Dedsec1/youtube-dl Mar 18, 2017
* [pornhub] Extract video URL from tv platform site (#12007, #12129)

* [vod.pl] Add new extractor

* [vodpl] Make more robust and add another test (closes #12122)

* [onetmvp] Add shortcut extractor

* [onetpl] Add support for onet.pl (closes #10507)

* [onetpl] Add support for businessinsider.com.pl and plejada.pl

* [pinkbike] Fix uploader extraction (closes #12054)

* [amcnetworks] fix extraction(closes #12127)

* [utils] Add support for quoted string literals in --match-filter (closes #8050, closes #12142, closes #12144)

* [test_YoutubeDL] Fix invalid escape sequences

* [devscripts/run_tests.sh] Change permission for script to 755

* [ceskatelevize] Prefix format ids

* [ceskatelevize] Lower priority for audio description sources (#12119)

* [ChangeLog] Actualize

* release 2017.02.16

* [elpais] Fix typo and improve extraction (closes #12139)

* [generic] Try parsing JWPlayer embedded videos (closes #12030)

* [einthusan] Relax _VALID_URL (closes #12141, closes #12159)

* [theplatform] Recognize URLs with whitespaces (closes #12044)

* [openload] Semifix extraction (closes #10408)

just updated the code. i don't do much python still i tried to convert my code. lemme know if there is any prob with it

* [openload] Fix extraction (closes #12002)

* [ellentv] Improve

* [heise] Support videos embedded in any article.

* [heise] Improve (closes #9725)

* [options] Add missing spaces

* [ChangeLog] Actualize

* release 2017.02.17

* [viceland] improve info extraction and update test

* [metacafe] Bypass family filter

If you don't send this user=ffilter: false cookie, it will 301 redirect you to a page asking about it, and then the title check will fail.

* [metacafe] Improve (closes #10371)

* [thisav] Add support for html5 media (closes #11771)

* [tvn24] Add extractor

* [tvn24] Improve extraction (closes #11679)

* [options] Mention quoted string literals for --match-filter

* [brightcove:legacy] restrict videoPlayer value(closes #12040)

* [utils] Introduce YoutubeDLError base class for all youtube-dl exceptions

* Add experimental geo restriction bypass mechanism
Based on faking X-Forwarded-For HTTP header

* [dramafever] Improve geo restriction detection and use geo bypass mechanism

* [go] Improve geo restriction detection and use geo bypass mechanism

* [itv] Improve geo restriction detection and use geo bypass mechanism

* [nrk] Improve geo restriction detection and use geo bypass mechanism

* [ondemandkorea] Improve geo restriction detection and use geo bypass mechanism

* [pbs] Improve geo restriction detection and use geo bypass mechanism

* [svt] Improve geo restriction detection and use geo bypass mechanism

* [vbox7] Improve geo restriction detection and use geo bypass mechanism

* [srgssr] Improve geo restriction detection

* [vgtv] Improve geo restriction detection

* [viki] Improve geo restriction detection

* [utils] Make random_ipv4 return unicode string

* Add faked X-Forwarded-For to formats' HTTP headers

* Rename bypass geo restriction options

* Improve geo bypass mechanism
* Rename options to preffixly match with --geo-verification-proxy
* Introduce _GEO_COUNTRIES for extractors
* Implement faking IP right away for sites with known geo restriction

* [options] Move geo restriction related options to separate section

* [downloader/dash] Honor HTTP headers when downloading fragments
For example, https://www.oppetarkiv.se/video/1196142/natten-ar-dagens-mor

* [downloader/ism] Honor HTTP headers when downloading fragments

* [tv4] Switch to hls3 protocol (closes #12177)

* [tv4] Bypass geo restriction and improve detection

* [travis] Run tests in parallel

[test_download] Print test names in case of network errors

[test_download] Add comments for nose parameters

[test_download] Modify outtmpl to prevent info JSON filename conflicts

Thanks @jaimeMF for the idea.

[travis] Only download tests should be run in parallel

* [nrk] Update _API_HOST and relax _VALID_URL

* [prosiebensat1] Throw ExtractionError on unsupported page type (closes #12180)

* [nrk] PEP 8

* [svt] PEP 8

* [commonmistakes] Disable UnicodeBOM extractor test for python 3.2

* [iprima] Modernize

* [iprima] Improve geo restriction detection and disable geo bypass

* [extractor/common] Emphasize geo bypass APIs are experimental

* [spankbang] Make uploader optional (closes #12193)

* [adobepass] add support for Time Warner Cable(closes #12191)

* [ninecninemedia] use geo bypass mechanism

* [extractor/common] Allow calling _initialize_geo_bypass from extractors (#11970)

* [extractor/common] Fix typo

* [extractor/common] Do not quit _initialize_geo_bypass on empty countries

* [extractor/common] Print origin country for fake IP

* [ChangeLog] Actualize

* release 2017.02.21

* [limelight] extract PlaylistService errors

* [telequebec] use geo bypass mechanism

* [cbc] use geo bypass mechanism

* [tfo] Improve geo restriction detection and use geo bypass mechanism

* Revert "[cbc] use geo bypass mechanism"

This reverts commit 86466a8.

* [viewster] use geo verifcation headers

* [crackle] use geo bypass mechanism

* [setup] Add python 3.6 classifier

* [setup] Actualize maintainer info

* [24video] Add support for 24video.tube (closes #12217)

* [vidzi] Add support for vidzi.cc

* [vidzi] Add test for #12213

* [dailymotion] Make comment count optional (closes #12209)
Not served anymore

* [crunchyroll] Fix descriptions with double quotes (closes #12124)

* [ChangeLog] Actualize

* release 2017.02.22

* [leeco] raise GeoRestrictedError and use geo bypass mechanism

* [sohu] raise GeoRestrictedError

* [mgtv] fix extraction

* [crunchyroll] extract playlist entries ids

* [instagram] Add support for multi video posts (closes #12226)

* [skylinewebcams] Add extractor (closes #12221)

* [devscripts/run_tests] Exclude youtube lists tests from core build

* [compat] Add compat_numeric_types

* [YoutubeDL] Add support for string formatting operations in output template

* [YoutubeDL] Rewrite outtmpl for playlist_index and autonumber for backward compatibility

* [YoutubeDL] Add more numeric fields for NA substitution in outtmpl

* [options] Deprecate --autonumber-size

* [README.md] Clarify sequence types in output template and document numeric string formatting operations

* [go] sign all uplynk urls and use geo bypass only for free videos(closes #12087)(closes #12210)

* [devscripts/make_lazy_extractors] Fix making lazy extractors on python 3 under Windows

* [postprocessor/ffmpeg] Add mising space (closes #12232)

* [lynda:course] Add webpage extraction fallback (closes #12238)

* Suppress help for all deprecated options and print warning when used

* [ChangeLog] Actualize

* release 2017.02.24

* [ivi] Raise GeoRestrictedError

* [openload] fix extraction(closes #10408)

* [tubitv] use geo bypass mechanism

* [thescene] Fix extraction (closes #12235)

* [thescene] Extract more metadata

* [noco] Swtich login URL to https (closes #12246)

* [noco] Modernize

* [ChangeLog] Actualize

* release 2017.02.24.1

* [amcnetworks] Relax _VALID_URL (#12127)

* [extractor/common] Add 'preference' to _parse_html5_media_entries

Some websites, like NJPWorld, put different qualities on different
player pages.

* [njpwworld] Add new extractor (closes #11561)

* [compat] Use try except for compat_numeric_types

* [etonline] Add extractor (closes #12236)

* [xhamster] Capture and output videoClosed error (#12263)

* [downloader/common] Limit displaying 2 digits after decimal point in sleep interval message

* [freshlive] Add extractor

* [freshlive] Fix issues and improve (closes #12175)

* [vevo] Fix videos with the new streams/streamsV3 format (closes #11719)

* [tvigle] Raise GeoRestrictedError

* [MDR] Relax _VALID_URL and playerURL matching and update _TESTS

Ref: #12169

* [dailymotion] Raise GeoRestrictedError

* [youtube] Raise GeoRestrictedError

* [crunchyroll] Extract uploader name that's not a link

Provide the Crunchyroll extractor with the ability to extract uploader
names that aren't links. Add a test for this new functionality.
This fixes #12267.

* [cda] Decode URL (fixes #12255)

* [scivee] Remove extractor (#9315)

The Wikipedia page is changed from active to down:
https://en.wikipedia.org/w/index.php?title=SciVee&diff=prev&oldid=723161154

Some other interesting bits:

$ nslookup www.scivee.tv
Server:         8.8.8.8
Address:        8.8.8.8#53

Non-authoritative answer:
www.scivee.tv   canonical name = scivee.rcsb.org.
Name:   scivee.rcsb.org
Address: 132.249.231.211

$ nslookup rcsb.org
Server:         8.8.8.8
Address:        8.8.8.8#53

Non-authoritative answer:
Name:   rcsb.org
Address: 132.249.231.77

Both IPs are from UCSD. I guess it's maintained by a lab and they don't
maintain it anymore.

* [youtube:channel] Remove duplicate test

* [npo] Add support for hetklokhuis.nl (closes #12293)

* [npo] Add support for zapp.nl

* [hetklokhuis] Fix IE_NAME

* [npo] Relax _VALID_URL for zapp.nl

* [ChangeLog] Actualize

* release 2017.02.27

* [douyu] Fix extraction and update _TESTS

They've switched from flv to hls

Closes #12301

* [aes] Add aes_cbc_encrypt

Used in daisuki.net (#4738)

* [utils] Add pkcs1pad

Used in daisuki.net (#4738)

* [utils] Add bytes_to_long() and long_to_bytes()

Used in daisuki.net (#4738)

Both are adapted from public domain PyCrypto:
https://github.com/dlitz/pycrypto/blob/master/lib/Crypto/Util/number.py

* [daisuki] Add new extractor (closes #4738)

* [daisuki] Add subtitles (#4738)

* [youtube:playlist] Recognize another playlist pattern (closes #11928, closes #12286)

* [azmedien:showplaylist] Add support for all episodes playlists

* [azmedien:showplaylist] Improve (closes #12160)

* [ChangeLog] Actualize

* release 2017.02.28

* [npo] Adapt to app.php API (closes #12311)

* [npo] Improve extraction and update tests

* [npo] Add support for audio

* [YoutubeDL] Don't sanitize identifiers (closes #12317)

* [utils] Carry long doc string

* [adobepass] Add Charter Spectrum (#11465)

Thanks @tv21 for the fix!

* [youtube] Mark errors about rental videos as expected

Closes #12324

* [facebook] Fix extraction (closes #12323)

Almost all videos now use the pagelet type 'permalink_video_pagelet'

* [ChangeLog] Actualize

* release 2017.03.02

* [ruutu] Disable DASH formats (closes #12322)
Due to causing out of sync issue

* [24video] Use original host for requests (closes #12339)

* [go] Relax _VALID_URL (closes #12341)

* [brightcove:new] Raise GeoRestrictedError

* [vrak] Add extractor

* [brightcove:new] Add ability to smuggle geo_countries into URL

* [vrak] Improve and update test (closes #11452)

* [rutube] Relax _VALID_URL

* [generic] Add support for rutube embeds

* [douyutv] Switch to the PC API to escape the 5-min limitation

Thanks @spacemeowx2 for the algo.

Ref: https://gist.github.com/spacemeowx2/629b1d131bd7e240a7d28742048e80fc

Closes #12316

* [redbulltv] Add extractor

* [redbulltv] Improve extraction (closes #11948, closes #3919)

* [vier] Add support for vijf.be

vier.be and vijf.be run on the same CMS and are property of the same company,
so the same extractor can be used for both of them.

* Credit @ThomasChr for #12015 and #12245

* Credit @p2004a for vodpl (#12122)

* Credit @TobiX for #9725

* Credit @obilodeau for vrak (#11452)

* Credit @vierbergenlars for vijf.be (#12304)

* [twitch] Add basic support for two-factor authentication

* [twitch] Use better naming and simplify (closes #11974)

* [ChangeLog] Actualize

* release 2017.03.05

* [external:ffmpeg] In test harness, limit to 10k download size

Otherwise, if you screw up a playlist test by including a playlist
dictionary key, you'll be there for eons while it downloads all the
files before erroring out.

* [downloader/external] Add missing import and PEP8

* [extractor/common] Move jwplayer formats extraction in separate method

* [extractor/common] Improve height extraction and extract bitrate

* [extractor/common] Pass arguments to _parse_jwplayer_formats and PEP8

* [tunepk] Add extractor (closes #12197, closes #12243)

* [addanime] Skip an invalid test

* [drtv:live] Bypass geo restriction

* [drtv] Add geo countries to GeoRestrictedError

* [utils] Process bytestrings in urljoin (closes #12369)

* [ChangeLog] Actualize

* release 2017.03.06

* Openload fix extraction (#12357)

* Fix extraction

* [ChangeLog] Update after #12357

* [__init__] Metadata should be added after conversion

Fixes #5594

* [soundcloud] Update client id (closes #12376)

* [ChangeLog] Actualize

* release 2017.03.07

* Credit @benages for redbull.tv (#11948)

* Credit @puxlit for twitch 2fa (#11974)

* [twitch] PEP8

* [brightcove:legacy] Relax videoPlayer validation check (closes #12381)

* [extractor/generic] Add test for brigthcove UUID-like videoPlayer

* [openload] Fix extraction

* [telequebec] Fix description extraction and update test (closes #12399)

* [miomio] Fix extraction

Closes #12291
Closes #12388
Closes #12402

* [dplayit] Separate and rewrite extractor and bypass geo restriction (closes #12393)

* [dplayit] Relax playback info URL extraction

* [prosiebensat1] Improve title extraction (closes #12318)

* [prosiebensat1] Improve title extraction and add test

* [wdr:maus] Fix extraction and update tests

* [wdr] Actualize comment

* [extractor/generic] Make title optional for jwplayer embeds (closes #12410)

* [ChangeLog] Actualize

* release 2017.03.10

* [__init__] Fix missing subtitles if --add-metadata is used (#12423)

The previous fix for #5594 is incorrect

* [discoverygo:playlist] Add extractor (closes #12424)

* [discoverygo] Bypass geo restriction

* [discoverygo] Actualize test

* [crunchyroll] Extract season name

* [streamable] Update API URL

* [openload] Fix extraction

Just a minor fix for openload

* [mitele] Add support for ooyala videos (closes #12430)

* [telecinco] Add test for #12430

* [facebook] Make title optional (closes #12443)

* [Openload] Fixed Extraction

They did changed it again.

* [ChangeLog] Actualize

* release 2017.03.15

* [redbull] improve extraction

- extract 1080p quality
- correct ttml subtitle ext
- catch api errors
- reduce request size

* [extractor/generic] Add forgotten return for jwplayer formats

* [extractor/common] Extract SMIL formats from jwplayer

* [postprocessor/ffmpeg] Add support for flac

Requested at http://stackoverflow.com/q/42828041/35070

* [options] Mention flac support and sort alphabetically among the audio formats

* [ChangeLog] Actualize

* release 2017.03.16

* [test_InfoExtractor] Add some realworld tests for _extract_jwplayer_data

* [openload] Fix extraction (closes #10408)

Thanks to @makgun02

Ref: http://pastebin.com/raw/JX9gHFUz

* [discoverynetworks] add support for more domains and bypass geo restiction

* [vtm] Add extractor (closes #9974)

Implementation of the approach described in #9974.

* [medialaan] Fix and improve extraction (closes #11912)

* [openload] Fix extraction

* [adobepass] Detect and output error on authz token extraction (#12472)

* [medialaan] Remove unrelated test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants