@quakejs/master 1.0.7 → 1.0.9

package/master-base.js

@@ -1,26 +1,3 @@
-function generateChallenge () {
-  const CHALLENGE_CHARS = [
-    '!', '#', '$', '&', '\'', '(', ')', '*', '+', ',', '-', '.', '0', '1', '2', '3', '4', '5', '6', '7', '8', '9', ':', '<', '=', '>', '?', '@',
-    'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z',
-    '[', ']', '^', '_', '`',
-    'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z',
-    '{', '|', '}', '~'
-  ];
-  const CHALLENGE_MIN_LENGTH = 9;
-  const CHALLENGE_MAX_LENGTH = 12;
-
-  let challenge = '';
-  let length = CHALLENGE_MIN_LENGTH - 1;
-
-  length += parseInt(Math.random() * (CHALLENGE_MAX_LENGTH - CHALLENGE_MIN_LENGTH + 1), 10);
-
-  for (let i = 0; i < length; i++) {
-    challenge += CHALLENGE_CHARS[Math.floor(Math.random() * CHALLENGE_CHARS.length)];
-  }
-
-  return challenge;
-}
-
 function parseInfoString (str) {
   const info = {};
   let i = 0;
@@ -112,14 +89,24 @@ class Subscribe extends Message {
 class GetInfo extends Message {
   static id = 'getinfo';
 
-  constructor (msg) {
+  constructor (from) {
     super();
 
-    this.challenge = generateChallenge();
+    if (typeof from === 'string') {
+      this.challenge = from.substring(12);
+    } else {
+      this.challenge = from.challenge;
+    }
   }
 
   toString () {
-    return `${this.type} ${this.challenge}`;
+    let str = this.type;
+
+    if (this.challenge) {
+      str += ` ${this.challenge}`;
+    }
+
+    return str;
   }
 }
 
@@ -328,6 +315,29 @@ class MasterBase {
 
     return out;
   }
+
+  generateChallenge () {
+    const CHALLENGE_CHARS = [
+      '!', '#', '$', '&', '\'', '(', ')', '*', '+', ',', '-', '.', '0', '1', '2', '3', '4', '5', '6', '7', '8', '9', ':', '<', '=', '>', '?', '@',
+      'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z',
+      '[', ']', '^', '_', '`',
+      'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z',
+      '{', '|', '}', '~'
+    ];
+    const CHALLENGE_MIN_LENGTH = 9;
+    const CHALLENGE_MAX_LENGTH = 12;
+
+    let challenge = '';
+    let length = CHALLENGE_MIN_LENGTH - 1;
+
+    length += parseInt(Math.random() * (CHALLENGE_MAX_LENGTH - CHALLENGE_MIN_LENGTH + 1), 10);
+
+    for (let i = 0; i < length; i++) {
+      challenge += CHALLENGE_CHARS[Math.floor(Math.random() * CHALLENGE_CHARS.length)];
+    }
+
+    return challenge;
+  }
 }
 
 export default MasterBase;

package/master-server.js

@@ -118,7 +118,8 @@ class MasterServer extends MasterBase {
       return;
     }
 
-    const req = this.encode('getinfo');
+    const challenge = this.generateChallenge();
+    const req = this.encode('getinfo', { challenge });
 
     console.log(`${session.addr}:${session.port} <--- ${this.pretty(req)}`);
 
@@ -195,7 +196,7 @@ class MasterServer extends MasterBase {
               case 'getservers':
                 this.#handleGetServers(session, msg);
                 break;
-              case 'getinfoResponse':
+              case 'infoResponse':
                 this.#handleInfoResponse(session, msg);
                 break;
               default:

package/package.json

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