Check for authentication obj before delete operation #1415 (#1416)

Ride-The-Lightning · Jun 20, 2024 · c54edca · c54edca
1 parent b6dbd23
commit c54edca
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 10 deletions.
diff --git a/backend/utils/common.js b/backend/utils/common.js
@@ -40,11 +40,13 @@ export class CommonService {
             return obj;
         };
         this.removeAuthSecureData = (node) => {
-            delete node.authentication.macaroonPath;
-            delete node.authentication.runePath;
-            delete node.authentication.runeValue;
-            delete node.authentication.lnApiPassword;
-            delete node.authentication.options;
+            if (node.authentication) {
+                delete node.authentication.macaroonPath;
+                delete node.authentication.runePath;
+                delete node.authentication.runeValue;
+                delete node.authentication.lnApiPassword;
+                delete node.authentication.options;
+            }
             return node;
         };
         this.removeSecureData = (config) => {

diff --git a/server/utils/common.ts b/server/utils/common.ts
@@ -47,11 +47,13 @@ export class CommonService {
   };
 
   public removeAuthSecureData = (node: SelectedNode) => {
-    delete node.authentication.macaroonPath;
-    delete node.authentication.runePath;
-    delete node.authentication.runeValue;
-    delete node.authentication.lnApiPassword;
-    delete node.authentication.options;
+    if (node.authentication) {
+      delete node.authentication.macaroonPath;
+      delete node.authentication.runePath;
+      delete node.authentication.runeValue;
+      delete node.authentication.lnApiPassword;
+      delete node.authentication.options;
+    }
     return node;
   };