fix: get default vhost if empty string is provided

Author: l4mby

src/client.ts

@@ -704,7 +704,10 @@ export class Client {
   }
 
   static async connect(params: ClientParams, logger?: Logger): Promise<Client> {
-    return new Client(logger ?? new NullLogger(), params).start()
+    return new Client(logger ?? new NullLogger(), {
+      ...params,
+      vhost: getVhostOrDefault(params.vhost),
+    }).start()
   }
 }
 
@@ -837,3 +840,5 @@ const extractConsumerId = (extendedConsumerId: string) => {
 const extractPublisherId = (extendedPublisherId: string) => {
   return parseInt(extendedPublisherId.split("@").shift() ?? "0")
 }
+
+const getVhostOrDefault = (vhost: string) => (vhost === "" ? "/" : vhost)

test/e2e/connect.test.ts

@@ -46,6 +46,23 @@ describe("connect", () => {
     }, 5000)
   }).timeout(10000)
 
+  it("with default vhost as '/' ", async () => {
+    const [firstNode] = getTestNodesFromEnv()
+    client = await connect({
+      hostname: firstNode.host,
+      port: firstNode.port,
+      username,
+      password,
+      vhost: "",
+      frameMax: 0,
+      heartbeat: 0,
+    })
+
+    await eventually(async () => {
+      expect(await rabbit.getConnections()).lengthOf(1)
+    }, 5000)
+  }).timeout(10000)
+
   it("using EXTERNAL auth", async () => {
     client = await createTlsClient()