Skip to content

Add MANAGE_CLOUD_MEDIA_PROVIDERS permission for cloud media providers (Immich)#1320

Open
marx161-cmd wants to merge 1 commit into
crdroidandroid:16.0from
marx161-cmd:patch/add-cloud-media-provider-permission
Open

Add MANAGE_CLOUD_MEDIA_PROVIDERS permission for cloud media providers (Immich)#1320
marx161-cmd wants to merge 1 commit into
crdroidandroid:16.0from
marx161-cmd:patch/add-cloud-media-provider-permission

Conversation

@marx161-cmd

Copy link
Copy Markdown

This patch declares the missing android.permission.MANAGE_CLOUD_MEDIA_PROVIDERS permission that's required for apps like Immich to act as cloud media providers on Android 16.

Changes

core/res/AndroidManifest.xml

  • Adds the MANAGE_CLOUD_MEDIA_PROVIDERS permission declaration with signature|privileged protection level, placed alongside the existing MANAGE_MEDIA permission.

data/etc/privapp-permissions-platform.xml

  • Adds an allowlist entry for Immich (app.alextran.immich) so it can hold the MANAGE_CLOUD_MEDIA_PROVIDERS permission when installed as a privileged app or granted via root.

Context

The CloudMediaProvider implementation already exists in AOSP's android_packages_providers_MediaProvider (pulled from LineageOS). The only missing piece is the permission declaration itself. This benefits every crDroid user running Immich or any other cloud media provider app.

- core/res/AndroidManifest.xml: declare MANAGE_CLOUD_MEDIA_PROVIDERS
  permission with signature|privileged protection level
- data/etc/privapp-permissions-platform.xml: allowlist Immich
  (app.alextran.immich) to hold the new permission

The CloudMediaProvider implementation already exists in AOSP's
MediaProvider (pulled via LineageOS).  The only missing piece was
the permission declaration itself.
@neobuddy89

Copy link
Copy Markdown
Member

I dont see point of elevating this permission. If it is needed by app, add it for the app itself.

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

Successfully merging this pull request may close these issues.

2 participants