jeancsanchez / JcPlayer

:musical_note: A simple audio player for Android applications.
Apache License 2.0
269 stars 110 forks source link

The broadcast receiver is registered incorrectly #145

Open AhmedGaberAbass opened 4 months ago

AhmedGaberAbass commented 4 months ago

I got this error from Google Play, Is there a solution to it?

One problem has been detected We've spotted some common issues that prevent your app from being submitted for review. Please resolve these issues before submitting your changes for review. Technical quality The broadcast receiver is registered incorrectly Applications targeting Android version 13 or later must specify export behavior when registerReceiver() is executed. If the export behavior is not specified, your application may crash.

Your app targets Android version 13 or later and registers non-system broadcast receivers without specifying export behavior in the following places:

Please update your application to specify export behavior for registered broadcast receivers.

jeancsanchez commented 4 months ago

Hi @AhmedGaberAbass

I reviewed this issue and I couldn't find anything related to it. The JcPlayerNotificationReceiver already is registered in Manifest file with the exported property specified:

Screenshot 2024-07-10 at 20 18 28

I wonder if there is something in your application that is registering a context registered receivers somewhere.

dzmitrykavalioum commented 2 months ago

HI @AhmedGaberAbass was this issue solved ?

i observe the same issue at my side and what is interesting - google console didn't provide places where problem was found.

AhmedGaberAbass commented 2 months ago

Hi, how are you, I hope you are fine

في الاثنين، ١٩ أغسطس ٢٠٢٤، ٥:٣٠ م Dzmitry Kavaliou @.***> كتب:

HI @AhmedGaberAbass https://github.com/AhmedGaberAbass was this issue solved ?

i observe the same issue at my side and what is interesting - google console didn't provide places where problem was found.

— Reply to this email directly, view it on GitHub https://github.com/jeancsanchez/JcPlayer/issues/145#issuecomment-2296723668, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUCW6SKY3MHKMC3VFGCQ4DLZSH6R7AVCNFSM6AAAAABKMEFFJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJWG4ZDGNRWHA . You are receiving this because you were mentioned.Message ID: @.***>

dzmitrykavalioum commented 2 months ago

Hi Sorry for the delay

How do you register your BroadcastReceiver - could you show - how it was implemented at your side?

dzmitrykavalioum commented 2 months ago

I don't think that problem relates to permissions (in this case warning must contain permission error)