Remove JS input validation for Storage

.changeset/tiny-hounds-rest.md

@@ -0,0 +1,6 @@
+---
+"firebase: major
+"@firebase/storage": major
+---
+
+This releases removes all input validation. Please use our TypeScript types to validate API usage. 

packages/storage/src/implementation/list.ts

@@ -20,7 +20,6 @@
  */
 import { Location } from './location';
 import * as json from './json';
-import * as type from './type';
 import { ListResult } from '../list';
 import { StorageService } from '../service';
 
@@ -43,9 +42,6 @@ interface ListResultResponse {
   nextPageToken?: string;
 }
 
-const MAX_RESULTS_KEY = 'maxResults';
-const MAX_MAX_RESULTS = 1000;
-const PAGE_TOKEN_KEY = 'pageToken';
 const PREFIXES_KEY = 'prefixes';
 const ITEMS_KEY = 'items';
 
@@ -92,28 +88,3 @@ export function fromResponseString(
   const resource = (obj as unknown) as ListResultResponse;
   return fromBackendResponse(service, bucket, resource);
 }
-
-export function listOptionsValidator(p: unknown): void {
-  if (!type.isObject(p) || !p) {
-    throw 'Expected ListOptions object.';
-  }
-  for (const key in p) {
-    if (key === MAX_RESULTS_KEY) {
-      if (
-        !type.isInteger(p[MAX_RESULTS_KEY]) ||
-        (p[MAX_RESULTS_KEY] as number) <= 0
-      ) {
-        throw 'Expected maxResults to be a positive number.';
-      }
-      if ((p[MAX_RESULTS_KEY] as number) > 1000) {
-        throw `Expected maxResults to be less than or equal to ${MAX_MAX_RESULTS}.`;
-      }
-    } else if (key === PAGE_TOKEN_KEY) {
-      if (p[PAGE_TOKEN_KEY] && !type.isString(p[PAGE_TOKEN_KEY])) {
-        throw 'Expected pageToken to be string.';
-      }
-    } else {
-      throw 'Unknown option: ' + key;
-    }
-  }
-}

packages/storage/src/implementation/metadata.ts

@@ -27,6 +27,7 @@ import * as type from './type';
 import * as UrlUtils from './url';
 import { Reference } from '../reference';
 import { StorageService } from '../service';
+import { Code, FirebaseStorageError } from './error';
 
 export function noXform_<T>(metadata: Metadata, value: T): T {
   return value;
@@ -206,20 +207,23 @@ export function toResourceString(
   return JSON.stringify(resource);
 }
 
-export function metadataValidator(p: unknown): void {
-  if (!type.isObject(p) || !p) {
-    throw 'Expected Metadata object.';
-  }
+export function validateMetadata(p: { [key: string]: unknown }): void {
   for (const key in p) {
     if (p.hasOwnProperty(key)) {
       const val = p[key];
       if (key === 'customMetadata') {
         if (!type.isObject(val)) {
-          throw "Expected object for 'customMetadata' mapping.";
+          throw new FirebaseStorageError(
+            Code.INVALID_ARGUMENT,
+            "Expected object for 'customMetadata' mapping."
+          );
         }
       } else {
         if (type.isNonNullObject(val)) {
-          throw "Mapping for '" + key + "' cannot be an object.";
+          throw new FirebaseStorageError(
+            Code.INVALID_ARGUMENT,
+            "Mapping for '" + key + "' cannot be an object."
+          );
         }
       }
     }

packages/storage/src/implementation/string.ts

@@ -27,28 +27,6 @@ export const StringFormat = {
   DATA_URL: 'data_url'
 };
 
-export function formatValidator(stringFormat: unknown): void {
-  switch (stringFormat) {
-    case StringFormat.RAW:
-    case StringFormat.BASE64:
-    case StringFormat.BASE64URL:
-    case StringFormat.DATA_URL:
-      return;
-    default:
-      throw (
-        'Expected one of the event types: [' +
-        StringFormat.RAW +
-        ', ' +
-        StringFormat.BASE64 +
-        ', ' +
-        StringFormat.BASE64URL +
-        ', ' +
-        StringFormat.DATA_URL +
-        '].'
-      );
-  }
-}
-
 /**
  * @struct
  */

packages/storage/src/implementation/type.ts

@@ -47,14 +47,6 @@ export function isString(p: unknown): p is string {
   return typeof p === 'string' || p instanceof String;
 }
 
-export function isInteger(p: unknown): p is number {
-  return isNumber(p) && Number.isInteger(p);
-}
-
-export function isNumber(p: unknown): p is number {
-  return typeof p === 'number' || p instanceof Number;
-}
-
 export function isNativeBlob(p: unknown): p is Blob {
   return isNativeBlobDefined() && p instanceof Blob;
 }