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

Calling stub online can break Hide the Magisk app function #8066

Open
pndwal opened this issue May 20, 2024 · 0 comments
Open

Calling stub online can break Hide the Magisk app function #8066

pndwal opened this issue May 20, 2024 · 0 comments

Comments

@pndwal
Copy link
Contributor

pndwal commented May 20, 2024

Device: Any
Android version: Any
Magisk version name: Any
Magisk version code: 27002

Hide the Magisk App function is clearly calling stub from online repo (per update JSON) so if Update channel is set to Stable or Beta with Canary loaded and stub version has been incremented, the Hide attempt will fail since this commit: Always update stub APK when upgrade, topjohnwu committed on 15 Mar.

Eg the Stub apk used by 27000 (27.0) is v38... 27002 builds use v39, so if channel is set to Stable with current Canary installed, stub installation breaks with only unspecified "error" popping up...

I know we can simply set the channel so it doesn't call JSON for older versions when taking Hide (or Restore?) the Magisk app, but I don't know why stub is still being called online... I think there were previous fixes to make installation 100% offline (ie. to retrieve stub.apk embedded in current binaries) so perhaps this is a regression?...

I do see stub.apk is still linked in JSON so I may be missing a reason it needs to work this way, but I believe current stub is embedded in the package binaries so logically it seems that the app should pull stub from these as it does the binaries used for patching images (not rely on channel set or GitHub repo or network connection)...

Is there any reason that this shouldn't or can't work entirely offline?... Shouldn't the update JSON only be called for notes.md (changelog) and the complete Magisk package.apk (when updating App) these days?

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

No branches or pull requests

1 participant