Would Connection.connect(dbname) support username/password? about nodejs-idb-pconnector HOT 7 CLOSED

    let conn = new Connection();
    
    conn.debug(true);
    dbConn.connect('*LOCAL', 'fake', 'fake');

SQLConnect(-1): conn obj [18031b410] handler [2]

 **** ERROR *****
SQLSTATE: 08001
Native Error Code: -30082
Authorization failure on distributed database connection attempt. 
``

Looking @ the underlying idb-connector code I think I know why the error was not thrown. 


Before converting the idb-connector to be napi compliant the [void throwErrMsg(int handleType, Isolate* isolate)](https://bitbucket.org/litmis/nodejs-idb-connector/src/6ca9ff8060a012177f002d5833d8ad7f0ed3e58c/src/db2ia/dbconn.h?at=1.0.13&fileviewer=file-view-default#dbconn.h-119) would throw an error at the end.

The but somehow the Napi version of [void throwErrMsg(int handleType, Isolate* isolate)](https://bitbucket.org/litmis/nodejs-idb-connector/src/16bf93548e688b92594d7951b886d661e0ecf0aa/src/db2ia/dbconn.h?at=master&fileviewer=file-view-default#dbconn.h-89) does not throw the java script error at the end.

I suspect adding `Napi::Error::New(env, Napi::String::New(env, errMsg)).ThrowAsJavaScriptException();` to line 90 of dbconn.h would fix this :smiley: .

#!js
  ...
  /**
   * Instantiates a new Connection instance
   */
  newConnection() {
    let me = this,
      {url, username = '', password = ''} = me.database;

    me.conn = new idbp.Connection();
    if (!username && !password) {
      me.conn.connect(url);
    } else {
      me.conn.connect(url, username, password);
    }
  }

#!js
  ...
  /**
   * Instantiates a new Connection instance
   */
  newConnection() {
    let me = this,
      {url, username = '', password = ''} = me.database,
      isLocal = url === '*LOCAL';

    me.conn = new idbp.Connection();
    if (isLocal && !username && !password) {
      me.conn.connect(url);
    } else {
      me.conn.connect(url, username, password);
    }
  }

#!javascript
  ...

  /**
   * Constructor to instantiate a new instance of a Connection class given the `poolIndex` and `config`
   * @param poolIndex An identifier for debug purposes
   * @param config Object includes `database`
   */
  constructor(poolIndex, config = {database: {url: '*LOCAL'}}) {
    let me = this,
      {database} = config;

    // Defaults are initialized if `config` does not have all properties defined at instantiation.
    me.database = database || {url: '*LOCAL'};
    ...
    me.newConnection();
    ...
  }

  /**
   * Instantiates a new Connection instance
   */
  newConnection() {
    let me = this,
      {url, username = '', password = ''} = me.database,
      isLocal = url === '*LOCAL';

    me.conn = new idbp.Connection();
    if (isLocal) {
      me.conn.connect(url);
    } else {
      // Username and Password are assumed blank if not specified with non-local URL.
      me.conn.connect(url, username, password);
    }
  }