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

Option to fall back to Piped proxy when Piped proxy disabled and video unreachable #4053

Merged
merged 3 commits into from
Jul 1, 2023

Conversation

Bnyro
Copy link
Member

@Bnyro Bnyro commented Jun 19, 2023

closes #4038

@Bnyro Bnyro force-pushed the fallback-piped-proxy branch 4 times, most recently from c69515f to 2fbf206 Compare June 19, 2023 18:10
@Bnyro Bnyro changed the title Option to fallback to Piped proxy when proxy disabled Option to fall back to Piped proxy when Piped proxy disabled and video unreachable Jun 20, 2023
@Bnyro
Copy link
Member Author

Bnyro commented Jun 30, 2023

Is there a video to test this with? For me it appears that YT music videos work as expected even without the Proxy somehow?

@Bnyro Bnyro force-pushed the fallback-piped-proxy branch 2 times, most recently from 818f9d8 to ada94e0 Compare June 30, 2023 08:19
@Bnyro Bnyro marked this pull request as draft June 30, 2023 10:44
@Bnyro Bnyro marked this pull request as ready for review June 30, 2023 13:51
@ehea617
Copy link

ehea617 commented Jul 1, 2023

I'm currently getting an error at the bottom of the screen when opening any song in the "YT Music Songs" search filter, "Initial player response is not valid"

This error message appears regardless of the song selected or combination of options of piped proxy enabled/disabled and fallback enabled/disabled.

@Bnyro
Copy link
Member Author

Bnyro commented Jul 1, 2023

Try a different instance, see TeamPiped/Piped#2487

@Bnyro
Copy link
Member Author

Bnyro commented Jul 1, 2023

That's not related to the changes in this PR.

@ehea617
Copy link

ehea617 commented Jul 1, 2023

Thanks for the info - I can confirm this works great after I switched the instance over to projectsegfau.lt and do not notice any issues specific to this now.

@Bnyro
Copy link
Member Author

Bnyro commented Jul 1, 2023

Alright, thanks for testing!

@Bnyro Bnyro merged commit 78595ad into libre-tube:master Jul 1, 2023
@Bnyro Bnyro deleted the fallback-piped-proxy branch July 1, 2023 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fallback to Piped Proxy Option
3 participants