node-firebird 2.0.1 → 2.2.0

package/lib/wire/service.js

@@ -135,102 +135,110 @@ class ServiceManager extends Events.EventEmitter {
     }
 
     _processquery(buffer, callback) {
-        //console.log(buffer);
-        var br = new BlrReader(buffer);
-        var tinfo = br.readByteCode();
-        var res = {};
-        res.result = 0;
-        for (; tinfo !== Const.isc_info_end; tinfo = br.readByteCode()) {
-            switch (tinfo) {
-                case Const.isc_info_svc_server_version:
-                case Const.isc_info_svc_implementation:
-                case Const.isc_info_svc_user_dbpath:
-                case Const.isc_info_svc_get_env:
-                case Const.isc_info_svc_get_env_lock:
-                case Const.isc_info_svc_get_env_msg:
-                    res[this._infosmapping[tinfo]] = br.readString();
-                    break;
-                case Const.isc_info_svc_version:
-                    res[this._infosmapping[tinfo]] = br.readInt32();
-                    break;
-                case Const.isc_info_svc_svr_db_info:
-                    this._processdbinfo(br, res);
-                    break;
-                case Const.isc_info_svc_limbo_trans:
-                    // not implemented
-                    for (; tinfo !== isc_info_flag_end; tinfo = br.readByteCode())
+        if (!Buffer.isBuffer(buffer)) {
+            doError(new Error('Malformed service-manager response: missing BLR buffer'), callback);
+            return;
+        }
+
+        try {
+            var br = new BlrReader(buffer);
+            var tinfo = br.readByteCode();
+            var res = {};
+            res.result = 0;
+            for (; tinfo !== Const.isc_info_end; tinfo = br.readByteCode()) {
+                switch (tinfo) {
+                    case Const.isc_info_svc_server_version:
+                    case Const.isc_info_svc_implementation:
+                    case Const.isc_info_svc_user_dbpath:
+                    case Const.isc_info_svc_get_env:
+                    case Const.isc_info_svc_get_env_lock:
+                    case Const.isc_info_svc_get_env_msg:
+                        res[this._infosmapping[tinfo]] = br.readString();
                         break;
-                case Const.isc_info_svc_get_users:
-                    br.pos += 2
-                    res[this._infosmapping[tinfo]] = [];
-                    break;
-                case Const.isc_spb_sec_username:
-                    var tuser = res[this._infosmapping[68]];
-                    tuser.push({});
-                    tuser[tuser.length - 1].username = br.readString();
-                    break;
-                case Const.isc_spb_sec_firstname:
-                    var tuser = res[this._infosmapping[68]];
-                    var user = tuser[tuser.length-1];
-                    user.firstname = br.readString();
-                    break;
-                case Const.isc_spb_sec_middlename:
-                    var tuser = res[this._infosmapping[68]];
-                    var user = tuser[tuser.length-1];
-                    user.middlename = br.readString();
-                    break;
-                case Const.isc_spb_sec_lastname:
-                    var tuser = res[this._infosmapping[68]];
-                    var user = tuser[tuser.length-1];
-                    user.lastname = br.readString();
-                    break;
-                case Const.isc_spb_sec_groupid:
-                    var tuser = res[this._infosmapping[68]];
-                    var user = tuser[tuser.length-1];
-                    user.groupid = br.readInt32();
-                    break;
-                case Const.isc_spb_sec_userid:
-                    var tuser = res[this._infosmapping[68]];
-                    var user = tuser[tuser.length-1];
-                    user.userid = br.readInt32();
-    
-                    break;
-                case Const.isc_spb_sec_admin:
-                    var tuser = res[this._infosmapping[68]];
-                    var user = tuser[tuser.length-1];
-                    user.admin = br.readInt32();
-                    break;
-    
-                case Const.isc_info_svc_line:
-                    res.line = br.readString();
-                    break;
-    
-                case Const.isc_info_svc_to_eof:
-                    res.line = br.readString();
-                    break;
-    
-                case Const.isc_info_truncated:
-                    res.result = 1; // too much data for the result buffer increase size of it (buffersize parameter))
-                    break;
-    
-                case Const.isc_info_data_not_ready:
-                    res.result = 2;
-                    break;
-    
-                case Const.isc_info_svc_timeout:
-                    res.result = 3;
-                    break;
-    
-                case Const.isc_info_svc_stdin:
-    
-                    break;
-    
-                case Const.isc_info_svc_capabilities:
-                    this._processcapabilities(br, res);
-                    break;
+                    case Const.isc_info_svc_version:
+                        res[this._infosmapping[tinfo]] = br.readInt32();
+                        break;
+                    case Const.isc_info_svc_svr_db_info:
+                        this._processdbinfo(br, res);
+                        break;
+                    case Const.isc_info_svc_limbo_trans:
+                        // not implemented
+                        for (; tinfo !== isc_info_flag_end; tinfo = br.readByteCode())
+                            break;
+                    case Const.isc_info_svc_get_users:
+                        br.pos += 2
+                        res[this._infosmapping[tinfo]] = [];
+                        break;
+                    case Const.isc_spb_sec_username:
+                        var tuser = res[this._infosmapping[68]];
+                        tuser.push({});
+                        tuser[tuser.length - 1].username = br.readString();
+                        break;
+                    case Const.isc_spb_sec_firstname:
+                        var tuser = res[this._infosmapping[68]];
+                        var user = tuser[tuser.length-1];
+                        user.firstname = br.readString();
+                        break;
+                    case Const.isc_spb_sec_middlename:
+                        var tuser = res[this._infosmapping[68]];
+                        var user = tuser[tuser.length-1];
+                        user.middlename = br.readString();
+                        break;
+                    case Const.isc_spb_sec_lastname:
+                        var tuser = res[this._infosmapping[68]];
+                        var user = tuser[tuser.length-1];
+                        user.lastname = br.readString();
+                        break;
+                    case Const.isc_spb_sec_groupid:
+                        var tuser = res[this._infosmapping[68]];
+                        var user = tuser[tuser.length-1];
+                        user.groupid = br.readInt32();
+                        break;
+                    case Const.isc_spb_sec_userid:
+                        var tuser = res[this._infosmapping[68]];
+                        var user = tuser[tuser.length-1];
+                        user.userid = br.readInt32();
+        
+                        break;
+                    case Const.isc_spb_sec_admin:
+                        var tuser = res[this._infosmapping[68]];
+                        var user = tuser[tuser.length-1];
+                        user.admin = br.readInt32();
+                        break;
+        
+                    case Const.isc_info_svc_line:
+                        res.line = br.readString();
+                        break;
+        
+                    case Const.isc_info_svc_to_eof:
+                        res.line = br.readString();
+                        break;
+        
+                    case Const.isc_info_truncated:
+                        res.result = 1; // too much data for the result buffer increase size of it (buffersize parameter))
+                        break;
+        
+                    case Const.isc_info_data_not_ready:
+                        res.result = 2;
+                        break;
+        
+                    case Const.isc_info_svc_timeout:
+                        res.result = 3;
+                        break;
+        
+                    case Const.isc_info_svc_stdin:
+        
+                        break;
+        
+                    case Const.isc_info_svc_capabilities:
+                        this._processcapabilities(br, res);
+                        break;
+                }
             }
+            callback(null, res);
+        } catch (err) {
+            doError(new Error('Malformed service-manager response: ' + err.message), callback);
         }
-        callback(null, res);
     }
 
     detach(callback, force) {
@@ -1047,4 +1055,4 @@ class ServiceManager extends Events.EventEmitter {
 ServiceManager.ShutdownMode = ShutdownMode;
 ServiceManager.ShutdownKind = ShutdownKind;
 
-module.exports = ServiceManager;
+module.exports = ServiceManager;

package/lib/wire/statement.js

@@ -25,15 +25,15 @@ class Statement {
             this.connection.dropStatement(this, callback);
     }
 
-    execute(transaction, params, callback, custom) {
+    execute(transaction, params, callback, options) {
         if (params instanceof Function) {
-            custom = callback;
+            options = callback;
             callback = params;
             params = undefined;
         }
 
-        this.custom = custom;
-        this.connection.executeStatement(transaction, this, params, callback, custom);
+        this.options = options;
+        this.connection.executeStatement(transaction, this, params, callback, options);
     }
 
     fetch(transaction, count, callback) {

package/lib/wire/transaction.js

@@ -25,9 +25,9 @@ class Transaction {
         }
     }
 
-    execute(query, params, callback, custom) {
+    execute(query, params, callback, options) {
         if (params instanceof Function) {
-            custom = callback;
+            options = callback;
             callback = params;
             params = undefined;
         }
@@ -100,13 +100,13 @@ class Transaction {
                         break;
                 }
 
-            }, custom);
+            }, options);
         });
     }
 
-    sequentially(query, params, on, callback, asArray) {
+    sequentially(query, params, on, callback, options = {}) {
         if (params instanceof Function) {
-            asArray = callback;
+            options = callback;
             callback = on;
             on = params;
             params = undefined;
@@ -117,7 +117,7 @@ class Transaction {
         }
 
         if (callback instanceof Boolean) {
-            asArray = callback;
+            options = callback;
             callback = undefined;
         }
 
@@ -152,11 +152,23 @@ class Transaction {
             }
         };
 
-        self.execute(query, params, callback, { asObject: !asArray, asStream: true, on: _on });
+        // back compatibility - options parameter is a boolean
+        if (typeof options === 'boolean') {
+            options = { asObject: !options, asStream: true, on: _on };
+        } else {
+            options = {
+                asStream: true,
+                asObject: true,
+                on: _on,
+                ...options,
+            };
+        }
+
+        self.execute(query, params, callback, options);
         return self;
     }
 
-    query(query, params, callback) {
+    query(query, params, callback, options = {}) {
         if (params instanceof Function) {
             callback = params;
             params = undefined;
@@ -165,7 +177,13 @@ class Transaction {
         if (callback === undefined)
             callback = noop;
 
-        this.execute(query, params, callback, { asObject: true, asStream: callback === undefined || callback === null });
+        options = {
+            asObject: true,
+            asStream: callback === undefined || callback === null,
+            ...options,
+        };
+
+        this.execute(query, params, callback, options);
     }
 
     commit(callback) {

package/lib/wire/xsqlvar.js

@@ -199,6 +199,44 @@ class SQLVarInt128 {
 
 //------------------------------------------------------
 
+class SQLVarDecFloat16 {
+    decode(data, lowerV13) {
+        var ret = data.readDecFloat16();
+
+        if (!lowerV13 || !data.readInt()) {
+            return ret;
+        }
+
+        return null;
+    }
+
+    calcBlr(blr) {
+        blr.addByte(Const.blr_dec64);
+        blr.addShort(0);
+    }
+}
+
+//------------------------------------------------------
+
+class SQLVarDecFloat34 {
+    decode(data, lowerV13) {
+        var ret = data.readDecFloat34();
+
+        if (!lowerV13 || !data.readInt()) {
+            return ret;
+        }
+
+        return null;
+    }
+
+    calcBlr(blr) {
+        blr.addByte(Const.blr_dec128);
+        blr.addShort(0);
+    }
+}
+
+//------------------------------------------------------
+
 class SQLVarFloat {
     decode(data, lowerV13) {
         var ret = data.readFloat();
@@ -295,6 +333,92 @@ class SQLVarTimeStamp {
 
 //------------------------------------------------------
 
+class SQLVarTimeTz {
+    decode(data, lowerV13) {
+        var time = data.readUInt();
+        data.readInt(); // skip timezone info
+
+        if (!lowerV13 || !data.readInt()) {
+            var d = new Date(0);
+            d.setMilliseconds(Math.floor(time / 10) + d.getTimezoneOffset() * MsPerMinute);
+            return d;
+        }
+        return null;
+    }
+
+    calcBlr(blr) {
+        blr.addByte(Const.blr_sql_time_tz);
+    }
+}
+
+//------------------------------------------------------
+
+class SQLVarTimeTzEx extends SQLVarTimeTz {
+    decode(data, lowerV13) {
+        var time = data.readUInt();
+        data.readInt(); // skip timezone info
+        data.readInt(); // skip ext_offset
+
+        if (!lowerV13 || !data.readInt()) {
+            var d = new Date(0);
+            d.setMilliseconds(Math.floor(time / 10) + d.getTimezoneOffset() * MsPerMinute);
+            return d;
+        }
+        return null;
+    }
+
+    calcBlr(blr) {
+        blr.addByte(Const.blr_ex_time_tz);
+    }
+}
+
+//------------------------------------------------------
+
+class SQLVarTimeStampTz {
+    decode(data, lowerV13) {
+        var date = data.readInt();
+        var time = data.readUInt();
+        data.readInt(); // skip timezone info
+
+        if (!lowerV13 || !data.readInt()) {
+            var d = new Date(0);
+            d.setMilliseconds((date - DateOffset) * TimeCoeff + Math.floor(time / 10) + d.getTimezoneOffset() * MsPerMinute);
+            return d;
+        }
+
+        return null;
+    }
+
+    calcBlr(blr) {
+        blr.addByte(Const.blr_timestamp_tz);
+    }
+}
+
+//------------------------------------------------------
+
+class SQLVarTimeStampTzEx extends SQLVarTimeStampTz {
+    decode(data, lowerV13) {
+        var date = data.readInt();
+        var time = data.readUInt();
+        data.readInt(); // skip timezone info
+        data.readInt(); // skip ext_offset
+
+        if (!lowerV13 || !data.readInt()) {
+            var d = new Date(0);
+            d.setMilliseconds((date - DateOffset) * TimeCoeff + Math.floor(time / 10) + d.getTimezoneOffset() * MsPerMinute);
+            return d;
+        }
+
+        return null;
+    }
+
+    calcBlr(blr) {
+        blr.addByte(Const.blr_ex_timestamp_tz);
+    }
+}
+
+//------------------------------------------------------
+
 class SQLVarBoolean {
     decode(data, lowerV13) {
         var ret = data.readInt();
@@ -378,6 +502,50 @@ class SQLParamInt128 {
 
 //------------------------------------------------------
 
+class SQLParamDecFloat16 {
+    constructor(value) {
+        this.value = value;
+    }
+
+    calcBlr(blr) {
+        blr.addByte(Const.blr_dec64);
+        blr.addShort(0);
+    }
+
+    encode(data) {
+        if (this.value != null) {
+            data.addDecFloat16(this.value);
+        } else {
+            data.addDecFloat16(0);
+            data.addInt(1);
+        }
+    }
+}
+
+//------------------------------------------------------
+
+class SQLParamDecFloat34 {
+    constructor(value) {
+        this.value = value;
+    }
+
+    calcBlr(blr) {
+        blr.addByte(Const.blr_dec128);
+        blr.addShort(0);
+    }
+
+    encode(data) {
+        if (this.value != null) {
+            data.addDecFloat34(this.value);
+        } else {
+            data.addDecFloat34(0);
+            data.addInt(1);
+        }
+    }
+}
+
+//------------------------------------------------------
+
 class SQLParamDouble {
     constructor(value) {
         this.value = value;
@@ -509,6 +677,8 @@ module.exports = {
     SQLVarInt,
     SQLVarInt64,
     SQLVarInt128,
+    SQLVarDecFloat16,
+    SQLVarDecFloat34,
     SQLVarFloat,
     SQLVarNull,
     SQLVarShort,
@@ -516,12 +686,18 @@ module.exports = {
     SQLVarText,
     SQLVarTime,
     SQLVarTimeStamp,
+    SQLVarTimeTz,
+    SQLVarTimeTzEx,
+    SQLVarTimeStampTz,
+    SQLVarTimeStampTzEx,
     SQLParamBool,
     SQLParamDate,
     SQLParamDouble,
     SQLParamInt,
     SQLParamInt64,
     SQLParamInt128,
+    SQLParamDecFloat16,
+    SQLParamDecFloat34,
     SQLParamQuad,
     SQLParamString,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "node-firebird",
-    "version": "2.0.1",
+    "version": "2.2.0",
     "description": "Pure JavaScript and Asynchronous Firebird client for Node.js.",
     "keywords": [
         "firebird",
@@ -25,15 +25,16 @@
     "types": "./lib/index.d.ts",
     "license": "MPL-2.0",
     "scripts": {
-        "test": "vitest run"
+        "test": "vitest run",
+        "lint": "oxlint"
     },
     "dependencies": {
         "big-integer": "^1.6.51",
         "long": "^5.2.3"
     },
     "devDependencies": {
-        "coveralls-next": "^4.2.0",
-        "nyc": "^15.0.1",
+        "coveralls-next": "^6.0.1",
+        "oxlint": "^0.15.11",
         "vitest": "^4.0.18"
     }
 }

package/vitest.config.js

@@ -8,6 +8,17 @@ module.exports = defineConfig({
         fileParallelism: false,
         maxWorkers: 1,
         isolate: false,
-        include: ['test/arc4.js', 'test/protocol.js', 'test/srp.js', 'test/service.js', 'test/utf8-user-identification.js', 'test/index.js', 'test/db-crypt-config.js'],
+        include: [
+            'test/arc4.js',
+            'test/protocol.js',
+            'test/srp.js',
+            'test/service.js',
+            'test/utf8-user-identification.js',
+            'test/index.js',
+            'test/db-crypt-config.js',
+            'test/timezone.js',
+            'test/decfloat.js',
+            'test/mock-server.js'
+        ],
     },
 });

package/0001-fix-attachEvent-host-resolution.patch

@@ -1,72 +0,0 @@
-From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
-From: Gobbo89 <gobbo89@users.noreply.github.com>
-Date: Thu, 19 Feb 2026 00:00:00 +0000
-Subject: [PATCH] fix(events): resolve event channel host from connection options
-
-When Firebird is configured with RemoteBindAddress unset (default),
-auxConnection() returns socket_info.host = "0.0.0.0", which is a
-bind-any address and not a routable destination for remote clients.
-
-This commit changes the host selection logic in attachEvent to prefer
-connection.options.host (the address that the user explicitly provided
-and that is proven reachable, since the initial DB attach succeeded).
-
-Only if options.host is absent does it fall back to socket_info.host,
-and even then only when that host is a concrete routable address (i.e.
-not "0.0.0.0" and not "::"). As a last resort it uses the socket's
-remoteAddress, normalizing IPv6-mapped IPv4 (::ffff:x.x.x.x → x.x.x.x).
-
-Fixes #386.
-
----
- lib/wire/database.js | 25 +++++++++++++++++++++++--
- 1 file changed, 23 insertions(+), 2 deletions(-)
-
-diff --git a/lib/wire/database.js b/lib/wire/database.js
-index xxxxxxx..yyyyyyy 100644
---- a/lib/wire/database.js
-+++ b/lib/wire/database.js
-@@ -1,3 +1,18 @@
-+/**
-+ * Resolves the host to use for the auxiliary event channel.
-+ *
-+ * Priority:
-+ *   1. connection.options.host  – explicitly provided by the caller; known reachable.
-+ *   2. socket_info.host         – only when it is a concrete routable address
-+ *                                 (i.e. not "0.0.0.0" and not "::").
-+ *   3. socket.remoteAddress     – the OS-level address of the main connection socket,
-+ *                                 with IPv6-mapped IPv4 prefixes stripped.
-+ *
-+ * @param {object} connection   The active database connection object.
-+ * @param {string} auxHost      The host returned by the auxConnection() call.
-+ * @returns {string}            A connectable host string.
-+ */
-+function resolveEventHost(connection, auxHost) {
-+    // 1. Prefer the host the caller explicitly supplied.
-+    if (connection.options && connection.options.host) {
-+        return connection.options.host;
-+    }
-+
-+    // 2. Use auxConnection host only when it is a real address.
-+    var BIND_ANY = ['0.0.0.0', '::'];
-+    if (auxHost && BIND_ANY.indexOf(auxHost) === -1) {
-+        return auxHost;
-+    }
-+
-+    // 3. Fall back to the OS-reported remote address of the main socket,
-+    //    normalising IPv6-mapped IPv4 (e.g. "::ffff:10.0.0.1" → "10.0.0.1").
-+    var remoteAddress = connection._socket && connection._socket.remoteAddress;
-+    if (remoteAddress) {
-+        return remoteAddress.replace(/^::ffff:/, '');
-+    }
-+
-+    // Last resort: return whatever auxConnection gave us and let the caller fail
-+    // with a meaningful socket error rather than a silent undefined.
-+    return auxHost;
-+}
-+
- // ... (existing code) ...
- 
--        var eventConnection = new EventConnection(socket_info.host, socket_info.port, self.connection.options, eventCallback);
-+        var eventHost = resolveEventHost(self.connection, socket_info.host);
-+        var eventConnection = new EventConnection(eventHost, socket_info.port, self.connection.options, eventCallback);