feat: add Renegade X support

[RattleSN4K3]

Adds support for Renegade X, using master server
Related issues: #629

Note

Setup and Response updated to reflect made changes during developing this PR.

Query single server

Setup:

GameDig.query({
  type: 'renegade10',
  address: '1.0.0.1',
  port: 7778

Example response

{
  "name": "[PREFIX] Server Name",
  "map": "CNC-Field",
  "password": false,
  "raw": {
    "Name": "Server Name",
    "NamePrefix": "[PREFIX]",
    "Current Map": "CNC-Field",
    "Bots": 0,
    "Players": 12,
    "Game Version": "Open Beta 5.84.767",
    "Variables": {
      "Mine Limit": 24,
      "bSteamRequired": false,
      "bPrivateMessageTeamOnly": false,
      "bPassworded": false,
      "bAllowPrivateMessaging": true,
      "bRanked": true,
      "Game Type": 1,
      "Player Limit": 64,
      "Vehicle Limit": 20,
      "bAutoBalanceTeams": false,
      "Team Mode": 6,
      "bSpawnCrates": true,
      "CrateRespawnAfterPickup": 35,
      "Time Limit": 50
    },
    "Port": 7778,
    "IP": "1.0.0.1"
  },
  "version": "Open Beta 5.84.767",
  "maxplayers": 64,
  "numplayers": 12,
  "players": [],
  "bots": [],
  "queryPort": 7777,
  "connect": "1.0.0.1:7778",
  "ping": 0
}

Query and receive master list

Setup:

GameDig.query({
  type: 'renegade10master'

Example response

{
  "name": "",
  "map": "",
  "password": false,
  "raw": {
    "servers": [
      {
        "name": "Server Name",
        "map": "CNC-Walls",
        "password": false,
        "raw": {
          "Name": "Server Name",
          "Current Map": "CNC-Walls",
          "Bots": 0,
          "Players": 0,
          "Game Version": "Open Beta 5.71.534",
          "Variables": {
            "Mine Limit": 30,
            "bSteamRequired": false,
            "bPrivateMessageTeamOnly": false,
            "bPassworded": false,
            "bAllowPrivateMessaging": true,
            "bRanked": true,
            "Game Type": 1,
            "Player Limit": 24,
            "Vehicle Limit": 11,
            "bAutoBalanceTeams": false,
            "Team Mode": 6,
            "bSpawnCrates": true,
            "CrateRespawnAfterPickup": 30,
            "Time Limit": 0
          },
          "Port": 7777,
          "IP": "1.0.0.1"
        },
        "version": "Open Beta 5.71.534",
        "maxplayers": 24,
        "numplayers": 0,
        "players": [],
        "bots": [],
        "queryPort": 0
      },
      {
        "name": "[PREFIX] Test Server",
        "map": "CNC-Uphill",
        "password": true,
        "raw": {
          "Name": "Test Server",
          "NamePrefix": "[PREFIX]",
          "Current Map": "CNC-Uphill",
          "Bots": 0,
          "Players": 0,
          "Game Version": "Open Beta 5.79.698",
          "Variables": {
            "Mine Limit": 24,
            "bSteamRequired": false,
            "bPrivateMessageTeamOnly": false,
            "bPassworded": true,
            "bAllowPrivateMessaging": true,
            "bRanked": true,
            "Game Type": 1,
            "Player Limit": 32,
            "Vehicle Limit": 10,
            "bAutoBalanceTeams": false,
            "Team Mode": 1,
            "bSpawnCrates": false,
            "CrateRespawnAfterPickup": 30,
            "Time Limit": 60
          },
          "Port": 7777,
          "IP": "1.0.0.2"
        },
        "version": "Open Beta 5.79.698",
        "maxplayers": 32,
        "numplayers": 0,
        "players": [],
        "bots": [],
        "queryPort": 0
      }
    ]
  },
  "version": "",
  "maxplayers": 0,
  "numplayers": 0,
  "players": [],
  "bots": [],
  "connect": "undefined:undefined",
  "ping": 0
}

[RattleSN4K3]

ID test fails but I don't think the ID should be forced as renegade10.

[CosminPerRam]

Unfortunately it needs to be that due to the current GID formatting rules.
We have gotten a bunch of negative feedback regarding it and will most likely be changed in the next major version (see #514 for more info), but for now we've got to hold onto the old system.

[xCausxn]

Even though 0 is falsy, I think we need to make sure password is either true or false

[RattleSN4K3]

Even though 0 is falsy, I think we need to make sure password is either true or false

Good catch. I actually wanted to enforce "0"/"1" and ended up having 0/1. The downside of working on severel protocols in parallel and and not checking the Results output.

[RattleSN4K3]

I changed processing password. Due to some commit message, I wanted to rebase the PR, but since it was a draft, I see no problem with it (I generel would not prefer doing so).

PR is ready for review.

[RattleSN4K3]

Updated the PR to reflect the proposed changes.

My points regarding the last changes still stand tho:

• Alternative protocols separated into single files could be messy... in case of protocols being dependent on existing ones (see master and server-query based on master protocol)

  With possible changes (for instance with feat: handle alternative query for some games #608) the file structured could be scattered.

• Inconsistency in protocol names and game id