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

[org.thoughtcrime.securesms] Push notifications, priority is changed to PRIORITY_UNKNOWN #1745

Open
billie80 opened this issue Jul 8, 2022 · 4 comments

Comments

@billie80
Copy link

billie80 commented Jul 8, 2022

Affected app
Name: [Signal]
Package id: [org.thoughtcrime.securesms]

Describe the bug
Push notification priority is changed to PRIORITY_UNKNOWN.
During the discussion regarding a push notification issue signalapp/Signal-Android#12325 for Signal (which has been fixed) a Signal developer noted that their push notification priority signalapp/Signal-Android#12325 (comment) is changed from high-priority to PRIORITY_UNKNOWN. Also a developer of ShiftOS stated that with the ShiftOS-L flavor there are no GMS available and they are not changing any priority signalapp/Signal-Android#12325 (comment) the question arose if this might be an issue with microG.

I noticed that there was this issue #338 requesting to put the app into a temporary whitelist when a high-priority push message is retrieved which might be related. However the pull request #732 for the implementation was already resolved long ago in commit 6002388.

Would be great if you could share any insight regarding this behaviour.

System
Android Version: [10]
Custom ROM: [ShiftOS-L 3.7 (20220511)]

microG
microG Core version: [v0.2.24.214816]
microG Self-Check results: [All ticked]

@mar-v-in
Copy link
Member

As I understand, ShiftOS-L does not ship with microG. Can you point out how you installed microG?

@billie80
Copy link
Author

No it does not ship it by default but it enables signature spoofing. I installed it using your F-Droid repo.

@mar-v-in
Copy link
Member

In this case, microG does not have privileged access to the device idle controller and can't wake up apps that are not exempted from battery optimizations. Did you exempt Signal from battery optimizations?

@billie80
Copy link
Author

Yes Signal is excluded from battery optimisation.

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

No branches or pull requests

2 participants