node-red-contrib-uos-nats 1.2.0 → 1.2.2

package/nodes/datahub-input.js

@@ -214,7 +214,14 @@ module.exports = function (RED) {
 
             // If Wildcard (empty targetIds and isWildcard=true) -> Request ALL.
             // If Specific (targetIds has items) -> Request specific.
-            const snapshotMsg = await nc.request(subjects.readVariablesQuery(this.providerId), payloads.buildReadVariablesQuery(targetIds), { timeout: 2000 });
+            // Use serialRequest via Config Node to prevent concurrency issues on the connection
+            let snapshotMsg;
+            if (typeof connection.serialRequest === 'function') {
+              snapshotMsg = await connection.serialRequest(subjects.readVariablesQuery(this.providerId), payloads.buildReadVariablesQuery(targetIds), { timeout: 5000 });
+            } else {
+              // Fallback for older config nodes (should not happen if package updated correctly)
+              snapshotMsg = await nc.request(subjects.readVariablesQuery(this.providerId), payloads.buildReadVariablesQuery(targetIds), { timeout: 5000 });
+            }
 
             const bb = new flatbuffers.ByteBuffer(snapshotMsg.data);
             const snapshotObj = ReadVariablesQueryResponse.getRootAsReadVariablesQueryResponse(bb);
@@ -239,7 +246,9 @@ module.exports = function (RED) {
 
 
 
-        // Initial snapshot
+        // Initial snapshot with random jitter into prevent concurrency overload
+        // (If multiple nodes start simultaneously)
+        await new Promise(r => setTimeout(r, Math.floor(Math.random() * 500) + 100));
         await performSnapshot();
 
         // Setup polling if configured

package/nodes/uos-config.js

@@ -175,6 +175,31 @@ module.exports = function (RED) {
       return this.nc;
     };
 
+    this.requestQueue = Promise.resolve();
+
+    this.serialRequest = async (subject, payload, options = {}) => {
+      // Enqueue request to run sequentially
+      const result = this.requestQueue.then(async () => {
+        const nc = await this.ensureConnection();
+        return nc.request(subject, payload, options);
+      });
+
+      // Ensure queue does not block on failures (catch individually inside the chain logic)
+      // Actually, chaining the result directly makes the next one wait for result resolution.
+      // We want to update queue pointer to catch errors so next request still runs.
+      this.requestQueue = result.catch(() => { });
+
+      return result;
+    };
+
+    /**
+     * @deprecated Use serialRequest for concurrency safety
+     */
+    this.acquire = async () => {
+      this.users += 1;
+      return this.ensureConnection();
+    };
+
     this.getGrantedScopes = async () => {
       await this.getToken();
       return this.tokenInfo?.grantedScope || '';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-red-contrib-uos-nats",
-  "version": "1.2.0",
+  "version": "1.2.2",
   "description": "Node-RED nodes for Weidmüller u-OS Data Hub. Read, write, and provide variables via NATS protocol with OAuth2 authentication. Features: Variable Key resolution, custom icons, example flows, and provider definition caching.",
   "author": {
     "name": "IoTUeli",