mirror of
https://github.com/node-red/node-red.git
synced 2023-10-10 13:36:53 +02:00
Fix for Debug node circular references (Issue 17) - Also now limit debug message length ...(more). Added debugMaxLength setting to settings.js - default 512 - so can be overridden if required. (no doubt this will change)
This commit is contained in:
parent
939178720a
commit
6fc2555a2f
@ -19,6 +19,7 @@ var RED = require("../../red/red");
|
|||||||
var util = require("util");
|
var util = require("util");
|
||||||
var ws = require('ws');
|
var ws = require('ws');
|
||||||
var events = require("events");
|
var events = require("events");
|
||||||
|
var debuglength = RED.settings.debugMaxLength||512;
|
||||||
|
|
||||||
function DebugNode(n) {
|
function DebugNode(n) {
|
||||||
RED.nodes.createNode(this,n);
|
RED.nodes.createNode(this,n);
|
||||||
@ -50,14 +51,28 @@ DebugNode.send = function(msg) {
|
|||||||
msg.msg = "(Object) "+JSON.stringify(msg.msg,null,1);
|
msg.msg = "(Object) "+JSON.stringify(msg.msg,null,1);
|
||||||
}
|
}
|
||||||
catch (err) {
|
catch (err) {
|
||||||
console.log(msg.msg);
|
// DCJ - either can just report the circularity...
|
||||||
console.log(err);
|
//msg.msg = "[Error] Can't stringify object with circular reference - see console log.";
|
||||||
msg.msg = "[Error] Can't stringify object with circular reference - see console log.";
|
// or dump out bits we can... (but things like http are mahoosive objects)
|
||||||
|
var seen = [];
|
||||||
|
msg.msg = "(Circular Object) " + JSON.stringify(msg.msg, function(key, value) {
|
||||||
|
if (typeof value === 'object' && value !== null) {
|
||||||
|
if (seen.indexOf(value) !== -1) { return; }
|
||||||
|
seen.push(value);
|
||||||
|
}
|
||||||
|
return value;
|
||||||
|
});
|
||||||
|
seen = null;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
else if (typeof msg.msg === "boolean") msg.msg = "(boolean) "+msg.msg.toString();
|
else if (typeof msg.msg === "boolean") msg.msg = "(boolean) "+msg.msg.toString();
|
||||||
else if (msg.msg === 0) msg.msg = "0";
|
else if (msg.msg === 0) msg.msg = "0";
|
||||||
|
|
||||||
|
if (msg.msg.length > debuglength) {
|
||||||
|
msg.msg = msg.msg.substr(0,debuglength) +" ...(more)";
|
||||||
|
}
|
||||||
|
|
||||||
for (var i in DebugNode.activeConnections) {
|
for (var i in DebugNode.activeConnections) {
|
||||||
var ws = DebugNode.activeConnections[i];
|
var ws = DebugNode.activeConnections[i];
|
||||||
try {
|
try {
|
||||||
|
Loading…
Reference in New Issue
Block a user