search

Manveer1999 / flutter_sms_mms

A flutter plugin to send SMS & MMS via native messengers
MIT License
1 stars 2 forks source link

Do not use QUERY_ALL_PACKAGES permission #2

Closed fatherOfLegends closed 8 months ago

fatherOfLegends commented 1 year ago

Currently this plugin is adding this permission to the manifest: <uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" />

This is a restricted permission when targeting Android API >= 30. Google Play now requires targeting API 33+ Please follow the guide here to specify the correct limited permissions so consumers do not need to hack away QUERY_ALL_PACKAGES: https://developer.android.com/training/package-visibility

UmairSaqibBhutta commented 1 year ago

Same issue. Playstore rejected my app due to this permission. Google clearly mentions that Less broad app-visibility method should be used.

fatherOfLegends commented 1 year ago

@UmairSaqibBhutta just so you know it is possible to work around this issue by specifying a tag to remove this permission when merging manifests. I still think it should be fixed though.

UmairSaqibBhutta commented 1 year ago

I also tried to remove permission by using this in manifest. <uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" tools:node="remove" /> but it's not working.

rbarzel commented 10 months ago

@UmairSaqibBhutta tools:node="remove" worked for me. Did you remember to also change the manifest tag?

<manifest xmlns:android="http://schemas.android.com/apk/res/android" xmlns:tools="http://schemas.android.com/tools">
Manveer1999 commented 8 months ago

This issue has been fixed so, closing this Issue in sms_mms: 0.0.3