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
Specify android:exported
to the activities having <intent-filter>
#1516
Conversation
Set flag `android:exported` to all the activities having `<intent-filter>`.
I doubt this is all the change that is needed? IIRC the permissions model has to change too. Does it work? |
Yeah this change is needed. Otherwise consumer apps will manually need to override entries in the manifest by exporting it as false and merging nodes. |
Is it sufficient though to make the app work? I think there are other changes needed. The change itself seems fine, just not exporting the activities |
As far as I saw, this change seems fine. What do you think what kind of other changes are needed which are mentioned here? |
IIRC the main thing is the permissions model. If you target Android 9+ it won't work. I can't upload the app at this point as a result (but it's not going to be released further anyway) |
But as you can see, I haven't updated SDK target or compile version. I've just specified |
Right. I am saying that this is likely insufficient for Android 12 compatibility. Change the title? |
So the thing is app is unable to compile when this library is included in app which is targetting android 12. Manifest issues are occurring because this library haven't specified this flag (exported). So this change will make sure that no compile time issues will occur whenever any app is using it. Otherwise, every app will need to override Manifest entries at their end manually. |
android:exported
to the activities having <intent-filter>
Cool, I've updated PR title |
While targetting to Android 12 SDK (API 31), there's a restriction to providing the flag
android:exported
to all the components (activity, service, receivers) having<intent-filter>
. This PR sets this flag to all activities having intent filters which can be helpful for the client applications which are targeting to Android 12.Closes: #1517