Fix duplicate error unwrapping

Signal/src/AppDelegate.m

@@ -718,12 +718,10 @@ static NSTimeInterval launchStartedAt;
             [Environment.shared.contactsManager fetchSystemContactsOnceIfAlreadyAuthorized];
             [[AppEnvironment.shared.messageFetcherJob run] retainUntilComplete];
             
-            [[LokiAPI getMessagesObjc]
+            [[LokiAPI objc_getMessages].then(^(id result) {
-             .then(^(id result) {
+                // TODO: Handle result
-                // TODO: handle result
                 
-             })
+             }).catch(^(NSError *error) {
-             .catch(^(NSError *error) {
                 
              }) retainUntilComplete];
            
@@ -1161,11 +1159,9 @@ static NSTimeInterval launchStartedAt;
 {
     OWSLogInfo(@"performing background fetch");
     [AppReadiness runNowOrWhenAppDidBecomeReady:^{
-        [[LokiAPI getMessagesObjc]
+        [[LokiAPI objc_getMessages].then(^(id result) {
-         .then(^(id result) {
             completionHandler(UIBackgroundFetchResultNewData);
-         })
+         }).catch(^(NSError *error) {
-         .catch(^(NSError *error) {
             completionHandler(UIBackgroundFetchResultFailed);
          }) retainUntilComplete];
 

SignalServiceKit/src/Loki/LokiAPI.swift

@@ -66,24 +66,15 @@ import PromiseKit
     }
     
     // MARK: Obj-C API
-    @objc public static func getMessagesObjc() -> AnyPromise {
+    @objc public static func objc_getMessages() -> AnyPromise {
         let promise = getMessages()
         let anyPromise = AnyPromise(promise)
         anyPromise.retainUntilComplete()
         return anyPromise
     }
     
-    @objc public static func sendSignalMessage(_ signalMessage: SignalMessage, to destination: String, requiringPoW isPoWRequired: Bool) -> AnyPromise {
+    @objc public static func objc_sendSignalMessage(_ signalMessage: SignalMessage, to destination: String, requiringPoW isPoWRequired: Bool) -> AnyPromise {
-        let promise = LokiMessage.fromSignalMessage(signalMessage, requiringPoW: isPoWRequired)
+        let promise = LokiMessage.fromSignalMessage(signalMessage, requiringPoW: isPoWRequired).then(sendMessage)
-            .then(sendMessage)
-            .recover(on: DispatchQueue.global()) { error -> Promise<RawResponse> in
-                switch error {
-                case NetworkManagerError.taskError(_, let underlyingError):
-                    throw underlyingError
-                default:
-                    throw error
-                }
-        }
         let anyPromise = AnyPromise(promise)
         anyPromise.retainUntilComplete()
         return anyPromise

SignalServiceKit/src/Messages/OWSMessageSender.m

@@ -1111,31 +1111,27 @@ NSString *const OWSMessageSenderRateLimitedException = @"RateLimitedException";
     // Convert the message to a Loki message and send it using the Loki messaging API
     NSDictionary *signalMessage = deviceMessages.firstObject;
     BOOL isPoWRequired = YES; // TODO: Base on message type
-    
+    [[LokiAPI objc_sendSignalMessage:signalMessage to:recipient.recipientId requiringPoW:isPoWRequired]
-    [[LokiAPI sendSignalMessage:signalMessage to:recipient.recipientId requiringPoW:isPoWRequired]
         .thenOn([OWSDispatch sendingQueue], ^(id result) {
             [self messageSendDidSucceed:messageSend
                          deviceMessages:deviceMessages
                             wasSentByUD:false
                      wasSentByWebsocket:false];
         })
-        .catchOn([OWSDispatch sendingQueue], ^(NSError *error) {
+        .catchOn(OWSDispatch.sendingQueue, ^(NSError *error) {
             NSUInteger statusCode = 0;
             NSData *_Nullable responseData = nil;
             if ([error.domain isEqualToString:TSNetworkManagerErrorDomain]) {
                 statusCode = error.code;
-                
                 NSError *_Nullable underlyingError = error.userInfo[NSUnderlyingErrorKey];
                 if (underlyingError) {
-                    responseData
+                    responseData = underlyingError.userInfo[AFNetworkingOperationFailingURLResponseDataErrorKey];
-                    = underlyingError.userInfo[AFNetworkingOperationFailingURLResponseDataErrorKey];
                 } else {
                     OWSFailDebug(@"Missing underlying error: %@", error);
                 }
             } else {
                 OWSFailDebug(@"Unexpected error: %@", error);
             }
-            
             [self messageSendDidFail:messageSend
                       deviceMessages:deviceMessages
                           statusCode:statusCode