Storage: Do not throw for path containing ".." (#5315)

Author: hsubox76

.changeset/big-lobsters-tell.md

@@ -0,0 +1,5 @@
+---
+'@firebase/storage': patch
+---
+
+Change `ref()` to not throw if given a path with '..' in it.

packages/storage/src/service.ts

@@ -74,9 +74,6 @@ function refFromPath(
   } else {
     // ref is a Reference
     if (path !== undefined) {
-      if (path.includes('..')) {
-        throw invalidArgument('`path` param cannot contain ".."');
-      }
       return _getChild(ref, path);
     } else {
       return ref;

packages/storage/test/unit/service.exp.test.ts

@@ -372,12 +372,6 @@ GOOG4-RSA-SHA256`
       const newRef = ref(reference);
       expect(newRef.toString()).to.equal('gs://bucket/object');
     });
-    it('Throws calling ref(reference, path) if path contains ".."', () => {
-      const error = testShared.assertThrows(() => {
-        ref(reference, `../child/path`);
-      }, 'storage/invalid-argument');
-      expect(error.message).to.match(/"\.\."/);
-    });
   });
 
   describe('Deletion', () => {