@react-native-firebase/storage 14.11.1 → 14.11.2

package/CHANGELOG.md

@@ -3,6 +3,13 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [14.11.2](https://github.com/invertase/react-native-firebase/compare/v14.11.1...v14.11.2) (2022-09-17)
+
+### Bug Fixes
+
+- **storage, ios:** correct storage metadata update / delete ([2dcb079](https://github.com/invertase/react-native-firebase/commit/2dcb0790c1812a33100cceea9dcb407d6a64cb87))
+- **storage, ios:** surface underlying reason for unknown errors if possible ([6cd53ea](https://github.com/invertase/react-native-firebase/commit/6cd53eaca16ef52c52a28a7b209a7c8313fef08b))
+
 ## [14.11.1](https://github.com/invertase/react-native-firebase/compare/v14.11.0...v14.11.1) (2022-06-17)
 
 **Note:** Version bump only for package @react-native-firebase/storage

package/ios/RNFBStorage/RNFBStorageCommon.h

@@ -56,7 +56,8 @@
 
 + (NSString *)getTaskStatus:(FIRStorageTaskStatus)status;
 
-+ (FIRStorageMetadata *)buildMetadataFromMap:(NSDictionary *)metadata;
++ (FIRStorageMetadata *)buildMetadataFromMap:(NSDictionary *)metadata
+                            existingMetadata:(FIRStorageMetadata *)existingMetadata;
 
 + (NSArray *)getErrorCodeMessage:(NSError *)error;
 

package/ios/RNFBStorage/RNFBStorageCommon.m

@@ -392,9 +392,64 @@
   }
 }
 
-+ (FIRStorageMetadata *)buildMetadataFromMap:(NSDictionary *)metadata {
-  FIRStorageMetadata *storageMetadata = [[FIRStorageMetadata alloc] initWithDictionary:metadata];
-  storageMetadata.customMetadata = [metadata[@"customMetadata"] mutableCopy];
++ (FIRStorageMetadata *)buildMetadataFromMap:(NSDictionary *)metadata
+                            existingMetadata:(nullable FIRStorageMetadata *)existingMetadata {
+  // If an existing metadata was passed in, modify it with our map, otherwise init a fresh copy
+  FIRStorageMetadata *storageMetadata = existingMetadata;
+  if (storageMetadata == nil) {
+    storageMetadata = [[FIRStorageMetadata alloc] init];
+  }
+
+  if (metadata[@"cacheControl"] == [NSNull null]) {
+    storageMetadata.cacheControl = nil;
+  } else {
+    storageMetadata.cacheControl = metadata[@"cacheControl"];
+  }
+
+  if (metadata[@"contentLanguage"] == [NSNull null]) {
+    storageMetadata.contentLanguage = nil;
+  } else {
+    storageMetadata.contentLanguage = metadata[@"contentLanguage"];
+  }
+
+  if (metadata[@"contentEncoding"] == [NSNull null]) {
+    storageMetadata.contentEncoding = nil;
+  } else {
+    storageMetadata.contentEncoding = metadata[@"contentEncoding"];
+  }
+
+  if (metadata[@"contentDisposition"] == [NSNull null]) {
+    storageMetadata.contentDisposition = nil;
+  } else {
+    storageMetadata.contentDisposition = metadata[@"contentDisposition"];
+  }
+
+  if (metadata[@"contentType"] == [NSNull null]) {
+    storageMetadata.contentType = nil;
+  } else {
+    storageMetadata.contentType = metadata[@"contentType"];
+  }
+
+  if (metadata[@"customMetadata"] == [NSNull null]) {
+    storageMetadata.customMetadata = @{};
+  } else {
+    NSMutableDictionary *customMetadata = [metadata[@"customMetadata"] mutableCopy];
+    for (NSString *key in customMetadata.allKeys) {
+      if (customMetadata[key] == [NSNull null] || customMetadata[key] == nil) {
+        [customMetadata removeObjectForKey:key];
+      }
+    }
+    storageMetadata.customMetadata = customMetadata;
+  }
+
+  // md5hash may be settable but is not update-able, so just test for existence and carry it in
+  // FIXME this will need a fix related to
+  // https://github.com/firebase/firebase-ios-sdk/issues/9849#issuecomment-1159292592 if
+  // (metadata[@"md5hash"]) {
+  //   NSLog(@"STORAGE md5hash was set");
+  //   storageMetadata.md5Hash = metadata[@"md5hash"];
+  // }
+
   return storageMetadata;
 }
 
@@ -417,7 +472,13 @@
         code = @"invalid-device-file-path";
         message = @"The specified device file path is invalid or is restricted.";
       } else {
-        message = @"An unknown error has occurred.";
+        if (userInfo[@"ResponseBody"]) {
+          message =
+              [NSString stringWithFormat:@"An unknown error has occurred. (underlying reason '%@')",
+                                         userInfo[@"ResponseBody"]];
+        } else {
+          message = @"An unknown error has occurred.";
+        }
       }
       break;
     case FIRStorageErrorCodeObjectNotFound:

package/ios/RNFBStorage/RNFBStorageModule.m

@@ -143,17 +143,26 @@ RCT_EXPORT_METHOD(updateMetadata
                   : (RCTPromiseResolveBlock)resolve
                   : (RCTPromiseRejectBlock)reject) {
   FIRStorageReference *storageReference = [self getReferenceFromUrl:url app:firebaseApp];
-  FIRStorageMetadata *storageMetadata = [RNFBStorageCommon buildMetadataFromMap:metadata];
 
-  [storageReference
-      updateMetadata:storageMetadata
-          completion:^(FIRStorageMetadata *_Nullable metadata, NSError *_Nullable error) {
-            if (error != nil) {
-              [self promiseRejectStorageException:reject error:error];
-            } else {
-              resolve([RNFBStorageCommon metadataToDict:metadata]);
-            }
-          }];
+  [storageReference metadataWithCompletion:^(FIRStorageMetadata *_Nullable fetchedMetadata,
+                                             NSError *_Nullable error) {
+    if (error != nil) {
+      [self promiseRejectStorageException:reject error:error];
+    } else {
+      FIRStorageMetadata *storageMetadata =
+          [RNFBStorageCommon buildMetadataFromMap:metadata existingMetadata:fetchedMetadata];
+
+      [storageReference updateMetadata:storageMetadata
+                            completion:^(FIRStorageMetadata *_Nullable updatedMetadata,
+                                         NSError *_Nullable error) {
+                              if (error != nil) {
+                                [self promiseRejectStorageException:reject error:error];
+                              } else {
+                                resolve([RNFBStorageCommon metadataToDict:updatedMetadata]);
+                              }
+                            }];
+    }
+  }];
 }
 
 /**
@@ -391,7 +400,8 @@ RCT_EXPORT_METHOD(putFile
                   : (nonnull NSNumber *)taskId
                   : (RCTPromiseResolveBlock)resolve
                   : (RCTPromiseRejectBlock)reject) {
-  FIRStorageMetadata *storageMetadata = [RNFBStorageCommon buildMetadataFromMap:metadata];
+  FIRStorageMetadata *storageMetadata = [RNFBStorageCommon buildMetadataFromMap:metadata
+                                                               existingMetadata:nil];
   FIRStorageReference *storageReference = [self getReferenceFromUrl:url app:firebaseApp];
 
   [RNFBStorageCommon
@@ -462,7 +472,8 @@ RCT_EXPORT_METHOD(putString
                   : (nonnull NSNumber *)taskId
                   : (RCTPromiseResolveBlock)resolve
                   : (RCTPromiseRejectBlock)reject) {
-  FIRStorageMetadata *storageMetadata = [RNFBStorageCommon buildMetadataFromMap:metadata];
+  FIRStorageMetadata *storageMetadata = [RNFBStorageCommon buildMetadataFromMap:metadata
+                                                               existingMetadata:nil];
   FIRStorageReference *storageReference = [self getReferenceFromUrl:url app:firebaseApp];
 
   __block FIRStorageUploadTask *uploadTask;

package/lib/utils.js

@@ -88,9 +88,9 @@ export function validateMetadata(metadata, update = true) {
           `firebase.storage.SettableMetadata invalid property '${key}' should be a string or null value.`,
         );
       }
-    } else if (!isObject(value)) {
+    } else if (!isObject(value) && !isNull(value)) {
       throw new Error(
-        'firebase.storage.SettableMetadata.customMetadata must be an object of keys and string values.',
+        'firebase.storage.SettableMetadata.customMetadata must be an object of keys and string values or null value.',
       );
     }
   }

package/lib/version.js

@@ -1,2 +1,2 @@
 // Generated by genversion.
-module.exports = '14.11.1';
+module.exports = '14.11.2';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-firebase/storage",
-  "version": "14.11.1",
+  "version": "14.11.2",
   "author": "Invertase <oss@invertase.io> (http://invertase.io)",
   "description": "React Native Firebase - React Native Firebase provides native integration with Cloud Storage, providing support to upload and download files directly from your device and from your Firebase Cloud Storage bucket.",
   "main": "lib/index.js",
@@ -29,10 +29,10 @@
     "download"
   ],
   "peerDependencies": {
-    "@react-native-firebase/app": "14.11.1"
+    "@react-native-firebase/app": "14.11.2"
   },
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "b1a430bbf36a41076b33550fec113fa0e44583b4"
+  "gitHead": "1da8ab19fe93ae1b61c0367e12acf0c999afc06a"
 }