@quakejs/master 1.0.17 → 1.0.19

package/master-client.js

@@ -2,40 +2,30 @@ import MasterBase from './master-base.js';
 
 class MasterClient extends MasterBase {
   #onserver;
-  #master;
 
   constructor (onserver) {
     super();
 
     this.#onserver = onserver;
-    this.#master = null;
   }
 
   subscribe (host, port) {
-    let resendTimeout;
-
-    if (this.#master) {
-      try {
-        this.#master.close();
-      } catch (e) {
-        console.error(e);
-      } finally {
-        this.#master = null;
-      }
-    }
+    const session = new WebTransport(`${host}:${port}`);
 
-    this.#master = new WebTransport(`${host}:${port}`);
+    /* prevent unhandled rejections before main loop */
+    session.ready.catch(() => {});
+    session.closed.catch(() => {});
 
     (async () => {
       try {
-        await this.#master.ready;
+        await session.ready;
 
-        const writer = this.#master.datagrams.writable.getWriter();
-        const reader = this.#master.datagrams.readable.getReader();
+        const writer = session.datagrams.writable.getWriter();
+        let resendTimeout;
 
         /* send until a response is received */
         const send = () => {
-          writer.write(this.encode('subscribe'));
+          writer.write(this.encode('subscribe')).catch(() => {});
 
           resendTimeout = setTimeout(send, 1000);
         };
@@ -43,15 +33,9 @@ class MasterClient extends MasterBase {
         send();
 
         /* read responses */
-        while (true) {
-          const { done, value } = await reader.read();
-
+        for await (const value of session.datagrams.readable) {
           clearTimeout(resendTimeout);
 
-          if (done) {
-            break;
-          }
-
           console.log(`${host}:${port} ---> ${this.pretty(value)}`);
 
           const msg = this.decode(value);
@@ -62,100 +46,124 @@ class MasterClient extends MasterBase {
             }
           }
         }
+
+        console.log(`Disconnected from master server`);
       } catch (e) {
-        console.error(`Stream error, connection to ${host}:${port} lost`, e);
+        console.error('Disconnected from master server due to error', e);
       } finally {
-        this.#master = null;
+        clearTimeout(resendTimeout);
       }
     })();
   }
 
   async getinfo (addr, port) {
     const session = new WebTransport(`https://${addr}:${port}`);
-    const writer = session.datagrams.writable.getWriter();
-    const reader = session.datagrams.readable.getReader();
     let resendTimeout;
-    let begin;
+    let res;
 
-    await session.ready;
+    session.ready.catch(() => {});
+    session.closed.catch(() => {});
 
-    /* send request multiple times until something is read back */
-    const send = () => {
-      begin = performance.now();
+    try {
+      let begin;
 
-      writer.write(this.encode('getinfo'));
+      await session.ready;
 
-      resendTimeout = setTimeout(send, 3000);
-    };
+      /* send request multiple times until something is read back */
+      const send = () => {
+        begin = performance.now();
 
-    send();
+        const writer = session.datagrams.writable.getWriter();
+        writer.write(this.encode('getinfo')).catch(() => {});
 
-    /* read response */
-    const { done, value } = await reader.read();
+        resendTimeout = setTimeout(send, 3000);
+      };
 
-    clearTimeout(resendTimeout);
+      send();
 
-    if (done) {
-      throw new Error('getinfo failed, no response received');
-    }
+      /* read response */
+      for await (const value of session.datagrams.readable) {
+        clearTimeout(resendTimeout);
 
-    console.log(`${addr}:${port} ---> ${this.pretty(value)}`);
+        console.log(`${addr}:${port} ---> ${this.pretty(value)}`);
 
-    const msg = this.decode(value);
+        const msg = this.decode(value);
 
-    if (!msg || msg.type !== 'infoResponse') {
-      throw new Error('getinfo failed, didn\'t receive infoResponse');
-    }
+        if (!msg || msg.type !== 'infoResponse') {
+          throw new Error('invalid response type');
+        }
+
+        msg.info.ping = Math.min((performance.now() - begin) >>> 0, 999);
+
+        res = msg.info;
 
-    msg.info.ping = Math.min((performance.now() - begin) >>> 0, 999);
+        break;
+      }
+    } catch (e) {
+      console.error('getinfo failed', e);
+      throw e;
+    } finally {
+      clearTimeout(resendTimeout);
 
-    session.close();
+      session.close();
+    }
 
-    return msg.info;
+    return res;
   }
 
   async getstatus (addr, port) {
     const session = new WebTransport(`https://${addr}:${port}`);
-    const writer = session.datagrams.writable.getWriter();
-    const reader = session.datagrams.readable.getReader();
     let resendTimeout;
-    let begin;
+    let res;
 
-    await session.ready;
+    session.ready.catch(() => {});
+    session.closed.catch(() => {});
 
-    /* send request multiple times until something is read back */
-    const send = () => {
-      begin = performance.now();
+    try {
+      let begin;
 
-      writer.write(this.encode('getstatus'));
+      await session.ready;
 
-      resendTimeout = setTimeout(send, 3000);
-    };
+      /* send request multiple times until something is read back */
+      const send = () => {
+        begin = performance.now();
 
-    send();
+        const writer = session.datagrams.writable.getWriter();
+        writer.write(this.encode('getstatus')).catch(() => {});
 
-    /* read response */
-    const { done, value } = await reader.read();
+        resendTimeout = setTimeout(send, 3000);
+      };
 
-    clearTimeout(resendTimeout);
+      send();
 
-    if (done) {
-      throw new Error('getstatus failed, no response received');
-    }
+      /* read response */
+      for await (const value of session.datagrams.readable) {
+        clearTimeout(resendTimeout);
 
-    console.log(`${addr}:${port} ---> ${this.pretty(value)}`);
+        console.log(`${addr}:${port} ---> ${this.pretty(value)}`);
 
-    const msg = this.decode(value);
+        const msg = this.decode(value);
 
-    if (!msg || msg.type !== 'statusResponse') {
-      throw new Error('getstatus failed, didn\'t receive statusResponse');
-    }
+        if (!msg || msg.type !== 'statusResponse') {
+          throw new Error('invalid response type');
+        }
+
+        msg.info.ping = Math.min((performance.now() - begin) >>> 0, 999);
 
-    msg.info.ping = Math.min((performance.now() - begin) >>> 0, 999);
+        res = { info: msg.info, players: msg.players };
 
-    session.close();
+        break;
+      }
+    } catch (e) {
+      console.error('getstatus failed', e);
+      throw e;
+    } finally {
+      clearTimeout(resendTimeout);
+
+      session.close();
+    }
 
-    return { info: msg.info, players: msg.players };
+    return res;
   }
 }
 

package/master-server.js

@@ -146,24 +146,19 @@ class MasterServer extends MasterBase {
         await this.#h3.ready;
 
         const sessionStream = await this.#h3.sessionStream('/');
-        const sessionReader = sessionStream.getReader();
 
         console.log(`listening on ${this.#h3.host}:${this.#h3.port}`);
 
-        while (true) {
-          const { done, value } = await sessionReader.read();
-          const session = value;
-
-          if (done) {
-            break;
-          }
+        for await (const session of sessionStream) {
+          /* prevent unhandled rejections before main loop */
+          session.ready.catch(() => {});
+          session.closed.catch(() => {});
 
           (async () => {
             try {
               await session.ready;
 
-              const writer = session.datagrams.createWritable().getWriter();
-              const reader = session.datagrams.readable.getReader();
+              const writer = session.datagrams.writable.getWriter();
               const [addr, port] = session.peerAddress.split(':');
               let first = true;
 
@@ -171,16 +166,10 @@ class MasterServer extends MasterBase {
               session.port = port;
 
               session.write = (data) => {
-                writer.write(data);
+                writer.write(data).catch(e => {});
               };
 
-              while (true) {
-                const { done, value } = await reader.read();
-
-                if (done) {
-                  break;
-                }
-
+              for await (const value of session.datagrams.readable) {
                 console.log(`${session.addr}:${session.port} ---> ${this.pretty(value)}`);
 
                 if (first) {
@@ -214,11 +203,10 @@ class MasterServer extends MasterBase {
                     break;
                 }
               }
-
-              console.log(`Client ${session.addr}:${session.port} disconnected`);
             } catch (e) {
-              console.error(`Client ${session.addr}:${session.port} disconnected due to error`, e);
+              console.error(`Session ${session.addr}:${session.port} terminated`, e);
             } finally {
+              console.log(`Removing client ${session.addr}:${session.port}`);
               this.#removeClient(session);
             }
           })();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quakejs/master",
-  "version": "1.0.17",
+  "version": "1.0.19",
   "type": "module",
   "main": "index.js",
   "bin": {