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

#7430 Follow up (clean_podcast_url needs more improvement) #7544

Closed
10 tasks done
mabdelfattah opened this issue Jul 8, 2023 · 0 comments · Fixed by #7556
Closed
10 tasks done

#7430 Follow up (clean_podcast_url needs more improvement) #7544

mabdelfattah opened this issue Jul 8, 2023 · 0 comments · Fixed by #7556
Labels
bug Bug that is not site-specific

Comments

@mabdelfattah
Copy link
Contributor

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

  • I'm reporting a bug unrelated to a specific site
  • I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
  • I've checked that all provided URLs are playable in a browser with the same IP and same login details
  • I've checked that all URLs and arguments with special characters are properly quoted or escaped
  • I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
  • I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood

This is a follow-up ticket on issue #7430

One more URL that is damaged also by the same method.

https://pdst.fm/e/2.gum.fm/chtbl.com/track/chrt.fm/track/34D33/pscrb.fm/rss/p/traffic.megaphone.fm/ITLLC7765286967.mp3?updated=1687282661

Is it better to add a flag to ignore clean_podcast_url()?

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

-
@mabdelfattah mabdelfattah added bug Bug that is not site-specific triage Untriaged issue labels Jul 8, 2023
@bashonly bashonly linked a pull request Jul 10, 2023 that will close this issue
9 tasks
@bashonly bashonly changed the title #7430 Follow up #7430 Follow up (clean_podcast_url needs more improvement) Jul 10, 2023
pukkandan pushed a commit that referenced this issue Jul 11, 2023
Authored by: mabdelfattah, bashonly
Closes #7544
@bashonly bashonly removed the triage Untriaged issue label Jul 11, 2023
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
bug Bug that is not site-specific
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants