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

Questions on permissions #56

Open
IzzySoft opened this issue Feb 9, 2024 · 3 comments
Open

Questions on permissions #56

IzzySoft opened this issue Feb 9, 2024 · 3 comments

Comments

@IzzySoft
Copy link

IzzySoft commented Feb 9, 2024

My scanner just got a few additional features in January, and now on today's update reported:

! repo/com.truemlgpro.wifiinfo_1610.apk declares flag(s): usesCleartextTraffic
! repo/com.truemlgpro.wifiinfo_1610.apk declares sensitive permission(s):
 android.permission.ACCESS_FINE_LOCATION android.permission.ACCESS_COARSE_LOCATION
 android.permission.ACCESS_BACKGROUND_LOCATION

Some of that I think I was able to cover (please check if it's correct), but why is background location needed?

image

PS: If you wonder about the DEPENDENCY_INFO_BLOCK, that's easy to get rid of:

android {
    dependenciesInfo {
        // Disables dependency metadata when building APKs.
        includeInApk = false
        // Disables dependency metadata when building Android App Bundles.
        includeInBundle = false
    }
}

For some background: that BLOB is supposed to be just a binary representation of your app's dependency tree. But as it's encrypted with a public key belonging to Google, only Google can read it – and nobody else can even verify what it really contains.

@TrueMLGPro
Copy link
Owner

TrueMLGPro commented Feb 9, 2024

Hello, thank you for filling this issue!
ACCESS_BACKGROUND_LOCATION permission is completely optional (user is given a dialog on first app launch stating so while also including an explanation what this permission is used for), it's only used to display the SSID, BSSID and Network ID in the notification while the app is in the background or is not running.

@IzzySoft
Copy link
Author

IzzySoft commented Feb 9, 2024

Thanks! Added that, so it's no longer "chocolated".

I leave the issue open in case you want to address that BLOB here as well.

@TrueMLGPro
Copy link
Owner

I leave the issue open in case you want to address that BLOB here as well.

Oh yes, thank you for adding info about the DEPENDENCY_INFO_BLOCK thingy. Added this to my TODO list!

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

2 participants