mirror of
https://github.com/sigma67/ytmusicapi.git
synced 2026-04-25 15:26:01 +03:00
[GH-ISSUE #642] Implement clear errors or sanitize requests with > or < in the playlist title. #433
Labels
No labels
a/b
bug
documentation
enhancement
good first issue
help wanted
invalid
pull-request
question
wontfix
yt-error
yt-update
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ytmusicapi#433
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @SuppliedOrange on GitHub (Aug 19, 2024).
Original GitHub issue: https://github.com/sigma67/ytmusicapi/issues/642
Is your feature request related to a problem? Please describe.
This is related to a problem.
Using
>and<in playlists will cause it to error with400 Bad Request.Originally found in https://github.com/linsomniac/spotify_to_ytmusic/issues/93
I could not find other characters affected by this. The error it returns does not make perfect sense either.
To reproduce:
This is what is returned:
Describe the solution you'd like
Might help to sanitize the request or return a more concise error. It's difficult to determine that it's related to this.
@sigma67 commented on GitHub (Aug 19, 2024):
Funny enough, this doesn't work on the Web either. Neither do they have a good error message for it. Not sure it's the task of this project to figure something out that YouTube's not doing
@sigma67 commented on GitHub (Aug 19, 2024):
I'd prefer to raise in that case. Not sure if this is just a temporary issue.
PR welcome
@SuppliedOrange commented on GitHub (Aug 20, 2024):
Cool, it would help to return the response object within your
YTMusicServerErrorand other exception classes. I'm guessing theINVALID_ARGUMENTstatus is probably related to this, so it'd be better to check it against that.Otherwise, would this be an appropriate change to make?
@SuppliedOrange commented on GitHub (Aug 20, 2024):
Silly me, I closed the issue.