fix: do not shift pings on 'publish' packets

Fixes #1863 #1861

src/lib/handlers/index.ts

@@ -26,7 +26,7 @@ const handle: PacketHandler = (client, packet, done) => {
 	client.pingResp = Date.now()
 
 	// do not shift on pingresp otherwise we would skip the pingreq sending
-	if (packet.cmd !== 'pingresp') {
+	if (!['pingresp', 'publish'].includes(packet.cmd)) {
 		client['_shiftPingInterval']()
 	}
 

test/abstract_client.ts

@@ -2009,6 +2009,14 @@ export default function abstractTest(server, config, ports) {
 			const spy = sinon.spy()
 			client['_checkPing'] = spy
 
+			server.once('client', (serverClient) => {
+				// send fake packet to client
+				serverClient.publish({
+					topic: 'foo',
+					payload: 'bar',
+				})
+			})
+
 			client.once('connect', () => {
 				client.publish('foo', 'bar')
 				clock.tick(intervalMs)