Updated firebase/util to check for hostname

Author: maneesht

.changeset/brown-pens-confess.md

@@ -1,5 +1,6 @@
 ---
 "@firebase/storage": patch
+"@firebase/util": patch
 ---
 
 Fixed issue where Storage on Firebase Studio throws CORS errors.

common/api-review/util.api.md

@@ -270,7 +270,7 @@ export function isBrowserExtension(): boolean;
 export function isCloudflareWorker(): boolean;
 
 // @public
-export function isCloudWorkstation(host: string): boolean;
+export function isCloudWorkstation(url: string): boolean;
 
 // Warning: (ae-missing-release-tag) "isElectron" is exported by the package, but it is missing a release tag (@alpha, @beta, @public, or @internal)
 //

packages/storage/src/platform/browser/connection.ts

@@ -70,7 +70,7 @@ abstract class XhrConnection<T extends ConnectionType>
     if (this.sent_) {
       throw internalError('cannot .send() more than once');
     }
-    if (isCloudWorkstation(new URL(url).hostname) && isUsingEmulator) {
+    if (isCloudWorkstation(url) && isUsingEmulator) {
       this.xhr_.withCredentials = true;
     }
     this.sent_ = true;

packages/util/src/url.ts

@@ -19,7 +19,11 @@
  * Checks whether host is a cloud workstation or not.
  * @public
  */
-export function isCloudWorkstation(host: string): boolean {
+export function isCloudWorkstation(url: string): boolean {
+  const host =
+    url.startsWith('http://') || url.startsWith('https://')
+      ? new URL(url).hostname
+      : url;
   return host.endsWith('.cloudworkstations.dev');
 }