pass

Author: fredzqm

packages/database/src/core/util/libs/parser.ts

@@ -181,6 +181,8 @@ export const parseDatabaseURL = function(
     const hostFirstPart = host.substring(0, dotInd);
     if (hostFirstPart.toLowerCase() === 'localhost') {
       domain = 'localhost';
+    } else if (dotInd === colonInd) {
+      return; // Not a valid URL.
     } else {
       domain = host.substring(dotInd+1);
       // Normalize namespaces to lowercase to share storage / connection.

packages/database/test/database.test.ts

@@ -50,12 +50,6 @@ describe('Database Tests', () => {
     expect(db.ref().toString()).to.equal('https://foo.bar.com/');
   });
 
-  it('Can get database with custom URL', () => {
-    const db = defaultApp.database('http://foo.euw1.firebasedatabase.app');
-    expect(db).to.be.ok;
-    expect(db.ref().toString()).to.equal('https://foo.euw1.firebasedatabase.app/');
-  });
-
   it('Can get database with custom URL and port', () => {
     const db = defaultApp.database('http://foo.bar.com:80');
     expect(db).to.be.ok;
@@ -68,6 +62,13 @@ describe('Database Tests', () => {
     expect(db.ref().toString()).to.equal('https://foo.bar.com/');
   });
 
+  it('Can get database with multi-region URL', () => {
+    const db = defaultApp.database('http://foo.euw1.firebasedatabase.app');
+    expect(db).to.be.ok;
+    expect(db.repo_.repoInfo_.namespace).to.equal('foo');
+    expect(db.ref().toString()).to.equal('https://foo.euw1.firebasedatabase.app/');
+  });
+
   it('Can get database with localhost URL and port', () => {
     const db = defaultApp.database('http://localhost:80');
     expect(db).to.be.ok;