Fix websocketonly flag

.changeset/twenty-shoes-cheer.md

@@ -0,0 +1,5 @@
+---
+"@firebase/database": patch
+---
+
+Fix websocketonly field to be set to true when using websocket db url.

CONTRIBUTING.md

@@ -77,7 +77,7 @@ Before you start working on a larger contribution, you should get in touch with
 
 * Create your patch, **including appropriate test cases**. Patches with tests are more likely to be merged.
 * Avoid checking in files that shouldn't be tracked (e.g `node_modules`, `gulp-cache`, `.tmp`, `.idea`). If your development setup automatically creates some of these files, please add them to the `.gitignore` at the root of the package (click [here][gitignore] to read more on how to add entries to the `.gitignore`).
-* Commit your changes using a commit message that follows our [commit message guidelines](#commit-message-guidelines).
+* Commit your changes
 
      ```shell
      git commit -a

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

@@ -92,8 +92,8 @@ export const parseRepoInfo = function (
       parsedUrl.host,
       parsedUrl.secure,
       namespace,
-      nodeAdmin,
       webSocketOnly,
+      nodeAdmin,
       /*persistenceKey=*/ '',
       /*includeNamespaceInQueryParams=*/ namespace !== parsedUrl.subdomain
     ),

packages/database/test/parser.test.ts

@@ -0,0 +1,31 @@
+/**
+ * @license
+ * Copyright 2017 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import { expect } from "chai";
+
+import { parseRepoInfo } from "../src/core/util/libs/parser";
+
+describe('parser', () => {
+    it('should set websocketUrl correctly based on the protocol', () => {
+        const httpsRepoInfo = parseRepoInfo('https://test-ns.firebaseio.com', false);
+        expect(httpsRepoInfo.repoInfo.webSocketOnly).to.equal(false);
+        const wssRepoInfo = parseRepoInfo('wss://test-ns.firebaseio.com', false);
+        expect(wssRepoInfo.repoInfo.webSocketOnly).to.equal(true);
+        const wsRepoInfo = parseRepoInfo('ws://test-ns.firebaseio.com', false);
+        expect(wsRepoInfo.repoInfo.webSocketOnly).to.equal(true);
+    });
+});