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

扩大 isFCMIntent 的范围 #163

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

RichardLuo0
Copy link

@RichardLuo0 RichardLuo0 commented Sep 30, 2024

以微信为例,他其中有如下相关intent filter:

com.google.firebase.MESSAGING_EVENT
com.google.firebase.INSTANCE_ID_EVENT
com.google.android.c2dm.intent.RECEIVE

我也不确定能不能解决什么问题,只是感觉这样比较保险

@kooritea
Copy link
Owner

kooritea commented Oct 3, 2024

startsWith​(​"com.google.firebase."​)​
这个范围太大了吧,改成匹配后缀吧。
另外根据 #153 也要兼容MicroG的情况不要com.google前缀

@RichardLuo0
Copy link
Author

startsWith​(​"com.google.firebase."​)​ 这个范围太大了吧,改成匹配后缀吧。 另外根据 #153 也要兼容MicroG的情况不要com.google前缀

改了一下,不过我看他microg也是com.google开头的
https://github.com/microg/GmsCore/blob/2149212967ed0038a8b22f60726ecc39476e5c96/play-services-basement/src/main/java/org/microg/gms/gcm/GcmConstants.java#L20

@RichardLuo0
Copy link
Author

虽然我还是保持原来的后缀匹配了

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