Include failed attempts when determining previouslyFailedSenders

signalapp · Jul 15, 2024 · 86de827 · 86de827
1 parent e91e950
commit 86de827
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/src/main/java/org/signal/registration/RegistrationService.java b/src/main/java/org/signal/registration/RegistrationService.java
@@ -190,6 +190,13 @@ public CompletableFuture<RegistrationSession> sendVerificationCode(final Message
               .map(RegistrationAttempt::getSenderName)
               .collect(Collectors.toSet());
 
+          // Add senders from failed attempts with a "provider unavailable" error
+          session.getFailedAttemptsList()
+              .stream()
+              .filter(attempt -> attempt.getFailedSendReason() == FailedSendReason.FAILED_SEND_REASON_UNAVAILABLE)
+              .map(FailedSendAttempt::getSenderName)
+              .forEach(previouslyFailedSenders::add);
+
           final SenderSelection selection = senderSelectionStrategy.chooseVerificationCodeSender(
               messageTransport, phoneNumberFromSession, languageRanges, clientType, senderName,
               previouslyFailedSenders);