Revert "Replace NSCoding with NSSecureCoding (#7831)" (#7863)

This reverts commit 34c4bdb.

Author: charlotteliang

FirebaseMessaging/CHANGELOG.md

@@ -1,6 +1,5 @@
 # 2021-04 -- v7.11.0
 - [changed] Refactor Messaging to internally not depending on InstanceID, but can co-exist. Will remove InstanceID dependency in the next Firebase breaking change. (#7814)
-- [changed] Replaced NSCoding with NSSecureCoding. (#7831)
 
 # 2021-02 -- v7.7.0
 - [fixed] Fixed an issue in which, when checking storage size before writing to disk, the client was checking document folders that were no longer used. (#7480)

FirebaseMessaging/Sources/FIRMessagingCode.h

@@ -220,11 +220,11 @@ typedef NS_ENUM(NSInteger, FIRMessagingMessageCode) {
   kFIRMessagingMessageCodeTokenManagerAPNSChangedTokenInvalidated = 34011,
   kFIRMessagingMessageCodeTokenManagerInvalidateStaleToken = 34012,
   // FIRMessagingTokenStore.m
-  // DO NOT USE 35002 - 35013
+  // DO NOT USE 15002 - 15013
   kFIRMessagingMessageCodeTokenStore000 = 35000,
   kFIRMessagingMessageCodeTokenStore001 = 35001,
-  kFIRMessagingMessageCodeTokenStoreUnarchivingTokenInfo = 35015,
-  kFIRMessagingMessageCodeTokenStoreArchiveError = 35016,
+  kFIRMessagingMessageCodeTokenStoreExceptionUnarchivingTokenInfo = 35015,
+
   // DO NOT USE 16000, 18004
 
   // FIRMessagingUtilities.m

FirebaseMessaging/Sources/Token/FIRMessagingAPNSInfo.h

@@ -22,7 +22,7 @@ NS_ASSUME_NONNULL_BEGIN
  *  Represents an APNS device token and whether its environment is for sandbox.
  *  It can read from and write to an NSDictionary for simple serialization.
  */
-@interface FIRMessagingAPNSInfo : NSObject <NSSecureCoding, NSCopying>
+@interface FIRMessagingAPNSInfo : NSObject <NSCoding, NSCopying>
 
 /// The APNs device token, provided by the OS to the application delegate
 @property(nonatomic, readonly, copy) NSData *deviceToken;

FirebaseMessaging/Sources/Token/FIRMessagingAPNSInfo.m

@@ -68,11 +68,7 @@ - (id)copyWithZone:(NSZone *)zone {
   return clone;
 }
 
-#pragma mark - NSSecureCoding
-
-+ (BOOL)supportsSecureCoding {
-  return YES;
-}
+#pragma mark - NSCoding
 
 - (nullable instancetype)initWithCoder:(NSCoder *)aDecoder {
   id deviceToken = [aDecoder decodeObjectForKey:kFIRInstanceIDAPNSInfoTokenKey];

FirebaseMessaging/Sources/Token/FIRMessagingTokenInfo.h

@@ -25,7 +25,7 @@ NS_ASSUME_NONNULL_BEGIN
  *  associated with it. It can read from and write to an NSDictionary object, for
  *  simple serialization.
  */
-@interface FIRMessagingTokenInfo : NSObject <NSSecureCoding>
+@interface FIRMessagingTokenInfo : NSObject <NSCoding>
 
 /// The authorized entity (also known as Sender ID), associated with the token.
 @property(nonatomic, readonly, copy) NSString *authorizedEntity;

FirebaseMessaging/Sources/Token/FIRMessagingTokenInfo.m

@@ -16,7 +16,6 @@
 
 #import "FirebaseMessaging/Sources/Token/FIRMessagingTokenInfo.h"
 
-#import <GoogleUtilities/GULSecureCoding.h>
 #import "FirebaseMessaging/Sources/FIRMessagingConstants.h"
 #import "FirebaseMessaging/Sources/FIRMessagingLogger.h"
 #import "FirebaseMessaging/Sources/FIRMessagingUtilities.h"
@@ -120,11 +119,7 @@ - (BOOL)isDefaultToken {
   return [self.scope isEqualToString:kFIRMessagingDefaultTokenScope];
 }
 
-#pragma mark - NSSecureCoding
-
-+ (BOOL)supportsSecureCoding {
-  return YES;
-}
+#pragma mark - NSCoding
 
 - (nullable instancetype)initWithCoder:(NSCoder *)aDecoder {
   // These value cannot be nil
@@ -145,6 +140,7 @@ - (nullable instancetype)initWithCoder:(NSCoder *)aDecoder {
   }
 
   // These values are nullable, so only fail the decode if the type does not match
+
   id appVersion = [aDecoder decodeObjectForKey:kFIRInstanceIDAppVersionKey];
   if (appVersion && ![appVersion isKindOfClass:[NSString class]]) {
     return nil;
@@ -162,15 +158,19 @@ - (nullable instancetype)initWithCoder:(NSCoder *)aDecoder {
 
   FIRMessagingAPNSInfo *APNSInfo = nil;
   if (rawAPNSInfo) {
-    NSError *error;
-    APNSInfo = [GULSecureCoding
-        unarchivedObjectOfClasses:[NSSet setWithObjects:FIRMessagingAPNSInfo.class, nil]
-                         fromData:rawAPNSInfo
-                            error:&error];
-    if (error) {
-      FIRMessagingLoggerDebug(
-          kFIRMessagingMessageCodeTokenInfoBadAPNSInfo,
-          @"Could not parse raw APNS Info while parsing unarchived token info: %@", error);
+    // TODO(chliangGoogle: Use the new API and secureCoding protocol.
+    @try {
+      [NSKeyedUnarchiver setClass:[FIRMessagingAPNSInfo class]
+                     forClassName:@"FIRInstanceIDAPNSInfo"];
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
+      APNSInfo = [NSKeyedUnarchiver unarchiveObjectWithData:rawAPNSInfo];
+#pragma clang diagnostic pop
+    } @catch (NSException *exception) {
+      FIRMessagingLoggerInfo(kFIRMessagingMessageCodeTokenInfoBadAPNSInfo,
+                             @"Could not parse raw APNS Info while parsing archived token info.");
+      APNSInfo = nil;
+    } @finally {
     }
   }
 
@@ -200,15 +200,15 @@ - (void)encodeWithCoder:(NSCoder *)aCoder {
   [aCoder encodeObject:self.firebaseAppID forKey:kFIRInstanceIDFirebaseAppIDKey];
   NSData *rawAPNSInfo;
   if (self.APNSInfo) {
-    NSError *error;
-    rawAPNSInfo = [GULSecureCoding archivedDataWithRootObject:self.APNSInfo error:&error];
-    if (error) {
-      FIRMessagingLoggerDebug(
-          kFIRMessagingMessageCodeTokenInfoBadAPNSInfo,
-          @"Could not parse raw APNS Info while parsing archived token info: %@", error);
-    }
+    // TODO(chliangGoogle: Use the new API and secureCoding protocol.
+    [NSKeyedArchiver setClassName:@"FIRInstanceIDAPNSInfo" forClass:[FIRMessagingAPNSInfo class]];
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
+    rawAPNSInfo = [NSKeyedArchiver archivedDataWithRootObject:self.APNSInfo];
+#pragma clang diagnostic pop
+
+    [aCoder encodeObject:rawAPNSInfo forKey:kFIRInstanceIDAPNSInfoKey];
   }
-  [aCoder encodeObject:[rawAPNSInfo copy] forKey:kFIRInstanceIDAPNSInfoKey];
   [aCoder encodeObject:self.cacheTime forKey:kFIRInstanceIDCacheTimeKey];
 }
 

FirebaseMessaging/Sources/Token/FIRMessagingTokenStore.m

@@ -16,7 +16,6 @@
 
 #import "FirebaseMessaging/Sources/Token/FIRMessagingTokenStore.h"
 
-#import <GoogleUtilities/GULSecureCoding.h>
 #import "FirebaseMessaging/Sources/FIRMessagingConstants.h"
 #import "FirebaseMessaging/Sources/FIRMessagingLogger.h"
 #import "FirebaseMessaging/Sources/FIRMessagingUtilities.h"
@@ -88,18 +87,21 @@ + (nullable FIRMessagingTokenInfo *)tokenInfoFromKeychainItem:(NSData *)item {
   // NOTE: Passing in nil to unarchiveObjectWithData will result in an iOS error logged
   // in the console on iOS 10 and below. Avoid by checking item.data's existence.
   if (item) {
-    NSError *unarchiverError;
-    tokenInfo = [GULSecureCoding
-        unarchivedObjectOfClasses:[NSSet
-                                      setWithObjects:FIRMessagingTokenInfo.class, NSDate.class, nil]
-                         fromData:item
-                            error:&unarchiverError];
-    if (unarchiverError) {
-      FIRMessagingLoggerDebug(kFIRMessagingMessageCodeTokenStoreUnarchivingTokenInfo,
+    // TODO(chliangGoogle: Use the new API and secureCoding protocol.
+    @try {
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
+      [NSKeyedUnarchiver setClass:[FIRMessagingTokenInfo class]
+                     forClassName:@"FIRInstanceIDTokenInfo"];
+      tokenInfo = [NSKeyedUnarchiver unarchiveObjectWithData:item];
+#pragma clang diagnostic pop
+
+    } @catch (NSException *exception) {
+      FIRMessagingLoggerDebug(kFIRMessagingMessageCodeTokenStoreExceptionUnarchivingTokenInfo,
                               @"Unable to parse token info from Keychain item; item was in an "
-                              @"invalid format %@",
-                              unarchiverError);
+                              @"invalid format");
       tokenInfo = nil;
+    } @finally {
     }
   }
   return tokenInfo;
@@ -113,13 +115,13 @@ - (void)saveTokenInfo:(FIRMessagingTokenInfo *)tokenInfo
               handler:(void (^)(NSError *))handler {  // Keep the cachetime up-to-date.
   tokenInfo.cacheTime = [NSDate date];
   // Always write to the Keychain, so that the cacheTime is up-to-date.
-  NSError *archiverError;
-  NSData *tokenInfoData = [GULSecureCoding archivedDataWithRootObject:tokenInfo
-                                                                error:&archiverError];
-  if (archiverError) {
-    FIRMessagingLoggerDebug(kFIRMessagingMessageCodeTokenStoreArchiveError,
-                            @"Failed to archive token info: %@", archiverError);
-  }
+  NSData *tokenInfoData;
+  // TODO(chliangGoogle: Use the new API and secureCoding protocol.
+  [NSKeyedArchiver setClassName:@"FIRInstanceIDTokenInfo" forClass:[FIRMessagingTokenInfo class]];
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
+  tokenInfoData = [NSKeyedArchiver archivedDataWithRootObject:tokenInfo];
+#pragma clang diagnostic pop
   NSString *account = FIRMessagingAppIdentifier();
   NSString *service = [[self class] serviceKeyForAuthorizedEntity:tokenInfo.authorizedEntity
                                                             scope:tokenInfo.scope];
@@ -128,15 +130,14 @@ - (void)saveTokenInfo:(FIRMessagingTokenInfo *)tokenInfo
 
 - (void)saveTokenInfoInCache:(FIRMessagingTokenInfo *)tokenInfo {
   tokenInfo.cacheTime = [NSDate date];
+  // TODO(chliangGoogle): Use the new API and secureCoding protocol.
   // Always write to the Keychain, so that the cacheTime is up-to-date.
-  NSError *archiverError;
-  NSData *tokenInfoData = [GULSecureCoding archivedDataWithRootObject:tokenInfo
-                                                                error:&archiverError];
-  if (archiverError) {
-    FIRMessagingLoggerDebug(kFIRMessagingMessageCodeTokenStoreArchiveError,
-                            @"Failed to archive token info: %@", archiverError);
-  }
-
+  NSData *tokenInfoData;
+  [NSKeyedArchiver setClassName:@"FIRInstanceIDTokenInfo" forClass:[FIRMessagingTokenInfo class]];
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
+  tokenInfoData = [NSKeyedArchiver archivedDataWithRootObject:tokenInfo];
+#pragma clang diagnostic pop
   NSString *account = FIRMessagingAppIdentifier();
   NSString *service = [[self class] serviceKeyForAuthorizedEntity:tokenInfo.authorizedEntity
                                                             scope:tokenInfo.scope];

FirebaseMessaging/Tests/UnitTests/FIRMessagingAPNSInfoTest.m

@@ -16,7 +16,6 @@
 
 #import <XCTest/XCTest.h>
 
-#import <GoogleUtilities/GULSecureCoding.h>
 #import "FirebaseMessaging/Sources/FIRMessagingConstants.h"
 #import "FirebaseMessaging/Sources/Token/FIRMessagingAPNSInfo.h"
 
@@ -77,13 +76,11 @@ - (void)testAPNSInfoEncodingAndDecoding {
   };
   FIRMessagingAPNSInfo *info =
       [[FIRMessagingAPNSInfo alloc] initWithTokenOptionsDictionary:validDictionary];
-  NSError *error;
-  NSData *archive = [GULSecureCoding archivedDataWithRootObject:info error:&error];
-  NSError *unarchiveError;
-  FIRMessagingAPNSInfo *restoredInfo = [GULSecureCoding
-      unarchivedObjectOfClasses:[NSSet setWithObjects:FIRMessagingAPNSInfo.class, nil]
-                       fromData:archive
-                          error:&unarchiveError];
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
+  NSData *archive = [NSKeyedArchiver archivedDataWithRootObject:info];
+  FIRMessagingAPNSInfo *restoredInfo = [NSKeyedUnarchiver unarchiveObjectWithData:archive];
+#pragma clang diagnostic pop
   XCTAssertEqualObjects(info.deviceToken, restoredInfo.deviceToken);
   XCTAssertEqual(info.sandbox, restoredInfo.sandbox);
 }

FirebaseMessaging/Tests/UnitTests/FIRMessagingAuthKeychainTest.m

@@ -18,12 +18,12 @@
 #if !TARGET_OS_MACCATALYST && !SWIFT_PACKAGE
 // Skip keychain tests on Catalyst and swift package
 
-#import <GoogleUtilities/GULSecureCoding.h>
 #import <XCTest/XCTest.h>
+#import "OCMock.h"
+
 #import "FirebaseMessaging/Sources/Token/FIRMessagingAuthKeychain.h"
 #import "FirebaseMessaging/Sources/Token/FIRMessagingCheckinPreferences.h"
 #import "FirebaseMessaging/Sources/Token/FIRMessagingTokenInfo.h"
-#import "OCMock.h"
 
 static NSString *const kFIRMessagingTestKeychainId = @"com.google.iid-tests";
 
@@ -406,8 +406,10 @@ - (NSData *)tokenDataWithAuthorizedEntity:(NSString *)authorizedEntity
                                                         token:token
                                                    appVersion:@"1.0"
                                                 firebaseAppID:kFirebaseAppID];
-  NSError *error;
-  return [GULSecureCoding archivedDataWithRootObject:tokenInfo error:&error];
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
+  return [NSKeyedArchiver archivedDataWithRootObject:tokenInfo];
+#pragma clang diagnostic pop
 }
 @end
 

FirebaseMessaging/Tests/UnitTests/FIRMessagingTokenInfoTest.m

@@ -16,8 +16,8 @@
 
 #import "FirebaseMessaging/Sources/Token/FIRMessagingTokenInfo.h"
 
-#import <GoogleUtilities/GULSecureCoding.h>
 #import <XCTest/XCTest.h>
+
 #import "FirebaseCore/Sources/Private/FirebaseCoreInternal.h"
 #import "FirebaseMessaging/Sources/FIRMessagingUtilities.h"
 #import "FirebaseMessaging/Sources/Token/FIRMessagingAPNSInfo.h"
@@ -76,14 +76,11 @@ - (void)tearDown {
 // yields the same values for all the fields.
 - (void)testTokenInfoEncodingAndDecoding {
   FIRMessagingTokenInfo *info = self.validTokenInfo;
-  NSError *archiverError;
-  NSData *archive = [GULSecureCoding archivedDataWithRootObject:info error:&archiverError];
-  NSError *unarchiverError;
-  FIRMessagingTokenInfo *restoredInfo =
-      [GULSecureCoding unarchivedObjectOfClasses:[NSSet setWithObjects:FIRMessagingTokenInfo.class,
-                                                                       NSDate.class, nil]
-                                        fromData:archive
-                                           error:&unarchiverError];
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wdeprecated-declarations"
+  NSData *archive = [NSKeyedArchiver archivedDataWithRootObject:info];
+  FIRMessagingTokenInfo *restoredInfo = [NSKeyedUnarchiver unarchiveObjectWithData:archive];
+#pragma clang diagnostic pop
   XCTAssertEqualObjects(restoredInfo.authorizedEntity, info.authorizedEntity);
   XCTAssertEqualObjects(restoredInfo.scope, info.scope);
   XCTAssertEqualObjects(restoredInfo.token, info.token);