From 794e254d90b8e44a65c119c742533a09b5885d36 Mon Sep 17 00:00:00 2001 From: Ravi Khadiwala Date: Fri, 7 Feb 2025 13:48:36 -0600 Subject: [PATCH] Add null check to FCM error log condition --- .../java/org/whispersystems/textsecuregcm/push/FcmSender.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/service/src/main/java/org/whispersystems/textsecuregcm/push/FcmSender.java b/service/src/main/java/org/whispersystems/textsecuregcm/push/FcmSender.java index a14371276..ad38ae334 100644 --- a/service/src/main/java/org/whispersystems/textsecuregcm/push/FcmSender.java +++ b/service/src/main/java/org/whispersystems/textsecuregcm/push/FcmSender.java @@ -104,7 +104,8 @@ public class FcmSender implements PushNotificationSender { if (firebaseMessagingException.getMessagingErrorCode() != null) { errorCode = firebaseMessagingException.getMessagingErrorCode().name(); - if (firebaseMessagingException.getMessagingErrorCode() == MessagingErrorCode.QUOTA_EXCEEDED) { + if (firebaseMessagingException.getMessagingErrorCode() == MessagingErrorCode.QUOTA_EXCEEDED + && firebaseMessagingException.getHttpResponse() != null) { logger.info("FCM request failed with quota exceeded; retry-after: {}, response body: {}", firebaseMessagingException.getHttpResponse().getHeaders().get("retry-after"), firebaseMessagingException.getHttpResponse().getContent());