sendToGroup: Don't fail send if we get 401 from multi_recipient/ request

ts/test-electron/util/sendToGroup_test.ts

@@ -15,12 +15,14 @@ import type { DeviceType } from '../../textsecure/Types.d';
 import {
   ConnectTimeoutError,
   HTTPError,
+  IncorrectSenderKeyAuthError,
   MessageError,
   OutgoingIdentityKeyError,
   OutgoingMessageError,
   SendMessageChallengeError,
   SendMessageNetworkError,
   SendMessageProtoError,
+  UnknownRecipientError,
   UnregisteredUserError,
 } from '../../textsecure/Errors';
 
@@ -219,6 +221,16 @@ describe('sendToGroup', () => {
       assert.isFalse(_shouldFailSend(error, 'testing generic 204'));
     });
 
+    it('returns false for specific errors', () => {
+      const unknownRecipient = new UnknownRecipientError();
+      assert.isFalse(
+        _shouldFailSend(unknownRecipient, 'testing unknown recipient')
+      );
+
+      const incorrectAuth = new IncorrectSenderKeyAuthError();
+      assert.isFalse(_shouldFailSend(incorrectAuth, 'testing incorrect auth'));
+    });
+
     it('returns true for a specified error codes', () => {
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
       const error: any = new Error('generic');

ts/textsecure/Errors.ts

@@ -305,4 +305,6 @@ export class ConnectTimeoutError extends Error {}
 
 export class UnknownRecipientError extends Error {}
 
+export class IncorrectSenderKeyAuthError extends Error {}
+
 export class WarnOnlyError extends Error {}

ts/util/sendToGroup.ts

@@ -34,6 +34,7 @@ import type {
 } from '../textsecure/SendMessage';
 import {
   ConnectTimeoutError,
+  IncorrectSenderKeyAuthError,
   OutgoingIdentityKeyError,
   SendMessageProtoError,
   UnknownRecipientError,
@@ -65,6 +66,7 @@ import { GLOBAL_ZONE } from '../SignalProtocolStore';
 import { waitForAll } from './waitForAll';
 
 const UNKNOWN_RECIPIENT = 404;
+const INCORRECT_AUTH_KEY = 401;
 const ERROR_EXPIRED_OR_MISSING_DEVICES = 409;
 const ERROR_STALE_DEVICES = 410;
 
@@ -569,6 +571,9 @@ export async function sendToGroupViaSenderKey(options: {
     if (error.code === UNKNOWN_RECIPIENT) {
       throw new UnknownRecipientError();
     }
+    if (error.code === INCORRECT_AUTH_KEY) {
+      throw new IncorrectSenderKeyAuthError();
+    }
 
     if (error.code === ERROR_EXPIRED_OR_MISSING_DEVICES) {
       await handle409Response(logId, error);
@@ -764,10 +769,13 @@ export function _shouldFailSend(error: unknown, logId: string): boolean {
     log.error(`_shouldFailSend/${logId}: ${message}`);
   };
 
-  // We need to fail over to a normal send if multi_recipient/ endpoint returns 404
+  // We need to fail over to a normal send if multi_recipient/ endpoint returns 404 or 401
   if (error instanceof UnknownRecipientError) {
     return false;
   }
+  if (error instanceof IncorrectSenderKeyAuthError) {
+    return false;
+  }
 
   if (
     error instanceof LibSignalErrorBase &&