Show warning when failing to parse attributes

lib/soap.js

@@ -85,6 +85,8 @@ if (XML_IGNORE_NAMESPACE) {
   };
 }
 
+let warnings;
+
 
 function parseBool(v) {
   v = "" + v;
@@ -93,7 +95,7 @@ function parseBool(v) {
   else if (v === "false" || v === "FALSE" || v === "False" || v === "0")
     return false;
   else
-    throw new Error("Invalid boolean value");
+    return null;
 }
 
 
@@ -103,9 +105,21 @@ function event(xml) {
 
 
 function parameterInfoList(xml) {
-  return xml.find("ParameterInfoStruct").map(
-    e => [e.get("Name").text(), parseBool(e.get("Writable").text())]
-  );
+  return xml.find("ParameterInfoStruct").map(function(e) {
+    let param = e.get("Name").text();
+    let value = e.get("Writable").text();
+    let parsed = parseBool(value);
+
+    if (parsed === null) {
+      warnings.push({
+        message: "Invalid writable attribute",
+        parameter: param
+      });
+      parsed = false;
+    }
+
+    return [param, parsed];
+  });
 }
 
 
@@ -113,15 +127,41 @@ function parameterValueList(xml) {
   return xml.find("ParameterValueStruct").map(function(e) {
     const valueElement = e.get("Value");
     const valueType = valueElement.attr("type").value();
+    let param = e.get("Name").text();
     let value = valueElement.text();
-    if (valueType === "xsd:boolean")
-      value = parseBool(value);
-    else if (valueType === "xsd:int" || valueType === "xsd:unsignedInt")
-      value = parseInt(value);
-    else if (valueType === "xsd:dateTime")
-      value = Date.parse(value);
+    let parsed = value;
+    if (valueType === "xsd:boolean") {
+      parsed = parseBool(value);
+      if (parsed === null) {
+        warnings.push({
+          message: "Invalid value attribute",
+          parameter: param
+        });
+        parsed = value;
+      }
+    }
+    else if (valueType === "xsd:int" || valueType === "xsd:unsignedInt") {
+      parsed = parseInt(value);
+      if (isNaN(parsed)) {
+        warnings.push({
+          message: "Invalid value attribute",
+          parameter: param
+        });
+        parsed = value;
+      }
+    }
+    else if (valueType === "xsd:dateTime") {
+      parsed = Date.parse(value);
+      if (isNaN(parsed)) {
+        warnings.push({
+          message: "Invalid value attribute",
+          parameter: param
+        });
+        parsed = value;
+      }
+    }
 
-     return [e.get("Name").text(), value, valueType];
+     return [param, value, valueType];
   });
 }
 
@@ -367,7 +407,9 @@ function fault(xml, cwmpVersion) {
 }
 
 
-function request(httpRequest, cwmpVersion) {
+function request(httpRequest, cwmpVersion, warn) {
+  warnings = warn;
+
   const rpc = {
     cwmpVersion: cwmpVersion
   };