-
-
Notifications
You must be signed in to change notification settings - Fork 448
feat(YouTube - Enable debugging): Add settings menu to share debug logs #5021
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
Conversation
This still needs to be tested as there could potentially be a buffer overflow error |
It might be useful to have a 'Clear logs' preference that when tapped clears the debug logs. |
I added a buffer clearing parameter for the future, initially I thought of clearing it after export, but it probably wouldn't be clear to the user. Maybe add a menu item at the bottom for clearing, or I don't remember if there is a way to handle something like |
and why are there lines in the log |
It still logs the litho components, even if log protocol buffer is off. It's useful to still log litho stuff but not log the protocol buffer, since the buffer can make manually searching more difficult since it has so much unrelated stuff. |
4fea209
to
1e3e7b2
Compare
d76017e
to
9d04f6f
Compare
9d04f6f
to
61a6ba2
Compare
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
…sts for devices that follow AOSP behavior)
This seems done? This feature should have been added ages ago. |
Everything seems to work for me. |
I checked ADB, and it works as expected. |
Ready to merge? |
I think yes. No problems were found. |
# [5.25.0](v5.24.0...v5.25.0) (2025-05-29) ### Bug Fixes * **Disable Pairip license check:** Change patch to default off ([74b6a94](74b6a94)) * **Hide ADB status:** Resolve app crash on startup ([#5029](#5029)) ([1abebd5](1abebd5)) * **Messenger:** Remove outdated `Disable switching emoji to sticker` patch ([#5044](#5044)) ([7b182ca](7b182ca)) * **Spotify Lite:** Remove obsolete `Enable on demand` patch ([#5046](#5046)) ([4886d47](4886d47)) * **YouTube - GmsCore support:** Restore patch functionality from prior merge ([7686bbe](7686bbe)) * **YouTube - Hide ads:** Hide new type of general ad ([#5004](#5004)) ([37e59d2](37e59d2)) * **YouTube - Open Shorts in regular player:** Do not exit app when pressing back button in regular player ([#5020](#5020)) ([3384f8d](3384f8d)) * **YouTube:** Better handle incorrect duplicate translations ([20abac5](20abac5)) * **Yuka - Unlock premium:** Remove broken patch that is no longer supported ([#5018](#5018)) ([fac6e59](fac6e59)) ### Features * Add `Disable pairip license check` patch ([#4927](#4927)) ([42d2c27](42d2c27)) * **Messenger:** Add `Remove Meta AI` patch ([#4945](#4945)) ([012dff7](012dff7)) * **Prime Video:** Add `Rename shared permissions` patch ([#5049](#5049)) ([80f1fc6](80f1fc6)) * **Spotify:** Add `Fix Facebook login` patch ([#5023](#5023)) ([34932dc](34932dc)) * **Threads:** Hide Ads ([#5064](#5064)) ([3c4cecb](3c4cecb)) * **YouTube - Enable debugging:** Add settings menu to share debug logs ([#5021](#5021)) ([1ec4a88](1ec4a88)) * **YouTube - Settings:** Add a color picker ([#4981](#4981)) ([1e0e398](1e0e398)) * **YouTube - Swipe controls:** Add separate color settings for the brightness and volume bars ([#5043](#5043)) ([80f50e8](80f50e8)) * **YouTube:** Add `Disable haptic feedback` patch ([#5033](#5033)) ([bbe7974](bbe7974))
Adds an option to copy the ReVanced debug logs to the clipboard.
Due to clipboard size limits in Android, the log is limited to about 1MB of the most recent logs entries. ADB can still be used to collect the logs and the log limit is much larger.
Example of the result in the clipboard