[issue112] Fix more UNICODE issues (#117)

* [issue112] Fix more UNICODE issues (#116)

* Fix column names in results being single characters when UNICODE is defined
* Fix statement on result object single character when UNICODE is defined
* Bump version to 2.3.4
* Update CHANGELOG
* Regenerate package-lock.json

Signed-off-by: Mark Irish <[email protected]>

Author: markirish

CHANGELOG.md

@@ -1,6 +1,11 @@
 # Changelog
 All notable changes to this project will be documented in this file.
 
+## [2.3.4] - 2020-08-09
+### Fixed
+- Fixed bug when UNICODE is defined where statement in result object and column name properties wouldn't encode correctly
+- Update package-lock.json with vulnerability fixes
+
 ## [2.3.3] - 2020-07-31
 ### Fixed
 - Fixed bug when UNICODE is defined where error message, error state, and column names wouldn't encode correctly

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "odbc",
   "description": "unixodbc bindings for node",
-  "version": "2.3.3",
+  "version": "2.3.4",
   "homepage": "http://github.com/markdirish/node-odbc/",
   "main": "lib/odbc.js",
   "types": "lib/odbc.d.ts",

package.json

@@ -294,7 +294,11 @@ Napi::Array ODBCConnection::ProcessDataForNapi(Napi::Env env, QueryData *data, N
   if (data->sql == NULL) {
     rows.Set(Napi::String::New(env, STATEMENT), env.Null());
   } else {
+    #ifdef UNICODE
+    rows.Set(Napi::String::New(env, STATEMENT), Napi::String::New(env, (const char16_t*)data->sql));
+    #else
     rows.Set(Napi::String::New(env, STATEMENT), Napi::String::New(env, (const char*)data->sql));
+    #endif
   }
 
   // set the 'parameters' property
@@ -434,7 +438,11 @@ Napi::Array ODBCConnection::ProcessDataForNapi(Napi::Env env, QueryData *data, N
       if (this->connectionOptions.fetchArray == true) {
         row.Set(j, value);
       } else {
+        #ifdef UNICODE
+        row.Set(Napi::String::New(env, (const char16_t*)columns[j]->ColumnName), value);
+        #else
         row.Set(Napi::String::New(env, (const char*)columns[j]->ColumnName), value);
+        #endif
       }
     }
     rows.Set(i, row);