-
Notifications
You must be signed in to change notification settings - Fork 448
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
Checking for Root apps on Android 11+ is ineffectual due to package-visibility changes #167
Comments
Thanks for raising this @BanHammerYKT The good news is we don't crash (we would have caught that earlier). Partly due to the way Android handles the But the bad news is that from my testing from Android 11 these checks for any root apps will be ineffective 😢 . Here's a test I ran with Sample Rootbeer looking for
|
Will adding apps list from this file
in manifest resolve this issue? |
Potentially, but I saw in one of these Google I/O 2021 talks What’s new in Android privacy or The state of Android security that Google are changing the Play policy to make access to package queries only available to apps that genuinely need it. I'm not sure they would agree that root checking qualifies given SafetyNet is offered?. Therefore I'm reluctant to add this feature as I could see apps getting pulled if we implemented this check. Although it does feel like we need to do something here to flag these checks are not running on Android 11+ maybe deprecating them and moving them out of the default checks 🤔 |
What if add an optional lib with manifest, something like
|
Yep, that could work @BanHammerYKT. Would welcome a PR |
https://developer.android.com/about/versions/11/privacy/package-visibility
Does rootbeer support these changes?
Or everything works fine on android 11?
The text was updated successfully, but these errors were encountered: