Fixed issue with connecting to Oracle Database, when the full path of the Node.js executable contains certain non-standard characters like '(' and ')'. (Issue #1664

Author: sharadraju

doc/src/release_notes.rst

@@ -39,6 +39,9 @@ Common Changes
 
 Thin Mode Changes
 ++++++++++++++++++
+#)  Fixed issue with connecting to Oracle Database, when the full path of
+    the Node.js executable contains certain non-standard characters like '(' and ')'.
+    See `Issue #1644 <https://github.com/oracle/node-oracledb/issues/1664>`__.
 
 #)  Added support for easy connect string entries in tnsnames.ora
     See `Issue #1644 <https://github.com/oracle/node-oracledb/issues/1644>`__.

lib/thin/sqlnet/navNodes.js

@@ -777,7 +777,8 @@ class NavDescription extends Description {
       if (this.connectData == null) {
         this.connectData = "(SERVICE_NAME=)";
       }
-      const cid = `(CID=(PROGRAM=${cInfo.program})(HOST=${cInfo.hostName})(USER=${cInfo.userName}))`;
+      const pgmName = "\"'" + cInfo.program + "'\"";
+      const cid = `(CID=(PROGRAM=${pgmName})(HOST=${cInfo.hostName})(USER=${cInfo.userName}))`;
       cOpts[i].CNdata.push(NavSchemaObject.CD);
       cOpts[i].CNdata.push(this.connectData);
       cOpts[i].CNdata.push(cid);