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 #79

Open
IzzySoft opened this issue Mar 30, 2024 · 2 comments
Open

Question on permissions #79

IzzySoft opened this issue Mar 30, 2024 · 2 comments

Comments

@IzzySoft
Copy link

My scanner just reported on your latest release:

! repo/com.tomfong.simpleqr_4010000.apk declares sensitive permission(s):
  android.permission.READ_MEDIA_IMAGES android.permission.READ_EXTERNAL_STORAGE
  android.permission.READ_CONTACTS android.permission.WRITE_CONTACTS android.permission.CAMERA
! repo/com.tomfong.simpleqr_4010000.apk contains signature block blobs: 0x504b4453 (DEPENDENCY_INFO_BLOCK; GOOGLE)

Could you please clarify what those permissions are needed for? Camera is clear (QR code scanning). Storage read access also, thanks to the app description (import QR codes from images). I assume read/write contacts is to create/import QR codes with contact information?

As for the DEPENDENCY_INFO_BLOCK, that can easily be avoided:

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.

Thanks in advance!

@tomfong
Copy link
Owner

tomfong commented Mar 31, 2024

For permissions, the purposes stated by you are all correct.

For BLOB, one of the dependencies supports reading BLOB value after update, but the app doesn't use the function.

@IzzySoft
Copy link
Author

IzzySoft commented Apr 4, 2024

For permissions, the purposes stated by you are all correct.

Thanks! Updated that on my end then:

image

For BLOB, one of the dependencies supports reading BLOB value after update, but the app doesn't use the function.

May I hen ask you to exclude it at least for the APK? As only Google can decode that BLOB, it's not useful in the FOSS ecosystem anyway.

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