Fix ping implement

Author: SeaLoong

src/index.d.ts

@@ -256,7 +256,7 @@ declare module 'minecraft-protocol' {
 	}
 
 	export interface OldPingResult {
-		maxPlayers: number,
+		maxPlayers: number
 		motd: string
 		playerCount: number
 		prefix: string
@@ -282,9 +282,11 @@ declare module 'minecraft-protocol' {
 			protocol: number
 		}
 		favicon?: string
-		latency: number
+		latency?: number
 	}
 
+	export type PingResult = OldPingResult | NewPingResult
+
 	export interface RealmsOptions {
 		realmId?: string
 		pickRealm?: (realms: Realm[]) => Realm
@@ -301,5 +303,5 @@ declare module 'minecraft-protocol' {
 	export function createSerializer({ state, isServer, version, customPackets }: SerializerOptions): any
 	export function createDeserializer({ state, isServer, version, customPackets }: SerializerOptions): any
 
-	export function ping(options: PingOptions, callback?: (error: Error, result: OldPingResult | NewPingResult) => void): Promise<OldPingResult | NewPingResult>
+	export function ping(options: PingOptions, callback?: (error: Error, result: PingResult) => void): Promise<PingResult>
 }

src/ping.js

@@ -40,22 +40,28 @@ function ping (options) {
     client.once('server_info', function (packet) {
       const data = JSON.parse(packet.response)
       const start = Date.now()
-      const maxTime = setTimeout(() => {
+      const maxTimer = setTimeout(() => {
         clearTimeout(closeTimer)
         client.end()
         resolve(data)
       }, options.noPongTimeout)
+      const time = BigInt(Date.now())
       client.once('ping', function (packet) {
         data.latency = Date.now() - start
-        clearTimeout(maxTime)
-        clearTimeout(closeTimer)
-        client.end()
-        resolve(data)
+        if (BigInt(packet.time) === time) {
+          // pong payload should be the same as ping payload
+          clearTimeout(maxTimer)
+          clearTimeout(closeTimer)
+          client.end()
+          resolve(data)
+        }
       })
-      client.write('ping', { time: [0, 0] })
+      client.write('ping', { time })
     })
     client.on('state', function (newState) {
-      if (newState === states.STATUS) { client.write('ping_start', {}) }
+      if (newState === states.STATUS) {
+        client.write('ping_start', {})
+      }
     })
     // TODO: refactor with src/client/setProtocol.js
     client.on('connect', function () {