-
-
Notifications
You must be signed in to change notification settings - Fork 6k
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
extractor/npo.py API query now needs to be POST, not GET #6398
Comments
the video is DRM protected. |
no the way how the API works has changed since monday. This Kodi video add-on was updated yeseterday |
OP's example URL is DRM-protected, but @VDK is correct that the API is requiring POST requests even for non-DRM content, e.g. https://www.npo3.nl/the-genius/21-11-2022/VPWON_1341105 diff --git a/yt_dlp/extractor/npo.py b/yt_dlp/extractor/npo.py
index 40fee24d0..9150b91c5 100644
--- a/yt_dlp/extractor/npo.py
+++ b/yt_dlp/extractor/npo.py
@@ -245,7 +245,7 @@ def _real_extract(self, url):
'quality': 'npoplus',
'tokenId': player_token,
'streamType': 'broadcast',
- })
+ }, data=b'')
if not streams:
continue
stream = streams.get('stream') with the above patch applied:
without:
this is likely not a complete fix though, as some of the other tests in the extractor still fail |
I can confirm that the patch works. |
Seems like the playlist IEs inherit from |
Also:
|
Closes #6398 Authored by: bartbroere
Closes yt-dlp#6398 Authored by: bartbroere
DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE
Checklist
Region
Netherlands
Provide a description that is worded well enough to be understood
npo.py line 240 has a json request that needs to be done with a POST method, where it previously was a GET.
Provide verbose output that clearly demonstrates the problem
yt-dlp -vU <your command line>
)'verbose': True
toYoutubeDL
params instead[debug] Command-line config
) and insert it belowComplete Verbose Output
The text was updated successfully, but these errors were encountered: