use name to identify servers not ip

.gitignore

@@ -230,4 +230,3 @@ data/db.sqlite-wal
 data/database-backups
 
 data/config.json
-data/servers.json

data/servers.json

@@ -2,73 +2,61 @@
   {
     "name": "WildPrison",
     "ip": "wildprison.net",
-    "type": "PC",
+    "type": "PC"
-    "id": 0
   },
   {
     "name": "Hypixel",
     "ip": "mc.hypixel.net",
-    "type": "PC",
+    "type": "PC"
-    "id": 1
   },
   {
-    "name": "CubeCraft",
+    "name": "Cubecraft",
     "ip": "play.cubecraft.net",
-    "type": "PC",
+    "type": "PC"
-    "id": 2
   },
   {
     "name": "Mineplex",
     "ip": "mineplex.com",
-    "type": "PC",
+    "type": "PC"
-    "id": 3
   },
   {
     "name": "2b2t",
     "ip": "2b2t.org",
-    "type": "PC",
+    "type": "PC"
-    "id": 4
   },
   {
     "name": "AkumaMC",
     "ip": "akumamc.net",
-    "type": "PC",
+    "type": "PC"
-    "id": 5
   },
   {
     "name": "Wynncraft",
     "ip": "play.wynncraft.com",
-    "type": "PC",
+    "type": "PC"
-    "id": 6
   },
   {
     "name": "Minehut",
     "ip": "minehut.com",
-    "type": "PC",
+    "type": "PC"
-    "id": 7
   },
   {
     "name": "Grand Theft Minecraft",
     "ip": "gtm.network",
-    "type": "PC",
+    "type": "PC"
-    "id": 8
   },
   {
     "name": "HiveMC",
     "ip": "geo.hivebedrock.network",
-    "type": "PE",
+    "type": "PE"
-    "id": 9
   },
   {
     "name": "Purple Prison",
     "ip": "MCSL.PURPLE.WTF",
-    "type": "PC",
+    "type": "PC"
-    "id": 10
   },
   {
     "name": "MinecraftOnline",
     "ip": "minecraftonline.com",
-    "type": "PC",
+    "type": "PC"
-    "id": 11
   }
 ]

src/server/server.ts

@@ -20,7 +20,6 @@ export enum ServerStatus {
 }
 
 type ServerOptions = {
-  id: number;
   name: string;
   ip: string;
   port?: number;
@@ -33,11 +32,6 @@ type DnsInfo = {
 };
 
 export default class Server {
-  /**
-   * The ID of the server.
-   */
-  private id: number;
-
   /**
    * The name of the server.
    */
@@ -71,8 +65,7 @@ export default class Server {
     hasResolved: false,
   };
 
-  constructor({ id, name, ip, port, type }: ServerOptions) {
+  constructor({ name, ip, port, type }: ServerOptions) {
-    this.id = id;
     this.name = name;
     this.ip = ip;
     this.port = port;
@@ -106,7 +99,7 @@ export default class Server {
       try {
         influx.writePoint(
           new Point("playerCount")
-            .tag("id", this.getID().toString())
+            .tag("name", this.getName())
             .tag("ip", this.getIP().toLowerCase())
             .intField("playerCount", response.playerCount)
             .timestamp(response.timestamp)
@@ -163,7 +156,6 @@ export default class Server {
 
         this.favicon = res.favicon; // Set the favicon
         resolve({
-          id: this.getID(),
           timestamp: Date.now(),
           ip: ip,
           playerCount: res.players.online,
@@ -189,7 +181,6 @@ export default class Server {
           }
 
           resolve({
-            id: this.getID(),
             timestamp: Date.now(),
             ip: this.getIP(),
             playerCount: res.currentPlayers,
@@ -199,15 +190,6 @@ export default class Server {
     });
   }
 
-  /**
-   * Returns the ID of the server.
-   *
-   * @returns the ID
-   */
-  public getID(): number {
-    return this.id;
-  }
-
   /**
    * Returns the name of the server.
    *

src/server/serverManager.ts

@@ -15,7 +15,6 @@ export default class ServerManager {
     logger.info("Loading servers");
     for (const configServer of Servers) {
       const server = new Server({
-        id: configServer.id,
         ip: configServer.ip,
         name: configServer.name,
         type: configServer.type as ServerType,

src/types/ping.ts

@@ -1,5 +1,4 @@
 export type Ping = {
-  id: number;
   timestamp: number;
   ip: string;
   playerCount: number;