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

question on permissions #767

Open
IzzySoft opened this issue Oct 9, 2024 · 2 comments
Open

question on permissions #767

IzzySoft opened this issue Oct 9, 2024 · 2 comments

Comments

@IzzySoft
Copy link

IzzySoft commented Oct 9, 2024

The scanners here at IzzyOnDroid just yielded:

! repo/moe.nb4a_180.apk declares intent-filter(s): android.net.VpnService
! repo/moe.nb4a_180.apk declares sensitive permission(s): android.permission.QUERY_ALL_PACKAGES
  android.permission.CAMERA android.permission.READ_EXTERNAL_STORAGE
! repo/moe.nb4a_180.apk contains signature block blobs: 0x504b4453 (DEPENDENCY_INFO_BLOCK; GOOGLE)

android.net.VpnService is clear (proxy uses Android's VPN service), so I've already added that to the app's "green list". But could you please clarify what those permissions are needed for?

Concerning DEPENDENCY_INFO_BLOCK: that can easily be avoided with a minor addition to your build.gradle:

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. More details can be found e.g. here: Ramping up security: additional APK checks are in place with the IzzyOnDroid repo.

Thanks in advance!

@Mahdi-zarei
Copy link

The Query all packages permission is for per-app proxy feature. Camera permission is for scanning qr code to import configs and the read external storage is for importing and exporting backup files (not sure about the last one)

@IzzySoft
Copy link
Author

Thanks! Added the first two to the app's "green list". As for storage: export/import would just need a single location, right? Wouldn't that be a classic candidate for SAF (Storage Access Framework), so no explicit (legacy) storage permission is needed?

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