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

fix(Boost for reddit - Fix missing audio in video downloads): Replace correct strings #3379

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

OctoNezd
Copy link
Contributor

Some refactor broke the audio download feature, resulting in v.redd.it and / strings getting replaced instead of audio endpoints resulting in client not even trying to download audio. This fixes that, by removing these two strings and keeping only /DASH_audio.mp4 and /audio. Both strings are unique in final version of Boost.

@oSumAtrIX oSumAtrIX changed the title fix(Boost for reddit - Fix missing audio in video downloads): Fix invalid string replacement fix(Boost for reddit - Fix missing audio in video downloads): Replace correct strings Jun 24, 2024
@oSumAtrIX oSumAtrIX merged commit b43db98 into ReVanced:dev Jun 24, 2024
2 checks passed
revanced-bot pushed a commit that referenced this pull request Jun 24, 2024
## [4.10.1-dev.2](v4.10.1-dev.1...v4.10.1-dev.2) (2024-06-24)

### Bug Fixes

* **Boost for reddit - Fix missing audio in video downloads:** Replace correct strings ([#3379](#3379)) ([b43db98](b43db98))
@Eagle1337
Copy link

I tested it using the 4.11.0-dev2 patches which afaik should have the fix and it still doesn't work for me

@OctoNezd
Copy link
Contributor Author

Tested patch with java -jar .\revanced-cli-4.6.0-all.jar patch -b revanced-patches-4.11.0-dev.2.jar -m revanced-integrations-1.11.0.apk --options=C:\Users\octo\rev\options.json C:\Users\octo\rev\src\boost.apk -d, audio is downloading correctly. Can you share link to post where audio is not downloading correctly?

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.

None yet

3 participants