Skip to content

Commit

Permalink
[extractor/youtube] Fix 5038f6d
Browse files Browse the repository at this point in the history
* [fragment] Fix `request_data`
* [youtube] Don't use POST for now. It may be easier to break in future

Authored by: bashonly, coletdjnz
  • Loading branch information
pukkandan committed Feb 28, 2023
1 parent 65f6e80 commit f34804b
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion yt_dlp/downloader/fragment.py
Expand Up @@ -466,7 +466,8 @@ def error_callback(err, count, retries):
for retry in RetryManager(self.params.get('fragment_retries'), error_callback):
try:
ctx['fragment_count'] = fragment.get('fragment_count')
if not self._download_fragment(ctx, fragment['url'], info_dict, headers):
if not self._download_fragment(
ctx, fragment['url'], info_dict, headers, info_dict.get('request_data')):
return
except (urllib.error.HTTPError, http.client.IncompleteRead) as err:
retry.error = err
Expand Down
1 change: 1 addition & 0 deletions yt_dlp/extractor/common.py
Expand Up @@ -132,6 +132,7 @@ class InfoExtractor:
is parsed from a string (in case of
fragmented media)
for MSS - URL of the ISM manifest.
* request_data Data to send in POST request to the URL
* manifest_url
The URL of the manifest file in case of
fragmented media:
Expand Down
1 change: 0 additions & 1 deletion yt_dlp/extractor/youtube.py
Expand Up @@ -3778,7 +3778,6 @@ def _extract_formats_and_subtitles(self, streaming_data, video_id, player_url, l
if no_audio or no_video:
CHUNK_SIZE = 10 << 20
dct.update({
'request_data': b'x',
'protocol': 'http_dash_segments',
'fragments': [{
'url': update_url_query(dct['url'], {
Expand Down

0 comments on commit f34804b

Please sign in to comment.