Skip to content

Allow legacy formats to be used even when dash and audio-only are unavailable #6977

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

Merged
merged 1 commit into from
Mar 8, 2025

Conversation

absidue
Copy link
Member

@absidue absidue commented Mar 5, 2025

Allow legacy formats to be used even when dash and audio-only are unavailable

Pull Request Type

  • Bugfix
  • Feature Implementation

Description

Currently when FreeTube receives a SABR-only response from YouTube we get a No valid URL to decipher error from the local API and then try falling back to the Invidious API which will fail unless the user is running their own Invidious instance and has configured it in the FreeTube settings. As the legacy format 360p stream is still available in those responses, this pull request makes sure that it can still be used even when the URLs are missing for the DASH streams.

While the goal is still to add SABR support to FreeTube, this allows people to still watch videos in the mean time, even if it is at a low quality.

Testing

You can replicate a SABR-only response by adding info.streaming_data.adaptive_formats.forEach(f => { f.url = undefined }) just inside the if (info.streaming_data) { line in src/renderer/helpers/api/local.js.

Check that the legacy formats get selected automatically, you can't switch to DASH or audio-only and the downloads menu only lists the subtitles and the legacy format stream.

Desktop

  • OS: Windows
  • OS Version: 10
  • FreeTube version: 36d533a

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Mar 5, 2025
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) March 5, 2025 19:12
@FreeTubeBot FreeTubeBot merged commit f1b0797 into FreeTubeApp:development Mar 8, 2025
5 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Mar 8, 2025
@absidue absidue deleted the fix-legacy-alone branch March 8, 2025 19:00
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Mar 11, 2025
* development: (97 commits)
  Translated using Weblate (German)
  Translated using Weblate (Czech)
  Translated using Weblate (Basque)
  Translated using Weblate (Hungarian)
  Translated using Weblate (Chinese (Simplified Han script))
  Translated using Weblate (Japanese)
  Translated using Weblate (Estonian)
  Translated using Weblate (Portuguese (Brazil))
  Bump the eslint group with 4 updates (FreeTubeApp#6985)
  Add support for channel's `Courses` tab (FreeTubeApp#6641)
  Bump electron from 34.3.0 to 35.0.1 (FreeTubeApp#6986)
  Bump shaka-player from 4.13.6 to 4.13.8 (FreeTubeApp#6990)
  Bump css-minimizer-webpack-plugin from 7.0.0 to 7.0.2 (FreeTubeApp#6988)
  Bump swiper from 11.2.4 to 11.2.5 (FreeTubeApp#6989)
  Bump lefthook from 1.11.2 to 1.11.3 (FreeTubeApp#6987)
  Update Invidious DASH xtags handling for YouTube.js changes (FreeTubeApp#6944)
  Translated using Weblate (French)
  Allow legacy formats to be used even when dash and audio-only are unavailable (FreeTubeApp#6977)
  Translated using Weblate (Hungarian)
  Fix search not working on mobile because of undefined ref (FreeTubeApp#6706)
  ...
efb4f5ff-1298-471a-8973-3d47447115dc pushed a commit to efb4f5ff-1298-471a-8973-3d47447115dc/FreeTube that referenced this pull request Mar 27, 2025
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.

5 participants