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

Open
IzzySoft opened this issue Jul 7, 2024 · 0 comments
Open

Question on permissions #196

IzzySoft opened this issue Jul 7, 2024 · 0 comments

Comments

@IzzySoft
Copy link

IzzySoft commented Jul 7, 2024

Your recent release just triggered a couple of warnings with the IzzyOnDroid scanners:

! repo/com.piwigo.piwigo_ng_230.apk declares flag(s): usesCleartextTraffic
! repo/com.piwigo.piwigo_ng_230.apk declares sensitive permission(s):
  android.permission.READ_EXTERNAL_STORAGE
  android.permission.READ_MEDIA_IMAGES 
  android.permission.READ_MEDIA_VIDEO
  android.permission.READ_MEDIA_AUDIO
  android.permission.CAMERA
! repo/com.piwigo.piwigo_ng_230.apk contains signature block blobs: 0x504b4453 (DEPENDENCY_INFO_BLOCK; GOOGLE)

usesCleartextTraffic is rather clear (locally self-hosted instances without SSL). CAMERA is a bit unclear: the app description does not mention anything about taking photos. All in between I can mostly guess (even if you use Storage Access Framework, up to Android-9 the permissions needed are quite a mess) – but a confirmation would be appreciated.

As for DEPENDENCY_INFO_BLOCK, that can easily be avoided with a minor adjustment in 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!

So far I've added to your app's "green list":

image

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