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

IPrima broken #7229

Closed
10 of 11 tasks
jirih opened this issue Jun 4, 2023 · 5 comments · Fixed by #7216
Closed
10 of 11 tasks

IPrima broken #7229

jirih opened this issue Jun 4, 2023 · 5 comments · Fixed by #7216
Labels
account-needed Account details are needed to test/fix this can-share-account Someone is willing to provide account details for development site-bug Issue with a specific website

Comments

@jirih
Copy link

jirih commented Jun 4, 2023

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

  • I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

Region

world

Provide a description that is worded well enough to be understood

IPrima website has probably recently changed and yt-dlp does not parse it correctly anymore. Downloading video using directly mpd works.

The video used in this report should be accessible from the whole world (quite surely at least EU).

User account needed, I can provide credentials. (However, only working email is required for a creation of the account, so anybody can create own.)

Provide verbose output that clearly demonstrates the problem

  • Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
  • If using API, add 'verbose': True to YoutubeDL params instead
  • Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output

[debug] Command-line config: ['-u', 'PRIVATE', '-p', 'PRIVATE', '-vU', 'https://www.iprima.cz/serialy/satna/satna/5-epizoda-5']
[debug] User config "C:\Users\jiri\yt-dlp.conf": ['--user-agent', 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/94.0.4606.71 Safari/537.36', '--no-check-certificate', '--sponsorblock-remove', 'all', '-o', 'D:\\Downloads\\yt-dlp\\%(autonumber)03d_%(title)s.%(ext)s', '--no-mtime', '--format', 'bestvideo[ext=mp4][height<=1080][fps<=30]+bestaudio[ext=m4a]/best[ext=mp4][height<=1080][fps<=30]/best[height<=1080][fps<=30]/best', '--merge-output-format', 'mkv', '--ffmpeg-location', 'C:\\Apps\\ffmpeg\\bin\\ffmpeg.exe', '--all-subs', '-i', '--compat-options', 'multistreams', '--remux-video', 'mkv', '--add-header', 'accept-language: cs,fi;q=0.9,en-US;q=0.8,en;q=0.7,en-GB;q=0.6', '--add-header', 'sec-ch-ua: "Chromium";v="94", "Google Chrome";v="94", ";Not A Brand";v="99"', '--add-header', 'sec-ch-ua-arch: "x86"', '--add-header', 'sec-ch-ua-bitness: "64"', '--add-header', 'sec-ch-ua-full-version: "94.0.4606.71"', '--add-header', 'sec-ch-ua-mobile: ?0', '--add-header', 'sec-ch-ua-model: ""', '--add-header', 'sec-ch-ua-platform: "Windows"', '--add-header', 'sec-ch-ua-platform-version: "10.0.0"', '--add-header', 'sec-fetch-dest: document', '--add-header', 'sec-fetch-mode: navigate', '--add-header', 'sec-fetch-site: none', '--add-header', 'sec-fetch-user: ?1', '--add-header', 'service-worker-navigation-preload: true', '--add-header', 'upgrade-insecure-requests: 1', '--add-header', 'alt-svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000,h3-T051=":443"; ma=2592000,h3-Q050=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,quic=":443"; ma=2592000; v="46,43"', '--add-header', 'cache-control: no-cache, no-store, max-age=0, must-revalidate']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version stable@2023.03.04 [392389b7d] (win_exe)
[debug] Compatibility options: multistreams
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 4.4-full_build-www.gyan.dev (setts), ffprobe 4.4-full_build-www.gyan.dev, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.17, brotli-1.0.9, certifi-2022.12.07, mutagen-1.46.0, sqlite3-2.6.0, websockets-10.4
[debug] Proxy map: {}
[debug] Loaded 1786 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version: stable@2023.03.04, Current version: stable@2023.03.04
Current Build Hash: 5590c57bd0433ed239a2deaaf92e2ad6f37fe50f53664c821575cafe106a9421
yt-dlp is up to date (stable@2023.03.04)
[IPrima] Downloading login page
[IPrima] Logging in
[IPrima] Downloading token
[IPrima] Extracting URL: https://www.iprima.cz/serialy/satna/satna/5-epizoda-5
[IPrima] 5-epizoda-5: Downloading webpage
ERROR: [IPrima] 5-epizoda-5: None: Failed to parse JSON (caused by JSONDecodeError('Expecting \',\' delimiter in \'5","Array"(0),"satna\': line 1 column 498 (char 497)')); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "yt_dlp\extractor\common.py", line 694, in extract
  File "yt_dlp\extractor\iprima.py", line 137, in _real_extract
  File "yt_dlp\extractor\common.py", line 1674, in _search_nuxt_data
  File "yt_dlp\extractor\common.py", line 995, in _parse_json
  File "yt_dlp\extractor\common.py", line 978, in __print_error

  File "yt_dlp\utils.py", line 623, in decode
  File "json\decoder.py", line 337, in decode
  File "json\decoder.py", line 353, in raw_decode
json.decoder.JSONDecodeError: Expecting ',' delimiter: line 1 column 498 (char 497)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "yt_dlp\extractor\common.py", line 992, in _parse_json
  File "json\__init__.py", line 370, in loads
  File "yt_dlp\utils.py", line 631, in decode
json.decoder.JSONDecodeError: Expecting ',' delimiter in '5","Array"(0),"satna': line 1 column 498 (char 497)
@std-move
Copy link
Contributor

std-move commented Jun 4, 2023

Should be fixed by changes in #7216 that are yet to be merged however

@jirih
Copy link
Author

jirih commented Jun 4, 2023

Should be fixed by changes in #7216 that are yet to be merged however

Nice to read!
I have tried the change and it helped.

@bashonly bashonly removed the triage Untriaged issue label Jun 4, 2023
@jirih
Copy link
Author

jirih commented Jun 30, 2023

Should be fixed by changes in #7216 that are yet to be merged however

Nice to read! I have tried the change and it helped.

Hmmm, apparently it is broken in a new way again. The provided fix #7216 does not work anymore.

Following output is from the current master with the fix in question.

[debug] Command-line config: ['-u', 'PRIVATE', '-p', 'PRIVATE', '-vU', 'https://www.iprima.cz/serialy/particka-xxl/prima-particka-2022-xxl/32-epizoda-32']
[debug] User config "/home/pi/.config/yt-dlp/config": ['--user-agent', 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/92.0.4515.131 Safari/537.36', '--no-check-certificate', '-o', '~/Downloads/yt-dlp/%(autonumber)03d_%(title)s.%(ext)s', '--no-mtime', '--format', 'bestvideo[ext=mp4][height<=1080][fps<=30]+bestaudio[ext=m4a]/best[ext=mp4][height<=1080][fps<=30]/best[height<=1080][fps<=30]/best', '--merge-output-format', 'mkv', '--remux-video', 'mkv', '--ffmpeg-location', '/usr/bin/ffmpeg', '--all-subs', '--compat-options', 'no-live-chat', '--live-from-start', '-i']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out UTF-8, error UTF-8, screen UTF-8
[debug] yt-dlp version stable@2023.06.22 [812cdfa06] (source)
[debug] Lazy loading extractors is disabled
[debug] Compatibility options: no-live-chat
[debug] Git HEAD: af1fd12f6
[debug] Python 3.7.3 (CPython armv7l 32bit) - Linux-5.15.87-v7l+-armv7l-with-debian-10.13 (OpenSSL 1.1.1n  15 Mar 2022, glibc 2.28)
[debug] exe versions: ffmpeg 4.1.11-0, ffprobe 4.1.11-0, phantomjs broken, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2021.10.08, mutagen-1.46.0, secretstorage-3.3.1, sqlite3-2.6.0, websockets-10.0, xattr-0.10.1
[debug] Proxy map: {}
[debug] Loaded 1854 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version: stable@2023.06.22, Current version: stable@2023.06.22
yt-dlp is up to date (stable@2023.06.22)
[IPrima] Downloading login page
[IPrima] Logging in
[IPrima] Downloading token
[IPrima] Extracting URL: https://www.iprima.cz/serialy/particka-xxl/prima-particka-2022-xxl/32-epizoda-32
[IPrima] 32-epizoda-32: Downloading webpage
ERROR: [IPrima] 32-epizoda-32: Unable to extract __NUXT__; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "/home/pi/git/yt-dlp-iprima-fix/yt_dlp/extractor/common.py", line 708, in extract
    ie_result = self._real_extract(url)
  File "/home/pi/git/yt-dlp-iprima-fix/yt_dlp/extractor/iprima.py", line 137, in _real_extract
    nuxt_data = self._search_nuxt_data(webpage, video_id, traverse='data')
  File "/home/pi/git/yt-dlp-iprima-fix/yt_dlp/extractor/common.py", line 1688, in _search_nuxt_data
    default=NO_DEFAULT if fatal else (None, None, None))
  File "/home/pi/git/yt-dlp-iprima-fix/yt_dlp/extractor/common.py", line 1256, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)

@std-move
Copy link
Contributor

std-move commented Jul 1, 2023

Should be fixed now, please pull the latest updates from the PR branch and verify that downloading from iPrima works again

@jirih
Copy link
Author

jirih commented Jul 1, 2023

Should be fixed now, please pull the latest updates from the PR branch and verify that downloading from iPrima works again

Works ok again. Thanks.

@bashonly bashonly added account-needed Account details are needed to test/fix this can-share-account Someone is willing to provide account details for development labels Jul 9, 2023
bashonly pushed a commit that referenced this issue Sep 21, 2023
Closes #7229
Authored by: std-move
aalsuwaidi pushed a commit to aalsuwaidi/yt-dlp that referenced this issue Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
account-needed Account details are needed to test/fix this can-share-account Someone is willing to provide account details for development site-bug Issue with a specific website
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants