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

Can no longer specify location of m3u due to sanitize_string #2224

Closed
dyay108 opened this issue Nov 9, 2024 · 2 comments
Closed

Can no longer specify location of m3u due to sanitize_string #2224

dyay108 opened this issue Nov 9, 2024 · 2 comments
Assignees
Labels
Bug Unexpected problem or unintended behavior that needs to be fixed

Comments

@dyay108
Copy link

dyay108 commented Nov 9, 2024

System OS

Linux

Python Version

3.10 (CPython)

Install Source

GitHub

Install version / commit hash

e4e3201

Expected Behavior vs Actual Behavior

Prior to commit e4e3201, the path for the generated m3u file could be defined by providing the complete file path as the file name. However, with the latest update, the filename string undergoes sanitization, which restricts the use of slashes in the file name, thus preventing path specification in this way.

Steps to reproduce - Ensure to include actual links!

spotdl sync playlist_url --m3u full/path/playlist.m3u

Traceback

.
.
.
.

Other details

No response

@dyay108 dyay108 added the Bug Unexpected problem or unintended behavior that needs to be fixed label Nov 9, 2024
@Silverarmor
Copy link
Member

@xnetcat in v4.2.6 #2141, Sanitize m3u file by @xnetcat appears to have caused this

@xnetcat
Copy link
Member

xnetcat commented Nov 14, 2024

will be fixed in v4.2.9

@xnetcat xnetcat closed this as completed Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Unexpected problem or unintended behavior that needs to be fixed
Projects
None yet
Development

No branches or pull requests

3 participants