@node-red/nodes 4.1.0-beta.1 → 4.1.0

package/core/common/20-inject.html

@@ -248,8 +248,9 @@
                                 errors.push(RED._("node-red:inject.errors.invalid-json", { prop: 'msg.'+v[i].p, error: e.message }))
                             }
                         } else if (v[i].vt === "num"){
-                            if (!/^[+-]?[0-9]*\.?[0-9]*([eE][-+]?[0-9]+)?$/.test(v[i].v)) {
-                                errors.push(RED._("node-red:inject.errors.invalid-prop", { prop: 'msg.'+v[i].p, error: v[i].v }))
+                            const numValidation = RED.utils.validateTypedProperty(v[i].v, 'num', { prop: 'msg.'+v[i].p });
+                            if (numValidation !== true) {
+                                errors.push(numValidation || RED._("node-red:inject.errors.invalid-prop", { prop: 'msg.'+v[i].p, error: v[i].v }))
                             }
                         }
                     }

package/core/common/lib/debug/debug-utils.js

@@ -536,10 +536,22 @@ RED.debug = (function() {
                 e.stopPropagation();
                 showMessageMenu(filterMessage,debugMessage,sourceNode&&sourceNode.id);
             });
-            $('<span class="red-ui-debug-msg-topic">'+
-                (o.topic?topic+' : ':'')+
-                (o.property?'msg.'+property:'msg')+" : "+format+
-                '</span>').appendTo(metaRow);
+            const msgInfos = `${o.topic ? " : " : ""}msg${o.property ? "." + property : ""} : ${format}`
+            const topicTruncated = RED.utils.truncateString((o.topic ? topic : ""), 120 - msgInfos.length);
+            const topicElem = $('<span class="red-ui-debug-msg-topic">' + topicTruncated + msgInfos + '</span>').appendTo(metaRow);
+            if (topic !== topicTruncated) {
+                RED.popover.create({
+                    content: topic,
+                    delay: { show: 750, hide: 50 },
+                    direction: "bottom",
+                    interactive: true,
+                    maxWidth: 300,
+                    target: topicElem,
+                    trigger: "hover",
+                    tooltip: true,
+                    size: "small"
+                });
+            }
         }
 
         var atBottom = (sbc.scrollHeight-messageList.height()-sbc.scrollTop) < 5;

package/core/function/90-exec.js

@@ -91,7 +91,13 @@ module.exports = function(RED) {
                     const opts = isWindows ? { ...node.spawnOpt, shell: true } : node.spawnOpt
                     /* istanbul ignore else  */
                     node.debug(cmd+" ["+arg+"]");
-                    child = spawn(cmd,arg,opts);
+                    if (opts.shell) {
+                        // When called with shell: true, passing in separate args is deprecated in Node 24
+                        // so we need to join the command and args into a single string.
+                        child = spawn([cmd].concat(arg).join(" "), opts);
+                    } else {
+                        child = spawn(cmd,arg,opts);
+                    }
                     node.status({fill:"blue",shape:"dot",text:"pid:"+child.pid});
                     var unknownCommand = (child.pid === undefined);
                     if (node.timer !== 0) {

package/core/network/10-mqtt.html

@@ -477,6 +477,16 @@
         }
         return RED._("node-red:mqtt.errors.invalid-topic");
     }
+    function setupTopicField(node, input) {
+        input.autoComplete({
+            minLength: 0,
+            completionPluginType: 'node-red-mqtt-topic-autocomplete-source',
+            context: {
+                node
+            }
+        })
+    }
+
     RED.nodes.registerType('mqtt-broker',{
         category: 'config',
         defaults: {
@@ -852,6 +862,7 @@
         },
         oneditprepare: function() {
             const node = this;
+            setupTopicField(this, $("#node-input-topic"));
             const isV5Broker = function() {
                 var confNode = RED.nodes.node($("#node-input-broker").val());
                 return confNode && confNode.protocolVersion === "5";
@@ -930,6 +941,7 @@
         },
         oneditprepare: function() {
             var that = this;
+            setupTopicField(this, $("#node-input-topic"));
 
             function showHideDynamicFields() {
                 var confNode = RED.nodes.node($("#node-input-broker").val());

package/core/network/21-httpin.html

@@ -25,11 +25,22 @@
         <option value="patch">PATCH</option>
         </select>
     </div>
-    <div class="form-row form-row-http-in-upload hide">
+
+    <div id="form-reqBody-http-in-controller" class="form-row hide" style="display: flex;">
         <label>&nbsp;</label>
-        <input type="checkbox" id="node-input-upload" style="display: inline-block; width: auto; vertical-align: top;">
-        <label for="node-input-upload" style="width: 70%;" data-i18n="httpin.label.upload"></label>
+        <div style="display: flex; margin-left: 5px; flex-direction: column-reverse; gap: 10px; justify-content: center;">
+            <div id="form-row-http-in-upload" class="hide" style="display: flex; gap: 10px">
+                <input type="checkbox" id="node-input-upload" style="margin: 0px;">
+                <label for="node-input-upload" style="text-wrap: nowrap; margin-bottom: 0;" data-i18n="httpin.label.upload"></label>
+            </div>
+            <div id="form-row-http-in-parsing" class="hide" style="display: flex; gap: 10px">
+                <input type="checkbox" id="node-input-skipBodyParsing" style="margin: 0px;">
+                <label for="node-input-skipBodyParsing" style="text-wrap: nowrap; margin-bottom: 0;" data-i18n="httpin.label.parsing"></label>
+            </div>
+        </div>
     </div>
+    
+
     <div class="form-row">
         <label for="node-input-url"><i class="fa fa-globe"></i> <span data-i18n="httpin.label.url"></span></label>
         <input id="node-input-url" type="text" placeholder="/url">
@@ -74,6 +85,7 @@
                   label:RED._("node-red:httpin.label.url")},
             method: {value:"get",required:true},
             upload: {value:false},
+            skipBodyParsing: {value:false},
             swaggerDoc: {type:"swagger-doc", required:false}
         },
         inputs:0,
@@ -115,16 +127,22 @@
                 $('.row-swagger-doc').hide();
             }
             $("#node-input-method").on("change", function() {
-                if ($(this).val() === "post") {
-                    $(".form-row-http-in-upload").show();
+                var method = $(this).val();
+                if(["post", "put", "patch","delete"].includes(method)){
+                    $("#form-reqBody-http-in-controller").show();
+                    $("#form-row-http-in-parsing").show();
+                    if (method === "post") {
+                        $("#form-row-http-in-upload").show();
+                    } else {
+                        $("#form-row-http-in-upload").hide();
+                    }
                 } else {
-                    $(".form-row-http-in-upload").hide();
+                    $("#form-row-http-in-parsing").hide();
+                    $("#form-row-http-in-upload").hide();
+                    $("#form-reqBody-http-in-controller").hide();
                 }
             }).change();
-
-
         }
-
     });
     var headerTypes = [
         {value:"content-type",label:"Content-Type",hasValue: false},

package/core/network/21-httpin.js

@@ -16,6 +16,7 @@
 
 module.exports = function(RED) {
     "use strict";
+    var rootApp;
     var bodyParser = require("body-parser");
     var multer = require("multer");
     var cookieParser = require("cookie-parser");
@@ -26,10 +27,11 @@ module.exports = function(RED) {
     var mediaTyper = require('media-typer');
     var isUtf8 = require('is-utf8');
     var hashSum = require("hash-sum");
+    var rawDataRoutes = new Set();
 
     function rawBodyParser(req, res, next) {
-        if (req.skipRawBodyParser) { next(); } // don't parse this if told to skip
-        if (req._body) { return next(); }
+        if (req.skipRawBodyParser || req._body || req.readableEnded) return next();
+        
         req.body = "";
         req._body = true;
 
@@ -71,7 +73,65 @@ module.exports = function(RED) {
         });
     }
 
-    var corsSetup = false;
+    /**
+     * This method retrieves the root app by traversing the parent hierarchy.
+     * @param {import('express').Application} app 
+     * @returns {import('express').Application}
+     */
+    function getRootApp(app) {
+        if (typeof app.parent === 'undefined') {
+            return app;
+        }
+        return getRootApp(app.parent);
+    }
+
+    /**
+     * It provide the unique route key
+     * @param {{method: string, url: string}} obj 
+     * @returns 
+     */
+    function getRouteKey(obj) {
+        var method = obj.method.toUpperCase();
+        // Normalize the URL by replacing double slashes with a single slash and removing the trailing slash
+        var url = obj.url.replace(/\/{2,}/g, '/').replace(/\/$/, '');
+        return `${method}:${url}`;
+    }
+
+    /**
+     * This middleware is for capture raw body
+     * @param {import('express').Request} req 
+     * @param {import('express').Response} _res 
+     * @param {import('express').NextFunction} next 
+     * @returns 
+     */
+    function rawBodyCapture(req, _res, next) {
+        var routeKey = getRouteKey({ method: req.method, url: req._parsedUrl.pathname });
+        // Check if routeKey exist in rawDataRoutes
+        if (rawDataRoutes.has(routeKey)) {
+            // Convert the request stream to buffer
+            getBody(req, {
+                length: req.headers['content-length'],
+            }, function (err, buf) {
+                if (err) {
+                    return next(err);
+                }
+                req.body = buf; 
+                // Skip the body parsing
+                req.skipRawBodyParser = true;
+                req._body = true;          
+                next();
+            })
+        } else {
+            next();
+        }
+    }
+
+    if(typeof RED.httpNode === 'function' && (rootApp = getRootApp(RED.httpNode))) {
+        // Add middleware to the stack
+        rootApp.use(rawBodyCapture);
+        // Move the middleware to top of the stack 
+        rootApp._router.stack.unshift(rootApp._router.stack.pop());
+    }
 
     function createRequestWrapper(node,req) {
         // This misses a bunch of properties (eg headers). Before we use this function
@@ -125,6 +185,7 @@ module.exports = function(RED) {
 
         return wrapper;
     }
+
     function createResponseWrapper(node,res) {
         var wrapper = {
             _res: res
@@ -188,9 +249,16 @@ module.exports = function(RED) {
             }
             this.method = n.method;
             this.upload = n.upload;
+            this.skipBodyParsing = n.skipBodyParsing;
             this.swaggerDoc = n.swaggerDoc;
 
             var node = this;
+            var routeKey = getRouteKey({method: this.method, url: RED.httpNode.path() + this.url});
+
+            // If the user enables raw body, add it to the raw data routes.
+            if(this.skipBodyParsing) {
+                rawDataRoutes.add(routeKey);
+            }
 
             this.errorHandler = function(err,req,res,next) {
                 node.warn(err);
@@ -227,7 +295,9 @@ module.exports = function(RED) {
             }
 
             var maxApiRequestSize = RED.settings.apiMaxLength || '5mb';
+
             var jsonParser = bodyParser.json({limit:maxApiRequestSize});
+
             var urlencParser = bodyParser.urlencoded({limit:maxApiRequestSize,extended:true});
 
             var metricsHandler = function(req,res,next) { next(); }
@@ -253,6 +323,7 @@ module.exports = function(RED) {
             if (this.upload) {
                 var mp = multer({ storage: multer.memoryStorage() }).any();
                 multipartParser = function(req,res,next) {
+                    if(req.readableEnded || req._body) return next();
                     mp(req,res,function(err) {
                         req._body = true;
                         next(err);
@@ -261,18 +332,21 @@ module.exports = function(RED) {
             }
 
             if (this.method == "get") {
-                RED.httpNode.get(this.url,cookieParser(),httpMiddleware,corsHandler,metricsHandler,this.callback,this.errorHandler);
+                RED.httpNode.get(this.url, cookieParser(), httpMiddleware, corsHandler, metricsHandler, this.callback, this.errorHandler);
             } else if (this.method == "post") {
-                RED.httpNode.post(this.url,cookieParser(),httpMiddleware,corsHandler,metricsHandler,jsonParser,urlencParser,multipartParser,rawBodyParser,this.callback,this.errorHandler);
+                RED.httpNode.post(this.url,cookieParser(), httpMiddleware, corsHandler, metricsHandler, jsonParser, urlencParser, multipartParser, rawBodyParser, this.callback, this.errorHandler);
             } else if (this.method == "put") {
-                RED.httpNode.put(this.url,cookieParser(),httpMiddleware,corsHandler,metricsHandler,jsonParser,urlencParser,rawBodyParser,this.callback,this.errorHandler);
+                RED.httpNode.put(this.url, cookieParser(), httpMiddleware, corsHandler, metricsHandler, jsonParser, urlencParser, rawBodyParser, this.callback, this.errorHandler);
             } else if (this.method == "patch") {
-                RED.httpNode.patch(this.url,cookieParser(),httpMiddleware,corsHandler,metricsHandler,jsonParser,urlencParser,rawBodyParser,this.callback,this.errorHandler);
+                RED.httpNode.patch(this.url, cookieParser(), httpMiddleware, corsHandler, metricsHandler, jsonParser, urlencParser, rawBodyParser, this.callback, this.errorHandler);
             } else if (this.method == "delete") {
-                RED.httpNode.delete(this.url,cookieParser(),httpMiddleware,corsHandler,metricsHandler,jsonParser,urlencParser,rawBodyParser,this.callback,this.errorHandler);
+                RED.httpNode.delete(this.url, cookieParser(), httpMiddleware, corsHandler, metricsHandler, jsonParser, urlencParser, rawBodyParser, this.callback, this.errorHandler);
             }
-
+            
+            
             this.on("close",function() {
+                // Remove it from the raw data routes if the node is closed
+                rawDataRoutes.delete(routeKey);
                 var node = this;
                 RED.httpNode._router.stack.forEach(function(route,i,routes) {
                     if (route.route && route.route.path === node.url && route.route.methods[node.method]) {
@@ -284,9 +358,9 @@ module.exports = function(RED) {
             this.warn(RED._("httpin.errors.not-created"));
         }
     }
+    
     RED.nodes.registerType("http in",HTTPIn);
 
-
     function HTTPOut(n) {
         RED.nodes.createNode(this,n);
         var node = this;
@@ -361,5 +435,6 @@ module.exports = function(RED) {
             done();
         });
     }
+
     RED.nodes.registerType("http response",HTTPOut);
 }

package/core/network/21-httprequest.js

@@ -599,7 +599,18 @@ in your Node-RED user directory (${RED.settings.userDir}).
                 }
             } else {
                 if (msg.hasOwnProperty('rejectUnauthorized')) {
-                    opts.https = { rejectUnauthorized: msg.rejectUnauthorized };
+                    if (typeof msg.rejectUnauthorized === 'boolean') {
+                        opts.https = { rejectUnauthorized: msg.rejectUnauthorized }
+                    } else if (typeof msg.rejectUnauthorized === 'string') {
+                        if (msg.rejectUnauthorized.toLowerCase() === 'true' || msg.rejectUnauthorized.toLowerCase() === 'false') {
+                            opts.https = { rejectUnauthorized: (msg.rejectUnauthorized.toLowerCase() === 'true') }
+                        } else {
+                            node.warn(RED._("httpin.errors.rejectunauthorized-invalid"))
+                        }
+                    } else {
+                        node.warn(RED._("httpin.errors.rejectunauthorized-invalid"))
+                    }
+                    
                 }
             }
 

package/locales/en-US/function/rbe.html

@@ -1,5 +1,5 @@
 <script type="text/html" data-help-name="rbe">
-    <p>Report by Exception (RBE) node - only passes on data if the payload has changed.
+    <p>filter node - only passes on data if the payload has changed.
        It can also block unless, or ignore if the value changes by a specified amount (Dead- and Narrowband mode).</p>
     <h3>Inputs</h3>
     <dl class="message-properties">
@@ -12,7 +12,7 @@
         </dt>
         <dd>if specified the function will work on a per topic basis. This property can be set by configuration.</dd>
         <dt class="optional">reset<span class="property-type">any</span></dt>
-        <dd>if set clears the stored value for the specified msg.topic, or
+        <dd>if set clears the stored value for the specified <code>msg.topic</code>, or
             all topics if msg.topic is not specified.</dd>
     </dl>
     <h3>Outputs</h3>

package/locales/en-US/messages.json

@@ -515,7 +515,8 @@
             "url": "URL",
             "doc": "Docs",
             "return": "Return",
-            "upload": "Accept file uploads?",
+            "upload": "Accept file uploads",
+            "parsing": "Do not parse request body",
             "status": "Status code",
             "headers": "Headers",
             "other": "other",
@@ -563,7 +564,8 @@
             "timeout-isnan": "Timeout value is not a valid number, ignoring",
             "timeout-isnegative": "Timeout value is negative, ignoring",
             "invalid-payload": "Invalid payload",
-            "invalid-url": "Invalid url"
+            "invalid-url": "Invalid url",
+            "rejectunauthorized-invalid": "msg.rejectUnauthorized should be a boolean"
         },
         "status": {
             "requesting": "requesting"

package/locales/en-US/network/21-httpin.html

@@ -50,6 +50,7 @@
     <p>If the content type of the request can be determined, the body will be parsed to
        any appropriate type. For example, <code>application/json</code> will be parsed to
        its JavaScript object representation.</p>
+    <p>The node can be configured to not parse the body, in which case it will be provided as a Buffer object.</p>
     <p><b>Note:</b> this node does not send any response to the request. The flow
        must include an HTTP Response node to complete the request.</p>
 </script>

package/locales/ja/function/rbe.html

@@ -1,7 +1,7 @@
 <!-- Source revision: https://github.com/node-red/node-red-nodes/commit/467907776088422882076f46d85e25601449564d -->
 
 <script type="text/html" data-help-name="rbe">
-    <p>Report by Exception(例外データの報告)ノード - ペイロードの値が変化した場合だけデータを送信。</p>
+    <p>filterノード - ペイロードの値が変化した場合だけデータを送信。</p>
     <p>値が指定した量変化するまでブロックすることもできます- 不感帯(deadband)モード。</p>
     <h3>入力</h3>    <dl class="message-properties">
         <dt>payload
@@ -12,7 +12,7 @@
         </dt>
         <dd>指定すると、トピックごとに動作します。</dd>
         <dt class="optional">reset<span class="property-type">任意</span></dt>
-        <dd>値を設定すると、保存した値をクリアします。msg.topicを指定した場合は対応する値、指定しなければ全てのトピックが対象となります。</dd>
+        <dd>値を設定すると、保存した値をクリアします。<code>msg.topic</code>を指定した場合は対応する値、指定しなければ全てのトピックが対象となります。</dd>
     </dl>
     <h3>出力</h3>
     <dl class="message-properties">

package/locales/ja/messages.json

@@ -406,6 +406,7 @@
         "label": {
             "unknown": "unknown"
         },
+        "manageModules": "モジュールを管理",
         "tip": "<p>現在のNode-RED環境では、本ノードの型が不明です。</p><p><i>現在の状態で本ノードをデプロイすると設定は保存されますが、不明なノードがインストールされるまでフローは実行されません。</i></p><p>詳細はノードの「情報」を参照してください。</p>"
     },
     "mqtt": {
@@ -515,6 +516,7 @@
             "doc": "ドキュメント",
             "return": "出力形式",
             "upload": "ファイルのアップロード",
+            "parsing": "リクエストボティをパースしない",
             "status": "ステータスコード",
             "headers": "ヘッダ",
             "other": "その他",
@@ -562,7 +564,8 @@
             "timeout-isnan": "タイムアウト値が数値ではないため無視します",
             "timeout-isnegative": "タイムアウト値が負数のため無視します",
             "invalid-payload": "不正なペイロード",
-            "invalid-url": "URLが不正"
+            "invalid-url": "URLが不正",
+            "rejectunauthorized-invalid": "msg.rejectUnauthorizedは真偽値である必要があります"
         },
         "status": {
             "requesting": "要求中"

package/locales/ja/network/10-mqtt.html

@@ -47,7 +47,7 @@
     <p>これらは、動的購読が設定されている場合のみ適用されます。</p>
     <dl class="message-properties">
        <dt>action <span class="property-type">文字列</span></dt>
-       <dd>本ノードが行う動作の名前。利用可能な動作は<code>"connect"</code>、<code>"disconnect"</code>、<code>"subscribe"</code>、<code>"unsubscribe"</code>です。</dd>
+       <dd>本ノードが行う動作の名前。利用可能な動作は<code>"connect"</code>、<code>"disconnect"</code>、<code>"getSubscriptions"</code>、<code>"subscribe"</code>、<code>"unsubscribe"</code>です。</dd>
        <dt class="optional">topic <span class="property-type">文字列|オブジェクト|配列</span></dt>
        <dd><code>"subscribe"</code>と<code>"unsubscribe"</code>の動作に対して、本プロパティはトピックを提供します。次のいずれかを設定できます:<ul>
            <li>トピックフィルターを含む文字列</li>
@@ -105,7 +105,7 @@
     <h3>入力</h3>
     <dl class="message-properties">
        <dt>action <span class="property-type">文字列</span></dt>
-       <dd>本ノードが行う動作の名前。利用可能な動作は<code>"connect"</code>、<code>"disconnect"</code>、<code>"subscribe"</code>、<code>"unsubscribe"</code>です。</dd>
+       <dd>本ノードが行う動作の名前。利用可能な動作は<code>"connect"</code>と<code>"disconnect"</code>です。</dd>
        <dt class="optional">broker <span class="property-type">broker</span> </dt>
        <dd><code>"connect"</code>の動作に対して、本プロパティは次の様な個々のブローカ設定を上書きします: <ul>
            <li><code>broker</code></li>

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@node-red/nodes",
-    "version": "4.1.0-beta.1",
+    "version": "4.1.0",
     "license": "Apache-2.0",
     "repository": {
         "type": "git",
@@ -26,7 +26,7 @@
         "cors": "2.8.5",
         "cronosjs": "1.7.1",
         "denque": "2.1.0",
-        "form-data": "4.0.2",
+        "form-data": "4.0.4",
         "fs-extra": "11.3.0",
         "got": "12.6.1",
         "hash-sum": "2.0.0",
@@ -36,10 +36,10 @@
         "js-yaml": "4.1.0",
         "media-typer": "1.1.0",
         "mqtt": "5.11.0",
-        "multer": "2.0.1",
+        "multer": "2.0.2",
         "mustache": "4.2.0",
         "node-watch": "0.7.4",
-        "on-headers": "1.0.2",
+        "on-headers": "1.1.0",
         "raw-body": "3.0.0",
         "tough-cookie": "5.1.2",
         "uuid": "9.0.1",