@vitormnm/node-red-simple-opcua 1.5.0 → 1.6.3

package/server/lib/opcua-server-runtime-child.js

@@ -36,6 +36,7 @@ class OpcUaServerProcess {
             throw new Error("Server already running");
         }
 
+        this.node.id = nodeId;
         this.node.name = settings.name;
         this.node.serverName = settings.serverName;
         this.node.server = null;
@@ -192,15 +193,18 @@ class OpcUaServerProcess {
             const target = msg && msg.opcuaServerIo ? msg.opcuaServerIo : {};
             const identifierType = this.resolveIdentifierType(target);
 
-            let result = {}
+            let result = {};
+            let readArrayResults = null;
 
             if (Array.isArray(payload)) {
                 if (!payload.length) {
                     throw new Error("msg.payload array does not contain any items");
                 }
 
+                readArrayResults = payload.map((item) => this.readPayloadItem(identifierType, item));
+
                 result = {
-                    payload: payload.map((item) => this.readPayloadItem(identifierType, item)),
+                    payload: readArrayResults,
                     identifiers: payload.map((item) => this.resolvePayloadItemIdentifier(item))
                 };
             } else if (payload && typeof payload === "object" && !Array.isArray(payload)) {
@@ -224,9 +228,18 @@ class OpcUaServerProcess {
                 };
             } else {
                 const identifier = this.resolveIdentifier(target);
+                let directValue = null;
+                let directError = null;
+                try {
+                    directValue = server.readValue(identifierType, identifier);
+                } catch (e) {
+                    directValue = null;
+                    directError = { identifier, message: e.message || String(e) };
+                }
                 result = {
-                    payload: server.readValue(identifierType, identifier),
-                    identifiers: [identifier]
+                    payload: directValue,
+                    identifiers: [identifier],
+                    directError
                 };
 
             }
@@ -260,6 +273,35 @@ class OpcUaServerProcess {
                 nodeId: nodeId
             });
 
+            // Emit a partialError for items that could not be read (array-of-objects mode only)
+            if (readArrayResults) {
+                const failed = readArrayResults
+                    .filter(item => item && item.status !== "Good")
+                    .map(item => ({ name: item.name, path: item.path, status: item.status }));
+
+                if (failed.length) {
+                    process.send({
+                        type: "partialError",
+                        error: "Some tags could not be read: " + failed.map(f => f.path).join(", "),
+                        failed: failed,
+                        originalMsg: msg,
+                        nodeId: nodeId
+                    });
+                }
+            }
+
+            // Emit a partialError for a direct single-tag read failure
+            if (result.directError) {
+                const { identifier, message } = result.directError;
+                process.send({
+                    type: "partialError",
+                    error: "Some tags could not be read: " + identifier,
+                    failed: [{ name: "", path: identifier, status: message }],
+                    originalMsg: msg,
+                    nodeId: nodeId
+                });
+            }
+
         } catch (error) {
 
             process.send({
@@ -334,6 +376,7 @@ class OpcUaServerProcess {
         try {
             let writtenPaths = null;
             let payload = msg ? msg.payload : undefined;
+            let directError = null;
 
             const target = msg && msg.opcuaServerIo ? msg.opcuaServerIo : {};
             const identifierType = this.resolveIdentifierType(target);
@@ -364,14 +407,18 @@ class OpcUaServerProcess {
             if (Buffer.isBuffer(payload) || payload instanceof Uint8Array) {
 
                 const identifier = this.resolveIdentifier(target);
-
-                this.node.writeValue(
-                    identifierType,
-                    identifier,
-                    Buffer.isBuffer(payload)
-                        ? payload
-                        : Buffer.from(payload)
-                );
+                try {
+                    this.node.writeValue(
+                        identifierType,
+                        identifier,
+                        Buffer.isBuffer(payload)
+                            ? payload
+                            : Buffer.from(payload)
+                    );
+                } catch (e) {
+                    directError = { identifier, message: e.message || String(e) };
+                    msg.payload = null;
+                }
 
                 writtenPaths = [identifier];
             }
@@ -383,14 +430,18 @@ class OpcUaServerProcess {
             ) {
 
                 const identifier = this.resolveIdentifier(target);
-
-                this.node.writeValue(
-                    identifierType,
-                    identifier,
-                    isByteString
-                        ? Buffer.from(payload)
-                        : payload
-                );
+                try {
+                    this.node.writeValue(
+                        identifierType,
+                        identifier,
+                        isByteString
+                            ? Buffer.from(payload)
+                            : payload
+                    );
+                } catch (e) {
+                    directError = { identifier, message: e.message || String(e) };
+                    msg.payload = null;
+                }
 
                 writtenPaths = [identifier];
             }
@@ -402,13 +453,13 @@ class OpcUaServerProcess {
                     throw new Error("msg.payload array does not contain any items");
                 }
 
-                payload.forEach(item => {
-                    this.writePayloadItem(identifierType, item);
-                });
-
-                writtenPaths = payload.map(item =>
-                    this.resolvePayloadItemIdentifier(item)
+                const writeArrayResults = payload.map(item =>
+                    this.writePayloadItem(identifierType, item)
                 );
+
+                msg.payload = writeArrayResults;
+
+                writtenPaths = writeArrayResults.map(item => item.path);
             }
 
             // Objeto { path: value }
@@ -428,11 +479,15 @@ class OpcUaServerProcess {
                 }
 
                 identifiers.forEach(identifier => {
-                    this.node.writeValue(
-                        identifierType,
-                        identifier,
-                        payload[identifier]
-                    );
+                    try {
+                        this.node.writeValue(
+                            identifierType,
+                            identifier,
+                            payload[identifier]
+                        );
+                    } catch (e) {
+                        // suppress per-item errors; unknown paths become undefined
+                    }
                 });
 
                 writtenPaths = identifiers;
@@ -442,12 +497,16 @@ class OpcUaServerProcess {
             else {
 
                 const identifier = this.resolveIdentifier(target);
-
-                this.node.writeValue(
-                    identifierType,
-                    identifier,
-                    payload
-                );
+                try {
+                    this.node.writeValue(
+                        identifierType,
+                        identifier,
+                        payload
+                    );
+                } catch (e) {
+                    directError = { identifier, message: e.message || String(e) };
+                    msg.payload = null;
+                }
 
                 writtenPaths = [identifier];
             }
@@ -484,6 +543,35 @@ class OpcUaServerProcess {
                 nodeId
             });
 
+            // Emit a partialError for items that could not be written (array-of-objects mode only)
+            if (Array.isArray(msg.payload) && msg.payload.length && msg.payload[0] && typeof msg.payload[0].status === "string") {
+                const failed = msg.payload
+                    .filter(item => item.status !== "Good")
+                    .map(item => ({ name: item.name, path: item.path, status: item.status }));
+
+                if (failed.length) {
+                    process.send({
+                        type: "partialError",
+                        error: "Some tags could not be written: " + failed.map(f => f.path).join(", "),
+                        failed: failed,
+                        originalMsg: msg,
+                        nodeId
+                    });
+                }
+            }
+
+            // Emit a partialError for a direct single-tag write failure
+            if (directError) {
+                const { identifier, message } = directError;
+                process.send({
+                    type: "partialError",
+                    error: "Some tags could not be written: " + identifier,
+                    failed: [{ name: "", path: identifier, status: message }],
+                    originalMsg: msg,
+                    nodeId
+                });
+            }
+
         } catch (error) {
 
             process.send({
@@ -540,20 +628,38 @@ class OpcUaServerProcess {
 
     readPayloadItem(identifierType, item) {
         const identifier = this.resolvePayloadItemIdentifier(item);
+        let value = null;
+        let status = "Good";
+        try {
+            value = this.node.readValue(identifierType, identifier);
+        } catch (e) {
+            value = null;
+            status = e.message || String(e);
+        }
         return {
             name: item.name,
             path: identifier,
-            value: this.node.readValue(identifierType, identifier)
+            value,
+            status
         };
     }
 
     writePayloadItem(identifierType, item) {
         const identifier = this.resolvePayloadItemIdentifier(item);
-        this.node.writeValue(identifierType, identifier, item.value);
+        let writtenValue = null;
+        let status = "Good";
+        try {
+            this.node.writeValue(identifierType, identifier, item.value);
+            writtenValue = item.value;
+        } catch (e) {
+            writtenValue = null;
+            status = e.message || String(e);
+        }
         return {
             name: item.name,
             path: identifier,
-            value: item.value
+            value: writtenValue,
+            status
         };
     }
 
@@ -859,7 +965,7 @@ process.on("uncaughtException", (err) => {
     process.send({
         type: "error",
         data: "Uncaught Exception: " + err.message,
-        nodeId: nodeId
+        nodeId: serverProcess.node.id
     });
 });
 
@@ -869,6 +975,6 @@ process.on("unhandledRejection", (reason) => {
     process.send({
         type: "error",
         data: "Unhandled Rejection: " + (reason?.message || reason),
-        nodeId: nodeId
+        nodeId: serverProcess.node.id
     });
 });

package/server/lib/opcua-server-runtime.js

@@ -67,7 +67,8 @@ class OpcUaServerRuntime {
             node: this.node,
             serverName: this.serverName,
             addressSpace: this.addressSpace,
-            allowAnonymous: this.allowAnonymous
+            allowAnonymous: this.allowAnonymous,
+            users: this.users
         });
 
 
@@ -238,10 +239,18 @@ class OpcUaServerRuntime {
         }
 
         const roles = [resolveNodeId("WellKnownRole_AuthenticatedUser")];
-        const customRole = this.resolveGroupRoleNodeId(user.group);
-        if (customRole) {
-            roles.push(customRole);
-        }
+        const groups = typeof user.group === "string"
+            ? user.group.split(",").map(g => g.trim()).filter(Boolean)
+            : Array.isArray(user.group)
+                ? user.group
+                : [];
+
+        groups.forEach((groupName) => {
+            const customRole = this.resolveGroupRoleNodeId(groupName);
+            if (customRole) {
+                roles.push(customRole);
+            }
+        });
         return roles;
     }
 

package/server/opcua-server-io.js

@@ -152,6 +152,15 @@ module.exports = function (RED) {
                 node.error(msg.error);
             }
 
+            if (msg.type === "partialError") {
+                // Route failed items to catch node without changing the node status
+                const catchMsg = Object.assign({}, msg.originalMsg || {}, {
+                    payload: msg.failed,
+                    error: msg.error
+                });
+                node.error(msg.error, catchMsg);
+            }
+
             if (msg.type === "sendMethod") {
 
                 node.status({

package/server/opcua-server.html

@@ -103,6 +103,7 @@
                 <a href="#" id="node-input-add-object" class="editor-button editor-button-small"><i class="fa fa-plus"></i> Add object</a>
                 <a href="#" id="node-input-add-folder" class="editor-button editor-button-small"><i class="fa fa-plus"></i> Add folder</a>
                 <a href="#" id="node-input-add-object-type" class="editor-button editor-button-small"><i class="fa fa-plus"></i> Add object type</a>
+                <a href="#" id="node-input-add-enumeration" class="editor-button editor-button-small"><i class="fa fa-plus"></i> Add enumeration</a>
                 <a href="#" id="node-input-add-namespace" class="editor-button editor-button-small"><i class="fa fa-plus"></i> Add namespace</a>
                 <a href="#" id="node-input-expand-all" class="editor-button editor-button-small"><i class="fa fa-angle-double-down"></i> Expand all</a>
                 <a href="#" id="node-input-collapse-all" class="editor-button editor-button-small"><i class="fa fa-angle-double-up"></i> Collapse all</a>
@@ -128,6 +129,8 @@
                     <a href="#" data-action="add-object"><i class="fa fa-plus"></i> Add Object</a>
                     <a href="#" data-action="add-variable"><i class="fa fa-plus"></i> Add Variable</a>
                     <a href="#" data-action="add-objecttype"><i class="fa fa-plus"></i> Add ObjectType</a>
+                    <a href="#" data-action="add-enumeration"><i class="fa fa-plus"></i> Add Enumeration</a>
+                    <a href="#" data-action="add-enum-variable"><i class="fa fa-plus"></i> Add Enum Variable</a>
                     <a href="#" data-action="add-alarm"><i class="fa fa-plus"></i> Add Alarm</a>
                     <a href="#" data-action="add-method"><i class="fa fa-plus"></i> Add Method</a>
                     <a href="#" data-action="edit"><i class="fa fa-pencil"></i> Edit</a>
@@ -150,7 +153,7 @@
     <h3>Inputs</h3>
     <p><code>msg.payload</code>: JSON object describing the OPC UA tree. When provided, the node validates the structure and rebuilds the dynamic namespace.</p>
     <h3>Details</h3>
-    <p>Supported variable types: <code>Int32</code>, <code>Float</code>, <code>Boolean</code>, <code>String</code>.</p>
+    <p>Supported variable types: <code>Int16</code>, <code>Int32</code>, <code>Int64</code>, <code>Float</code>, <code>Boolean</code>, <code>String</code>.</p>
     <p>Supported access modes: <code>readonly</code> and <code>readwrite</code>.</p>
     <p>Authentication can be configured in the editor with multiple local users and groups stored in Node-RED credentials, and anonymous login can be disabled.</p>
 </script>

package/server/opcua-server.js

@@ -21,6 +21,7 @@ module.exports = function (RED) {
         node.serverName = settings.serverName;
         node.server = null;
         node.namespace = null;
+        node.isClosing = false;
 
         node.status({ fill: "yellow", shape: "ring", text: "initializing OPC UA server" });
 
@@ -44,12 +45,35 @@ module.exports = function (RED) {
 
         registry.registerServerNames(node.serverName, node.serverName);
 
-        child.on("exit", () => {
+        let crashHandled = false;
+        function handleUnexpectedExit(code, signal) {
+            if (node.isClosing || crashHandled) {
+                return;
+            }
+            crashHandled = true;
+
+            const errorDetails = `OPC UA server child process exited unexpectedly with code ${code} and signal ${signal}`;
+            node.status({ fill: "red", shape: "dot", text: "Child process crashed" });
+
+            const catchMsg = {
+                topic: node.serverName,
+                payload: {
+                    status: "error",
+                    error: errorDetails
+                }
+            };
+            node.send(catchMsg);
+            node.error(errorDetails, catchMsg);
+        }
+
+        child.on("exit", (code, signal) => {
             registry.unregisterChild(node.serverName, child);
+            handleUnexpectedExit(code, signal);
         });
 
-        child.on("close", () => {
+        child.on("close", (code, signal) => {
             registry.unregisterChild(node.serverName, child);
+            handleUnexpectedExit(code, signal);
         });
 
 
@@ -112,6 +136,7 @@ module.exports = function (RED) {
 
         node.on("close", async function (removed, done) {
             try {
+                node.isClosing = true;
                 registry.unregisterChild(node.serverName, child);
                 registry.unregisterServerNames(node.serverName);
                 child.kill();

package/server/view/opcua-server.css

@@ -457,6 +457,10 @@ body.opcua-tree-modal-open {
     width: calc(100% - 100px);
 }
 
+.opcua-auth-card .red-ui-typedInput-container {
+    width: calc(100% - 100px) !important;
+}
+
 @media (max-width: 900px) {
     .opcua-tree-modal {
         padding: 10px;