Updates to address ETIMEDOUT error

Reference CHANGELOG.md for summary of changes.

CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to this project will be documented in this file. This project uses [Semantic Versioning](https://semver.org/).
 
+## v3.0.3
+* Updates to `tempestApi.ts` to minimize `ETIMEDOUT` error:
+  * Revert `axios.get` calls to include `headers:` declaration.
+  * Add `import https from 'https';`
+  * Add `axios.defaults.httpsAgent = new https.Agent({ keepAlive: true });`
+  * Add `axios.defaults.timeout = 10000;` and remove timeout from `axios.get` calls.
+
 ## v3.0.2
 * Update node-version: [16.x, 18.x, 20.x], remove 14.x which is no longer supported by homebridge.
 * Update `devDependencies` and `dependencies` to latest versions. Update/lock `axios` to version `1.5.0`.

package.json

@@ -2,7 +2,7 @@
   "private": false,
   "displayName": "Homebridge WeatherFlow Tempest",
   "name": "homebridge-weatherflow-tempest",
-  "version": "3.0.2",
+  "version": "3.0.3",
   "description": "Exposes WeatherFlow Tempest Station data as Temperature Sensors, Light Sensors, Humidity Sensors and Fan Sensors (for Wind Speed).",
   "license": "Apache-2.0",
   "repository": {

src/tempestApi.ts

@@ -1,5 +1,9 @@
 import { Logger } from 'homebridge';
 import axios, { AxiosResponse } from 'axios';
+import https from 'https';
+
+axios.defaults.timeout = 10000; // same as default interval
+axios.defaults.httpsAgent = new https.Agent({ keepAlive: true });
 
 export interface Observation {
   // temperature sensors
@@ -52,9 +56,11 @@ export class TempestApi {
   private async getStationObservation(): Promise<AxiosResponse | undefined> {
 
     try {
-      const url = `https://swd.weatherflow.com/swd/rest/observations/station/${this.station_id}?token=${this.token}`;
+      const url = `https://swd.weatherflow.com/swd/rest/observations/station/${this.station_id}`;
       const options = {
-        timeout: 1000,
+        headers: {
+          'Authorization': `Bearer ${this.token}`,
+        },
         validateStatus: (status: number) => status >= 200 && status < 300, // Default
       };
       const observation = await axios.get(url, options);
@@ -108,9 +114,11 @@ export class TempestApi {
 
   public async getTempestBatteryLevel(device_id): Promise<number> {
 
-    const url = `https://swd.weatherflow.com/swd/rest/observations/device/${device_id}?token=${this.token}`;
+    const url = `https://swd.weatherflow.com/swd/rest/observations/device/${device_id}`;
     const options = {
-      timeout: 1000,
+      headers: {
+        'Authorization': `Bearer ${this.token}`,
+      },
       validateStatus: (status: number) => status >= 200 && status < 300, // Default
     };
 
@@ -130,9 +138,11 @@ export class TempestApi {
 
   public async getTempestDeviceId(): Promise<number> {
 
-    const url = `https://swd.weatherflow.com/swd/rest/stations/${this.station_id}?token=${this.token}`;
+    const url = `https://swd.weatherflow.com/swd/rest/stations/${this.station_id}`;
     const options = {
-      timeout: 1000,
+      headers: {
+        'Authorization': `Bearer ${this.token}`,
+      },
       validateStatus: (status: number) => status >= 200 && status < 300, // Default
     };