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

bug: Manager sometimes immediately cancels installation and Export APK button stops working #2224

Open
4 tasks done
KobeW50 opened this issue Sep 23, 2024 · 12 comments
Open
4 tasks done
Labels
Bug report Something isn't working

Comments

@KobeW50
Copy link
Contributor

KobeW50 commented Sep 23, 2024

Bug description

Sometimes when trying to install a patched app, the Manager will immediately cancel the installation. The Export APK button will also not work. The issue only occurs when the Save patched app toggle in the Manager settings is enabled.

It is difficult to reproduce this bug. Below is the simplest and most consistent method I found.

Steps to reproduce

  1. Prepare the replication environment by uninstalling YouTube ReVanced and clearing the ReVanced Manager data
  2. Open the Manager settings and enable the Allow changing patch selection option
  3. Select the suggested YouTube APK from storage
  4. Patch and install YouTube with only the GmsCore support patch applied (the bug should not occur at this point)
  5. Without closing the Manager, return to the Patcher screen and repeat step 3 and 4

The bug should occur when repeating step 4 for the second time. (You may need to try again)

Demonstration:

YouCut_20240923_120743918.mp4

Version of ReVanced Manager and version & name of app you are patching

1.22.0
YouTube

Installation method

Regular

ReVanced Manager logs

revanced-manager_logcat_20240923114749497367.log.txt

Patch logs

- Device Info
ReVanced Manager: 1.22.0
Model: SM-S906U1
Android version: 14
Supported architectures: arm64-v8a, armeabi-v7a, armeabi
Root permissions: No

- Patch Info
App: com.google.android.youtube v19.16.39 (Suggested: 19.16.39)
Patches version: v4.15.0
Patches added: Default
Patches removed: Hide ads, Video ads, Copy video URL, Remove viewer discretion dialog, Downloads, Disable precise seeking gesture, Seekbar tapping, Swipe controls, Disable auto captions, Hide video action buttons, Hide autoplay button, Hide captions button, Hide cast button, Navigation buttons, Hide player buttons, Hide album cards, Comments, Hide crowdfunding box, Hide endscreen cards, Hide filter bar, Hide floating microphone button, Disable fullscreen ambient mode, Hide layout components, Hide info cards, Player flyout menu, Disable rolling number animations, Hide seekbar, Hide Shorts components, Disable suggested video end screen, Hide timestamp, Miniplayer, Disable player popup panels, Custom player overlay opacity, Return YouTube Dislike, Wide searchbar, Restore old seekbar thumbnails, SponsorBlock, Spoof app version, Change start page, Disable resuming Shorts on startup, Enable tablet layout, Theme, Alternative thumbnails, Bypass image region restrictions, Announcements, Always repeat, Remove background playback restrictions, Enable debugging, Spoof device dimensions, Check watch history domain name resolution, Spoof video streams, Bypass URL redirects, Remove tracking query parameter, Disable zoom haptics, Remember video quality, Playback speed, Restore old video quality menu, Open links externally
Default patch options changed: None

- Settings
Allow changing patch selection: true
Version compatibility check: true
Show universal patches: false
Patches source: revanced/revanced-patches
Integration source: revanced/revanced-integrations

- Logs
Reading APK
Decoding app manifest
Loading patches
Merging integrations
Deleting existing temporary files directory
Decoding resources
Executing patches
Applied 2 patches
Compiling patched dex files
Compiled 8 dex files
Compiling modified resources
Aligning APK
Signing APK
Patched APK
Installing patched app
Installation canceled

Acknowledgements

  • I have checked all open and closed bug reports and this is not a duplicate.
  • I have chosen an appropriate title.
  • All requested information has been provided properly.
  • The bug is only related to ReVanced Manager.
@KobeW50 KobeW50 added the Bug report Something isn't working label Sep 23, 2024
@KobeW50 KobeW50 changed the title bug: bug: Manager sometimes immediately cancels installation and Export APK button stops working Sep 23, 2024
@KobeW50

This comment was marked as resolved.

@oSumAtrIX oSumAtrIX mentioned this issue Sep 23, 2024
4 tasks
@oSumAtrIX
Copy link
Member

The logs only mention a NPE, which is caused by this issue. No more relevant logs apart from that.

@oSumAtrIX
Copy link
Member

Disable Play Protect and see if you can reproduce.

@KobeW50
Copy link
Contributor Author

KobeW50 commented Sep 23, 2024

Disable Play Protect and see if you can reproduce.

The issue still occurs.

SmartSelect_20240923_142012_Google Play Store

revanced-manager_logcat_20240923142020502224.log.txt

@KobeW50
Copy link
Contributor Author

KobeW50 commented Sep 23, 2024

After some more testing, I was unable to reproduce the issue when the Save patched app toggle in the Manager settings is disabled.

(Also, I was able to reproduce the issue with the Reddit app, even without changing it's package name.)

@oSumAtrIX
Copy link
Member

Enable Play Protect again and reproduce it.

@KobeW50
Copy link
Contributor Author

KobeW50 commented Sep 24, 2024

Enable Play Protect again and reproduce it.

The issue continues to occur (tested with YouTube, with the Save patched app toggle enabled)

revanced-manager_logcat_20240924195347171913.log.txt

@oSumAtrIX
Copy link
Member

oSumAtrIX commented Sep 25, 2024

So it is related to Play Protect somehow getting rid of the file. What if you disable save patched app.

@KobeW50
Copy link
Contributor Author

KobeW50 commented Sep 25, 2024

What if you disable save patched app.

The issue doesn't occur (tested with play protect enabled).
revanced-manager_logcat_20240924203015351253.log.txt

YouCut_20240924_203403959.mp4

@oSumAtrIX
Copy link
Member

@BenjaminHalko maybe you know about this, since it's related to saving the last patched app

@BenjaminHalko
Copy link
Member

I can try to take a look at it tmr

@Bighead9971

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug report Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants