-
Notifications
You must be signed in to change notification settings - Fork 5
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
[bug] search filtered by videos now may include podcast/episode which throws an error #14
Comments
If it's returning a In this case, podcasts aren't supported yet, and I'm not sure why YTM is returning podcasts when you are searching for songs anyway. I can't actually reproduce this because for me everything works correctly... you can try to compare what gets returned when you search for that on music.youtube.com |
thanks for the quick response! YT changed their search priorities and above code works fine just hours later, and I found another reproducible code:
happens in filter videos not songs, my apologies. exact same error msg |
working for me... can you tell me what happens when you search for that at https://music.youtube.com? do you get podcast results when you filtered for videos?? |
to be faster, you can just clone this repo and edit the
well the folks at YTM are bizarre. anyway, I guess the actual solution would be to add support for podcasts and episodes. |
or we can try-catch wrap |
yes we can. |
I recently bumped into some failed to parse cases at libmuse/script/parsers/search.js's
parse_search_song
reproducible with
the navigationEndPoint returns browseEndpoint instead of watchEndpoint:
![image](https://private-user-images.githubusercontent.com/106490582/336462916-af5e9a73-51d9-4c2a-bda8-0b4c267ce06c.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjAxNTQzNzYsIm5iZiI6MTcyMDE1NDA3NiwicGF0aCI6Ii8xMDY0OTA1ODIvMzM2NDYyOTE2LWFmNWU5YTczLTUxZDktNGMyYS1iZGE4LTBiNGMyNjdjZTA2Yy5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzA1JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcwNVQwNDM0MzZaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1hZDQxYmFkNzVhMTNhMGYzYzZjZDViY2VlOTQxYjRmZGYzYWRlZjY0Nzc2MjkxYjU1YTA3ZGYwOGE5ZDIwN2MyJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.ps7fLjQGV34ijOPSq0ShAdJ6AYarYtZ6MJuIjCvZsAM)
The text was updated successfully, but these errors were encountered: