Bug in ping() method

[PetrJajtner]

Expected Behavior

In pool/ping() is included 'path' from constructor config attribute together with '/ping'
Fix: https://node-influx.github.io/file/src/pool.js.html - line 188 ->
path: `${url.path}${path}`.replace(/\\/+/, '/')

Actual Behavior

The path parameter from config settings is missing in pool/ping() method
https://node-influx.github.io/file/src/pool.js.html - line 185, 188 resp.

Steps/Code to Reproduce the Problem

1. Set path in constructor: client = new InfluxDB({path: '/influx'});
2. Use ping method: client.ping(Infinity); // -> online: false, because not [server]/influx/ping but [server]/ping

import { InfluxDB } from 'influx';
const client = new InfluxDB({path: '/influx'});
await client.ping(Infinity); // online: false due to incorrect URL path

Specifications

• Version: 5.5.1
• Platform:
• Subsystem:

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[bencevans]

Thanks for reporting. A Pull Request fixing this would be very welcome 😸

[rnd-debug]

Btw, providing Infinity as param will fail with: RangeError [ERR_OUT_OF_RANGE]: The value of "timeout" is out of range. It must be a non-negative finite number. Received Infinity