Always use "conserve power" priority for challenge notifications

signalapp · Nov 21, 2023 · ff7a5f4 · ff7a5f4
1 parent 42a4740
commit ff7a5f4
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/service/src/main/java/org/whispersystems/textsecuregcm/push/APNSender.java b/service/src/main/java/org/whispersystems/textsecuregcm/push/APNSender.java
@@ -121,8 +121,12 @@ public CompletableFuture<SendPushNotificationResult> sendNotification(final Push
       pushType = notification.urgent() ? PushType.ALERT : PushType.BACKGROUND;
     }
 
-    final DeliveryPriority deliveryPriority =
-        (notification.urgent() || isVoip) ? DeliveryPriority.IMMEDIATE : DeliveryPriority.CONSERVE_POWER;
+    final DeliveryPriority deliveryPriority = switch (notification.notificationType()) {
+      case NOTIFICATION ->
+          (notification.urgent() || isVoip) ? DeliveryPriority.IMMEDIATE : DeliveryPriority.CONSERVE_POWER;
+      case ATTEMPT_LOGIN_NOTIFICATION_HIGH_PRIORITY -> DeliveryPriority.IMMEDIATE;
+      case CHALLENGE, RATE_LIMIT_CHALLENGE -> DeliveryPriority.CONSERVE_POWER;
+    };
 
     final String collapseId =
         (notification.notificationType() == PushNotification.NotificationType.NOTIFICATION && notification.urgent() && !isVoip)