@quakejs/master 1.0.18 → 1.0.19

package/master-client.js

@@ -18,15 +18,9 @@ class MasterClient extends MasterBase {
 
     (async () => {
       try {
-        const sessionClosed = session.closed.then(
-          () => ({ done: true, value: null }),
-          (err) => { throw err; }
-        );
-
         await session.ready;
 
         const writer = session.datagrams.writable.getWriter();
-        const reader = session.datagrams.readable.getReader();
         let resendTimeout;
 
         /* send until a response is received */
@@ -39,18 +33,9 @@ class MasterClient extends MasterBase {
         send();
 
         /* read responses */
-        while (true) {
-          const { done, value } = await Promise.race([
-            reader.read(),
-            sessionClosed
-          ]);
-
+        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);
@@ -80,21 +65,15 @@ class MasterClient extends MasterBase {
     session.closed.catch(() => {});
 
     try {
-      const sessionClosed = session.closed.then(
-        () => ({ done: true, value: null }),
-        (err) => { throw err; }
-      );
+      let begin;
 
       await session.ready;
 
-      const writer = session.datagrams.writable.getWriter();
-      const reader = session.datagrams.readable.getReader();
-      let begin;
-
       /* send request multiple times until something is read back */
       const send = () => {
         begin = performance.now();
 
+        const writer = session.datagrams.writable.getWriter();
         writer.write(this.encode('getinfo')).catch(() => {});
 
         resendTimeout = setTimeout(send, 3000);
@@ -103,28 +82,23 @@ class MasterClient extends MasterBase {
       send();
 
       /* read response */
-      const { done, value } = await Promise.race([
-        reader.read(),
-        sessionClosed
-      ]);
-
-      clearTimeout(resendTimeout);
+      for await (const value of session.datagrams.readable) {
+        clearTimeout(resendTimeout);
 
-      if (done) {
-        throw new Error('stream closed');
-      }
+        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('invalid response type');
+        }
 
-      if (!msg || msg.type !== 'infoResponse') {
-        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 = msg.info;
 
-      res = msg.info;
+        break;
+      }
     } catch (e) {
       console.error('getinfo failed', e);
       throw e;
@@ -146,21 +120,15 @@ class MasterClient extends MasterBase {
     session.closed.catch(() => {});
 
     try {
-      const sessionClosed = session.closed.then(
-        () => ({ done: true, value: null }),
-        (err) => { throw err; }
-      );
+      let begin;
 
       await session.ready;
 
-      const writer = session.datagrams.writable.getWriter();
-      const reader = session.datagrams.readable.getReader();
-      let begin;
-
       /* send request multiple times until something is read back */
       const send = () => {
         begin = performance.now();
 
+        const writer = session.datagrams.writable.getWriter();
         writer.write(this.encode('getstatus')).catch(() => {});
 
         resendTimeout = setTimeout(send, 3000);
@@ -169,28 +137,23 @@ class MasterClient extends MasterBase {
       send();
 
       /* read response */
-      const { done, value } = await Promise.race([
-        reader.read(),
-        sessionClosed
-      ]);
-
-      clearTimeout(resendTimeout);
+      for await (const value of session.datagrams.readable) {
+        clearTimeout(resendTimeout);
 
-      if (done) {
-        throw new Error('stream closed');
-      }
+        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('invalid response type');
+        }
 
-      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 };
 
-      res = { info: msg.info, players: msg.players };
+        break;
+      }
     } catch (e) {
       console.error('getstatus failed', e);
       throw e;

package/master-server.js

@@ -146,33 +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 {
-              const sessionClosed = session.closed.then(
-                () => ({ done: true, value: null }),
-                (err) => { throw err; }
-              );
-
               await session.ready;
 
               const writer = session.datagrams.writable.getWriter();
-              const reader = session.datagrams.readable.getReader();
               const [addr, port] = session.peerAddress.split(':');
               let first = true;
 
@@ -183,16 +169,7 @@ class MasterServer extends MasterBase {
                 writer.write(data).catch(e => {});
               };
 
-              while (true) {
-                const { done, value } = await Promise.race([
-                  reader.read(),
-                  sessionClosed
-                ]);
-
-                if (done) {
-                  break;
-                }
-
+              for await (const value of session.datagrams.readable) {
                 console.log(`${session.addr}:${session.port} ---> ${this.pretty(value)}`);
 
                 if (first) {
@@ -227,8 +204,9 @@ class MasterServer extends MasterBase {
                 }
               }
             } catch (e) {
-              console.error(`Client ${session.addr}:${session.port} terminated`, 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.18",
+  "version": "1.0.19",
   "type": "module",
   "main": "index.js",
   "bin": {