node-opcua-transport 2.68.0 → 2.70.0

package/dist/test_helpers/direct_transport.d.ts

@@ -1,6 +1,10 @@
 /// <reference types="node" />
+/// <reference types="node" />
 import { EventEmitter } from "events";
 import { HalfComChannel } from "./half_com_channel";
+export interface DirectTransport {
+    on(eventName: "end", eventHandler: () => void): this;
+}
 export declare class DirectTransport extends EventEmitter {
     client: HalfComChannel;
     server: HalfComChannel;

package/dist/test_helpers/direct_transport.js.map

@@ -1 +1 @@
-{"version":3,"file":"direct_transport.js","sourceRoot":"","sources":["../../test_helpers/direct_transport.ts"],"names":[],"mappings":";;;AAAA,mCAAsC;AACtC,yDAA2C;AAC3C,sCAA6C;AAC7C,yDAAoD;AAEpD,MAAa,eAAgB,SAAQ,qBAAY;IAM7C;QACI,KAAK,EAAE,CAAC;QAER,IAAI,CAAC,MAAM,GAAG,IAAI,iCAAc,EAAE,CAAC;QACnC,IAAI,CAAC,MAAM,GAAG,IAAI,iCAAc,EAAE,CAAC;QAEnC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,IAAI,EAAE,EAAE;YACjC,IAAA,0BAAM,EAAC,IAAI,YAAY,MAAM,CAAC,CAAC;YAC/B,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;QACnC,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,IAAI,EAAE,EAAE;YACjC,IAAA,0BAAM,EAAC,IAAI,YAAY,MAAM,CAAC,CAAC;YAC/B,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;QACnC,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,QAAQ,EAAE,GAAG,EAAE;YAC1B,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACxB,IAAI,CAAC,MAAM,CAAC,SAAS,GAAG,IAAI,CAAC;QACjC,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,QAAQ,EAAE,GAAG,EAAE;YAC1B,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACxB,IAAI,CAAC,MAAM,CAAC,SAAS,GAAG,IAAI,CAAC;QACjC,CAAC,CAAC,CAAC;QAEH,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,KAAK,EAAE,CAAC,GAAU,EAAE,EAAE;YACjC,IAAI,CAAC,IAAI,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;QAC1B,CAAC,CAAC,CAAC;QAEH,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,IAAY,EAAE,EAAE;YACpC,MAAM,IAAI,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;YAChC,IAAI,IAAI,EAAE;gBACN,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;aAC3B;QACL,CAAC,CAAC,CAAC;QAEH,IAAI,CAAC,GAAG,GAAG,mCAAmC,CAAC;IACnD,CAAC;IAEM,UAAU,CAAC,IAAgB;QAC9B,IAAA,yBAAgB,EAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QAC9B,IAAI,EAAE,CAAC;IACX,CAAC;IAEM,QAAQ,CAAC,IAAgB;QAC5B,IAAI,CAAC,MAAM,CAAC,GAAG,EAAE,CAAC;QAClB,IAAI,CAAC,MAAM,CAAC,GAAG,EAAE,CAAC;QAClB,IAAI,IAAI,EAAE;YACN,YAAY,CAAC,IAAI,CAAC,CAAC;SACtB;IACL,CAAC;IAEM,WAAW;QACd,IAAI,CAAC,IAAI,CAAC,UAAU,EAAE;YAClB,OAAO,IAAI,CAAC;SACf;QACD,OAAO,IAAI,CAAC,UAAU,CAAC,KAAK,EAAE,CAAC;IACnC,CAAC;IAEM,YAAY,CAAC,IAAoD;QACpE,IAAI,CAAC,UAAU,GAAG,IAAI,CAAC,UAAU,IAAI,EAAE,CAAC;QACxC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC/B,CAAC;CACJ;AAnED,0CAmEC"}
+{"version":3,"file":"direct_transport.js","sourceRoot":"","sources":["../../test_helpers/direct_transport.ts"],"names":[],"mappings":";;;AAAA,mCAAsC;AACtC,yDAA2C;AAC3C,sCAA6C;AAC7C,yDAAoD;AAKpD,MAAa,eAAgB,SAAQ,qBAAY;IAM7C;QACI,KAAK,EAAE,CAAC;QAER,IAAI,CAAC,MAAM,GAAG,IAAI,iCAAc,EAAE,CAAC;QACnC,IAAI,CAAC,MAAM,GAAG,IAAI,iCAAc,EAAE,CAAC;QAEnC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,IAAI,EAAE,EAAE;YACjC,IAAA,0BAAM,EAAC,IAAI,YAAY,MAAM,CAAC,CAAC;YAC/B,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;QACnC,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,IAAI,EAAE,EAAE;YACjC,IAAA,0BAAM,EAAC,IAAI,YAAY,MAAM,CAAC,CAAC;YAC/B,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;QACnC,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,QAAQ,EAAE,GAAG,EAAE;YAC1B,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACxB,IAAI,CAAC,MAAM,CAAC,SAAS,GAAG,IAAI,CAAC;QACjC,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,QAAQ,EAAE,GAAG,EAAE;YAC1B,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACxB,IAAI,CAAC,MAAM,CAAC,SAAS,GAAG,IAAI,CAAC;QACjC,CAAC,CAAC,CAAC;QAEH,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,KAAK,EAAE,CAAC,GAAW,EAAE,EAAE;YAClC,IAAI,CAAC,IAAI,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;QAC1B,CAAC,CAAC,CAAC;QAEH,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,IAAY,EAAE,EAAE;YACpC,MAAM,IAAI,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;YAChC,IAAI,IAAI,EAAE;gBACN,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;aAC3B;QACL,CAAC,CAAC,CAAC;QAEH,IAAI,CAAC,GAAG,GAAG,mCAAmC,CAAC;IACnD,CAAC;IAEM,UAAU,CAAC,IAAgB;QAC9B,IAAA,yBAAgB,EAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QAC9B,IAAI,EAAE,CAAC;IACX,CAAC;IAEM,QAAQ,CAAC,IAAgB;QAC5B,IAAI,CAAC,MAAM,CAAC,GAAG,EAAE,CAAC;QAClB,IAAI,CAAC,MAAM,CAAC,GAAG,EAAE,CAAC;QAClB,IAAI,IAAI,EAAE;YACN,YAAY,CAAC,IAAI,CAAC,CAAC;SACtB;IACL,CAAC;IAEM,WAAW;QACd,IAAI,CAAC,IAAI,CAAC,UAAU,EAAE;YAClB,OAAO,IAAI,CAAC;SACf;QACD,OAAO,IAAI,CAAC,UAAU,CAAC,KAAK,EAAE,CAAC;IACnC,CAAC;IAEM,YAAY,CAAC,IAAoD;QACpE,IAAI,CAAC,UAAU,GAAG,IAAI,CAAC,UAAU,IAAI,EAAE,CAAC;QACxC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC/B,CAAC;CACJ;AAnED,0CAmEC"}

package/dist/test_helpers/fake_server.d.ts

@@ -1,4 +1,6 @@
 /// <reference types="node" />
+/// <reference types="node" />
+/// <reference types="node" />
 import { EventEmitter } from "events";
 import * as net from "net";
 export declare class FakeServer extends EventEmitter {

package/dist/test_helpers/half_com_channel.d.ts

@@ -1,5 +1,12 @@
 /// <reference types="node" />
+/// <reference types="node" />
 import { EventEmitter } from "events";
+export interface HalfComChannel {
+    on(eventName: "data", eventHandler: (data: Buffer) => void): this;
+    on(eventName: "send_data", eventHandler: (data: Buffer) => void): this;
+    on(eventName: "ending", eventHandler: () => void): this;
+    on(eventName: "end", eventHandler: (err?: Error) => void): this;
+}
 export declare class HalfComChannel extends EventEmitter {
     _hasEnded: boolean;
     constructor();

package/dist/test_helpers/half_com_channel.js.map

@@ -1 +1 @@
-{"version":3,"file":"half_com_channel.js","sourceRoot":"","sources":["../../test_helpers/half_com_channel.ts"],"names":[],"mappings":";;;AAAA,yDAAyD;AACzD,mCAAsC;AACtC,yDAA2C;AAE3C,MAAa,cAAe,SAAQ,qBAAY;IAG5C;QACI,KAAK,EAAE,CAAC;QACR,IAAI,CAAC,SAAS,GAAG,KAAK,CAAC;IAC3B,CAAC;IAEM,KAAK,CAAC,IAAqB;QAC9B,IAAI,OAAO,IAAI,KAAK,QAAQ,EAAE;YAC1B,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;SAC5B;QACD,IAAA,0BAAM,EAAC,IAAI,YAAY,MAAM,EAAE,yCAAyC,CAAC,CAAC;QAC1E,MAAM,IAAI,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC;QACnC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,IAAI,CAAC,CAAC;IACjC,CAAC;IAEM,GAAG;QACN,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE;YACjB,IAAA,0BAAM,EAAC,CAAC,IAAI,CAAC,SAAS,EAAE,gDAAgD,CAAC,CAAC;YAC1E,IAAI,CAAC,SAAS,GAAG,IAAI,CAAC;YACtB,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YACpB,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;SACpB;IACL,CAAC;IAEM,OAAO,KAAU,CAAC;IAElB,UAAU,KAAU,CAAC;CAC/B;AA7BD,wCA6BC"}
+{"version":3,"file":"half_com_channel.js","sourceRoot":"","sources":["../../test_helpers/half_com_channel.ts"],"names":[],"mappings":";;;AAAA,yDAAyD;AACzD,mCAAsC;AACtC,yDAA2C;AAS3C,MAAa,cAAe,SAAQ,qBAAY;IAG5C;QACI,KAAK,EAAE,CAAC;QACR,IAAI,CAAC,SAAS,GAAG,KAAK,CAAC;IAC3B,CAAC;IAEM,KAAK,CAAC,IAAqB;QAC9B,IAAI,OAAO,IAAI,KAAK,QAAQ,EAAE;YAC1B,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;SAC5B;QACD,IAAA,0BAAM,EAAC,IAAI,YAAY,MAAM,EAAE,yCAAyC,CAAC,CAAC;QAC1E,MAAM,IAAI,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC;QACnC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,IAAI,CAAC,CAAC;IACjC,CAAC;IAEM,GAAG;QACN,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE;YACjB,IAAA,0BAAM,EAAC,CAAC,IAAI,CAAC,SAAS,EAAE,gDAAgD,CAAC,CAAC;YAC1E,IAAI,CAAC,SAAS,GAAG,IAAI,CAAC;YACtB,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YACpB,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;SACpB;IACL,CAAC;IAEM,OAAO,KAAU,CAAC;IAElB,UAAU,KAAU,CAAC;CAC/B;AA7BD,wCA6BC"}

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "node-opcua-transport",
-    "version": "2.68.0",
+    "version": "2.70.0",
     "description": "pure nodejs OPCUA SDK - module -transport",
     "main": "./dist/source/index.js",
     "types": "./dist/source/index.d.ts",
@@ -14,24 +14,23 @@
     "dependencies": {
         "chalk": "4.1.2",
         "node-opcua-assert": "2.66.0",
-        "node-opcua-basic-types": "2.68.0",
-        "node-opcua-binary-stream": "2.67.0",
-        "node-opcua-buffer-utils": "2.67.0",
-        "node-opcua-chunkmanager": "2.68.0",
-        "node-opcua-debug": "2.68.0",
-        "node-opcua-factory": "2.68.0",
-        "node-opcua-nodeid": "2.68.0",
-        "node-opcua-object-registry": "2.68.0",
-        "node-opcua-packet-assembler": "2.66.0",
-        "node-opcua-status-code": "2.67.0",
-        "node-opcua-types": "2.68.0",
-        "node-opcua-utils": "2.67.0"
+        "node-opcua-basic-types": "2.70.0",
+        "node-opcua-binary-stream": "2.69.0",
+        "node-opcua-buffer-utils": "2.69.0",
+        "node-opcua-chunkmanager": "2.70.0",
+        "node-opcua-debug": "2.69.0",
+        "node-opcua-factory": "2.70.0",
+        "node-opcua-nodeid": "2.70.0",
+        "node-opcua-object-registry": "2.69.0",
+        "node-opcua-packet-assembler": "2.69.0",
+        "node-opcua-status-code": "2.70.0",
+        "node-opcua-types": "2.70.0",
+        "node-opcua-utils": "2.69.0"
     },
     "devDependencies": {
-        "@types/node": "17.0.23",
-        "node-opcua-debug": "2.66.0",
+        "@types/node": "17.0.36",
         "should": "^13.2.3",
-        "sinon": "^13.0.1"
+        "sinon": "^14.0.0"
     },
     "author": "Etienne Rossignon",
     "license": "MIT",
@@ -48,5 +47,5 @@
         "internet of things"
     ],
     "homepage": "http://node-opcua.github.io/",
-    "gitHead": "363b466440d0910acddd0cde2c37792ce4724d76"
+    "gitHead": "3b692ffb3e2ef0542da32f349fb0c76b3acc75ce"
 }

package/source/client_tcp_transport.ts

@@ -56,12 +56,12 @@ function createClientSocket(endpointUrl: string): Socket {
     }
 }
 export interface ClientTCP_transport {
-    on(eventName: "message", eventHandler: (message: Buffer) => void): this;
+    on(eventName: "chunk", eventHandler: (messageChunk: Buffer) => void): this;
     on(eventName: "socket_closed", eventHandler: (err: Error | null) => void): this;
     on(eventName: "close", eventHandler: (err: Error | null) => void): this;
     on(eventName: "connection_break", eventHandler: () => void): this;
 
-    once(eventName: "message", eventHandler: (message: Buffer) => void): this;
+    once(eventName: "chunk", eventHandler: (messageChunk: Buffer) => void): this;
     once(eventName: "socket_closed", eventHandler: (err: Error | null) => void): this;
     once(eventName: "close", eventHandler: (err: Error | null) => void): this;
     once(eventName: "connection_break", eventHandler: () => void): this;
@@ -96,8 +96,8 @@ export interface ClientTCP_transport {
  *
  *    ....
  *
- *    transport.on("message",function(message_chunk) {
- *        // do something with message from server...
+ *    transport.on("chunk",function(message_chunk) {
+ *        // do something with chunk from server...
  *    });
  *
  *
@@ -106,6 +106,11 @@ export interface ClientTCP_transport {
  *
  */
 export class ClientTCP_transport extends TCP_transport {
+    public static defaultMaxChunk = 0; // 0 - no limits
+    public static defaultMaxMessageSize = 0; // 0 - no limits
+    public static defaultReceiveBufferSize = 1024 * 64 * 10;
+    public static defaultSendBufferSize = 1024 * 64 * 10; // 8192 min,
+
     public endpointUrl: string;
     public serverUri: string;
     public numberOfRetry: number;
@@ -121,6 +126,11 @@ export class ClientTCP_transport extends TCP_transport {
         this.serverUri = "";
         this._counter = 0;
         this.numberOfRetry = 0;
+
+        // initially before HEL/ACK
+        this.maxChunkCount = 1;
+        this.maxMessageSize = 4 * 1024;
+        this.receiveBufferSize = 4 * 1024;
     }
 
     public dispose(): void {
@@ -238,6 +248,7 @@ export class ClientTCP_transport extends TCP_transport {
         this._socket.once("error", _on_socket_error_for_connect);
         this._socket.once("end", _on_socket_end_for_connect);
         this._socket.once("connect", _on_socket_connect);
+
         this._install_socket(this._socket);
     }
 
@@ -280,7 +291,9 @@ export class ClientTCP_transport extends TCP_transport {
             responseClass = AcknowledgeMessage;
             _stream.rewind();
             response = decodeMessage(_stream, responseClass);
+
             this.parameters = response as AcknowledgeMessage;
+            this.setLimits(response as AcknowledgeMessage);
 
             // istanbul ignore next
             if (doTraceHelloAck) {
@@ -304,12 +317,11 @@ export class ClientTCP_transport extends TCP_transport {
         // the server will receive it as message from the client
         const helloMessage = new HelloMessage({
             endpointUrl: this.endpointUrl,
-            maxChunkCount: 0, // 0 - no limits
-            maxMessageSize: 0, // 0 - no limits
-
             protocolVersion: this.protocolVersion,
-            receiveBufferSize: 1024 * 64 * 10,
-            sendBufferSize: 1024 * 64 * 10 // 8192 min,
+            maxChunkCount: ClientTCP_transport.defaultMaxChunk,
+            maxMessageSize: ClientTCP_transport.defaultMaxMessageSize,
+            receiveBufferSize: ClientTCP_transport.defaultReceiveBufferSize,
+            sendBufferSize: ClientTCP_transport.defaultSendBufferSize
         });
         // istanbul ignore next
         if (doTraceHelloAck) {

package/source/index.ts

@@ -1,11 +1,13 @@
 /**
  * @module node-opcua-transport
  */
-export * from "./HelloMessage";
 export * from "./AcknowledgeMessage";
-export * from "./TCPErrorMessage";
 export * from "./client_tcp_transport";
+export * from "./HelloMessage";
+export * from "./message_builder_base";
 export * from "./server_tcp_transport";
+export * from "./status_codes";
 export * from "./tcp_transport";
+export * from "./TCPErrorMessage";
 export * from "./tools";
-export * from "./message_builder_base";
+export * from "./utils";

package/source/message_builder_base.ts

@@ -7,14 +7,17 @@ import { assert } from "node-opcua-assert";
 import { BinaryStream } from "node-opcua-binary-stream";
 import { createFastUninitializedBuffer } from "node-opcua-buffer-utils";
 import { readMessageHeader, SequenceHeader } from "node-opcua-chunkmanager";
-import { make_errorLog, make_debugLog } from "node-opcua-debug";
+import { make_errorLog, make_debugLog, make_warningLog } from "node-opcua-debug";
 import { MessageHeader, PacketAssembler, PacketInfo } from "node-opcua-packet-assembler";
+import { StatusCode } from "node-opcua-status-code";
 import { get_clock_tick } from "node-opcua-utils";
+import { StatusCodes2 } from "./status_codes";
 
 const doPerfMonitoring = process.env.NODEOPCUADEBUG && process.env.NODEOPCUADEBUG.indexOf("PERF") >= 0;
 
 const errorLog = make_errorLog("MessageBuilder");
 const debugLog = make_debugLog("MessageBuilder");
+const warningLog = make_warningLog("MessageBuilder");
 
 export function readRawMessageHeader(data: Buffer): PacketInfo {
     const messageHeader = readMessageHeader(new BinaryStream(data));
@@ -25,6 +28,24 @@ export function readRawMessageHeader(data: Buffer): PacketInfo {
     };
 }
 
+export interface MessageBuilderBaseOptions {
+    signatureLength?: number;
+    maxMessageSize?: number;
+    maxChunkCount?: number;
+    maxChunkSize?: number;
+}
+
+export interface MessageBuilderBase {
+    on(eventName: "startChunk", eventHandler: (info: PacketInfo, data: Buffer) => void): this;
+    on(eventName: "chunk", eventHandler: (chunk: Buffer) => void): this;
+    on(eventName: "error", eventHandler: (err: Error, statusCode: StatusCode, requestId: number | null) => void): this;
+    on(eventName: "full_message_body", eventHandler: (fullMessageBody: Buffer) => void): this;
+
+    emit(eventName: "startChunk", info: PacketInfo, data: Buffer): boolean;
+    emit(eventName: "chunk", chunk: Buffer): boolean;
+    emit(eventName: "error", err: Error, statusCode: StatusCode, requestId: number | null): boolean;
+    emit(eventName: "full_message_body", fullMessageBody: Buffer): boolean;
+}
 /**
  * @class MessageBuilderBase
  * @extends EventEmitter
@@ -35,8 +56,16 @@ export function readRawMessageHeader(data: Buffer): PacketInfo {
  *
  */
 export class MessageBuilderBase extends EventEmitter {
-    public readonly signatureLength: number;    
-    public readonly options: { signatureLength?: number };
+    public static defaultMaxChunkCount = 1000;
+    public static defaultMaxMessageSize = 1024 * 64;
+    public static defaultMaxChunkSize = 1024 * 8;
+
+    public readonly signatureLength: number;
+    public readonly maxMessageSize: number;
+    public readonly maxChunkCount: number;
+    public readonly maxChunkSize: number;
+
+    public readonly options: MessageBuilderBaseOptions;
     public readonly _packetAssembler: PacketAssembler;
     public channelId: number;
     public totalMessageSize: number;
@@ -57,7 +86,7 @@ export class MessageBuilderBase extends EventEmitter {
     private readonly _expectedChannelId: number;
     private offsetBodyStart: number;
 
-    constructor(options?: { signatureLength?: number }) {
+    constructor(options?: MessageBuilderBaseOptions) {
         super();
 
         this.id = "";
@@ -69,20 +98,29 @@ export class MessageBuilderBase extends EventEmitter {
         this.messageChunks = [];
         this._expectedChannelId = 0;
 
-        options = options || {};
+        options = options || {
+            maxMessageSize: 0,
+            maxChunkCount: 0,
+            maxChunkSize: 0
+        };
 
         this.signatureLength = options.signatureLength || 0;
 
+        this.maxMessageSize = options.maxMessageSize || MessageBuilderBase.defaultMaxMessageSize;
+        this.maxChunkCount = options.maxChunkCount || MessageBuilderBase.defaultMaxChunkCount;
+        this.maxChunkSize = options.maxChunkSize || MessageBuilderBase.defaultMaxChunkSize;
+
         this.options = options;
 
         this._packetAssembler = new PacketAssembler({
-            minimumSizeInBytes: 0,
-            readMessageFunc: readRawMessageHeader
+            minimumSizeInBytes: 8,
+            maxChunkSize: this.maxChunkSize,
+            readChunkFunc: readRawMessageHeader
         });
 
-        this._packetAssembler.on("message", (messageChunk) => this._feed_messageChunk(messageChunk));
+        this._packetAssembler.on("chunk", (messageChunk) => this._feed_messageChunk(messageChunk));
 
-        this._packetAssembler.on("newMessage", (info, data) => {
+        this._packetAssembler.on("startChunk", (info, data) => {
             if (doPerfMonitoring) {
                 // record tick 0: when the first data is received
                 this._tick0 = get_clock_tick();
@@ -94,7 +132,12 @@ export class MessageBuilderBase extends EventEmitter {
              * @param info
              * @param data
              */
-            this.emit("start_chunk", info, data);
+            this.emit("startChunk", info, data);
+        });
+
+        this._packetAssembler.on("error", (err) => {
+            warningLog("packet assembler ", err.message);
+            return this._report_error(StatusCodes2.BadTcpMessageTooLarge, "packet assembler: " + err.message);
         });
 
         this._securityDefeated = false;
@@ -135,15 +178,15 @@ export class MessageBuilderBase extends EventEmitter {
 
             // verifying secure ChannelId
             if (this._expectedChannelId && this.channelId !== this._expectedChannelId) {
-                return this._report_error("Invalid secure channel Id");
+                return this._report_error(StatusCodes2.BadTcpSecureChannelUnknown, "Invalid secure channel Id");
             }
             return true;
         } catch (err) {
-            return false;
+            return this._report_error(StatusCodes2.BadTcpInternalError, "_read_headers error " + (err as Error).message);
         }
     }
 
-    protected _report_error(errorMessage: string): false {
+    protected _report_error(statusCode: StatusCode, errorMessage: string): false {
         this._hasReceivedError = true;
         /**
          * notify the observers that an error has occurred
@@ -152,7 +195,7 @@ export class MessageBuilderBase extends EventEmitter {
          */
         debugLog("Error  ", this.id, errorMessage);
         // xx errorLog(new Error());
-        this.emit("error", new Error(errorMessage), this.sequenceHeader ? this.sequenceHeader.requestId : null);
+        this.emit("error", new Error(errorMessage), statusCode, this.sequenceHeader ? this.sequenceHeader.requestId : null);
         return false;
     }
 
@@ -177,13 +220,21 @@ export class MessageBuilderBase extends EventEmitter {
             return false;
         }
 
+        if (this.messageChunks.length + 1 > this.maxChunkCount) {
+            return this._report_error(StatusCodes2.BadTcpMessageTooLarge, `max chunk count exceeded: ${this.maxChunkCount}`);
+        }
+
         this.messageChunks.push(chunk);
         this.totalMessageSize += chunk.length;
 
+        if (this.totalMessageSize > this.maxMessageSize) {
+            return this._report_error(StatusCodes2.BadTcpMessageTooLarge, `max message size exceeded: ${this.maxMessageSize}`);
+        }
+
         const binaryStream = new BinaryStream(chunk);
 
         if (!this._read_headers(binaryStream)) {
-            return this._report_error(`Invalid message header detected`);
+            return this._report_error(StatusCodes2.BadTcpInternalError, `Invalid message header detected`);
         }
 
         assert(binaryStream.length >= 12);
@@ -192,6 +243,7 @@ export class MessageBuilderBase extends EventEmitter {
         if (this.messageHeader!.length !== chunk.length) {
             // tslint:disable:max-line-length
             return this._report_error(
+                StatusCodes2.BadTcpInternalError,
                 `Invalid messageChunk size: the provided chunk is ${chunk.length} bytes long but header specifies ${
                     this.messageHeader!.length
                 }`
@@ -254,7 +306,7 @@ export class MessageBuilderBase extends EventEmitter {
             this._init_new();
             return true;
         } else if (messageHeader.isFinal === "A") {
-            return this._report_error("received and Abort Message");
+            return this._report_error(StatusCodes2.BadRequestInterrupted, "received and Abort Message");
         } else if (messageHeader.isFinal === "C") {
             return this._append(chunk);
         }

package/source/server_tcp_transport.ts

@@ -55,12 +55,6 @@ const minimumBufferSize = 8192;
 export class ServerTCP_transport extends TCP_transport {
     public static throttleTime = 1000;
 
-    public receiveBufferSize: number;
-    public sendBufferSize: number;
-    public maxMessageSize: number;
-    public maxChunkCount: number;
-    public protocolVersion: number;
-
     private _aborted: number;
     private _helloReceived: boolean;
 
@@ -68,14 +62,16 @@ export class ServerTCP_transport extends TCP_transport {
         super();
         this._aborted = 0;
         this._helloReceived = false;
-        this.receiveBufferSize = 0;
-        this.sendBufferSize = 0;
-        this.maxMessageSize = 0;
-        this.maxChunkCount = 0;
-        this.protocolVersion = 0;
+
+        // before HEL/ACK
+        this.maxChunkCount = 1;
+        this.maxMessageSize = 4 * 1024;
+        this.receiveBufferSize = 4 * 1024;
     }
 
     protected _write_chunk(messageChunk: Buffer): void {
+
+        // istanbul ignore next
         if (this.sendBufferSize > 0 && messageChunk.length > this.sendBufferSize) {
             errorLog(
                 "write chunk exceed sendBufferSize messageChunk length = ",
@@ -84,6 +80,7 @@ export class ServerTCP_transport extends TCP_transport {
                 this.sendBufferSize
             );
         }
+        
         super._write_chunk(messageChunk);
     }
     /**
@@ -115,57 +112,63 @@ export class ServerTCP_transport extends TCP_transport {
     public abortWithError(statusCode: StatusCode, extraErrorDescription: string, callback: ErrorCallback): void {
         return this._abortWithError(statusCode, extraErrorDescription, callback);
     }
+
     private _abortWithError(statusCode: StatusCode, extraErrorDescription: string, callback: ErrorCallback): void {
-        if (debugLog) {
-            debugLog(chalk.cyan("_abortWithError"));
+        // When a fatal error occurs, the Server shall send an Error Message to the Client and
+        // closes the TransportConnection gracefully.
+        doDebug && debugLog(chalk.cyan("_abortWithError"));
+        
+        /* istanbul ignore next */
+        if (this._aborted) {
+            // already called
+            return callback(new Error(statusCode.name));
         }
+        this._aborted = 1;
 
-        assert(typeof callback === "function", "expecting a callback");
+        setTimeout(() => {
+            // send the error message and close the connection
+            this.sendErrorMessage(statusCode, statusCode.description);
 
-        /* istanbul ignore else */
-        if (!this._aborted) {
-            this._aborted = 1;
-            setTimeout(() => {
-                // send the error message and close the connection
-                assert(Object.prototype.hasOwnProperty.call(StatusCodes, statusCode.name));
+            this.disconnect(() => {
+                this._aborted = 2;
+                callback(new Error(extraErrorDescription + " StatusCode = " + statusCode.name));
+            });
+        }, ServerTCP_transport.throttleTime);
+    }
 
-                /* istanbul ignore next*/
-                if (doDebug) {
-                    debugLog(chalk.red(" Server aborting because ") + chalk.cyan(statusCode.name));
-                    debugLog(chalk.red(" extraErrorDescription   ") + chalk.cyan(extraErrorDescription));
-                }
+    private _send_ACK_response(helloMessage: HelloMessage): void {
+        assert(helloMessage.receiveBufferSize >= minimumBufferSize);
+        assert(helloMessage.sendBufferSize >= minimumBufferSize);
 
-                const errorResponse = new TCPErrorMessage({
-                    reason: statusCode.description,
-                    statusCode
-                });
+        const minBufferSize = 8192;
+        const maxBufferSize = 64 * 1024;
 
-                const messageChunk = packTcpMessage("ERR", errorResponse);
+        const minMaxMessageSize = 128 * 1024;
+        const defaultMaxMessageSize = 16 * 1024 * 1024;
+        const maxMaxMessageSize = 128 * 1024 * 1024;
 
-                this.write(messageChunk);
+        const minMaxChunkCount = 1;
+        const defaultMaxChunkCount = defaultMaxMessageSize / maxBufferSize;
+        const maxMaxChunkCount = 9000;
 
-                this.disconnect(() => {
-                    this._aborted = 2;
-                    callback(new Error(extraErrorDescription + " StatusCode = " + statusCode.name));
-                });
-            }, ServerTCP_transport.throttleTime);
-        } else {
-            callback(new Error(statusCode.name));
-        }
-    }
+        const receiveBufferSize = 32 * 1024;
+        const sendBufferSize = 32 * 1024;
 
-    private _send_ACK_response(helloMessage: HelloMessage): void {
-        assert(helloMessage.receiveBufferSize >= minimumBufferSize);
-        assert(helloMessage.sendBufferSize >= minimumBufferSize);
+        if (!helloMessage.maxChunkCount && helloMessage.sendBufferSize) {
+            helloMessage.maxChunkCount = helloMessage.maxMessageSize / helloMessage.sendBufferSize;
+        }
 
-        this.receiveBufferSize = clamp_value(helloMessage.receiveBufferSize, 8192, 512 * 1024);
-        this.sendBufferSize = clamp_value(helloMessage.sendBufferSize, 8192, 512 * 1024);
-        this.maxMessageSize = clamp_value(helloMessage.maxMessageSize, 100000, 64 * 1024 * 1024);
-        this.maxChunkCount = clamp_value(helloMessage.maxChunkCount, 0, 65535);
+        this.setLimits({
+            receiveBufferSize: clamp_value(helloMessage.receiveBufferSize || receiveBufferSize, minBufferSize, maxBufferSize),
+            sendBufferSize: clamp_value(helloMessage.sendBufferSize || sendBufferSize, minBufferSize, maxBufferSize),
+            maxMessageSize: clamp_value(helloMessage.maxMessageSize || defaultMaxMessageSize, minMaxMessageSize, maxMaxMessageSize),
+            maxChunkCount: clamp_value(helloMessage.maxChunkCount || defaultMaxChunkCount, minMaxChunkCount, maxMaxChunkCount)
+        });
 
         // istanbul ignore next
         if (doTraceHelloAck) {
             console.log(`received Hello \n${helloMessage.toString()}`);
+            console.log("Client accepts only message of size => ", this.maxMessageSize);
         }
 
         debugLog("Client accepts only message of size => ", this.maxMessageSize);

package/source/status_codes.ts

@@ -0,0 +1,109 @@
+import { StatusCodes } from "node-opcua-status-code";
+
+export const StatusCodes2 = {
+    /**
+     * The Server cannot process the request because it is too busy.
+     *
+     * It is up to the Server to determine when it needs to return this Message.
+     *
+     * A Server can control the how frequently a Client reconnects by waiting to return this error.
+     */
+    BadTcpServerTooBusy: StatusCodes.BadTcpServerTooBusy,
+    /**
+     * The type of the Message specified in the header invalid.
+     * Each Message starts with a 4-byte sequence of ASCII values that identifies the Message type.
+     * The Server returns this error if the Message type is not accepted.
+     * Some of the Message types are defined by the SecureChannel layer.
+     */
+    BadTcpMessageTypeInvalid: StatusCodes.BadTcpMessageTypeInvalid,
+    /**
+     * The SecureChannelId and/or TokenId are not currently in use.
+     * This error is reported by the SecureChannel layer.
+     */
+    BadTcpSecureChannelUnknown: StatusCodes.BadTcpSecureChannelUnknown,
+    /**
+     * The size of the Message specified in the header is too large.
+     * The Server returns this error if the Message size exceeds its maximum buffer size
+     * or the receive buffer size negotiated during the Hello/Acknowledge exchange.
+     */
+    BadTcpMessageTooLarge: StatusCodes.BadTcpMessageTooLarge,
+
+    /**
+     * A timeout occurred while accessing a resource.
+     * It is up to the Server to determine when a timeout occurs.
+     */
+    BadTimeout: StatusCodes.BadTimeout,
+    /**
+     * There are not enough resources to process the request.
+     * The Server returns this error when it runs out of memory or encounters similar resource problems.
+     * A Server can control the how frequently a Client reconnects by waiting to return this error.
+     */
+    BadTcpNotEnoughResources: StatusCodes.BadTcpNotEnoughResources,
+    /**
+     * An internal error occurred.
+     * This should only be returned if an unexpected configuration or programming error occurs.
+     */
+    BadTcpInternalError: StatusCodes.BadTcpInternalError,
+    /**
+     * The Server does not recognize the EndpointUrl specified.
+     */
+    BadTcpEndpointUrlInvalid: StatusCodes.BadTcpEndpointUrlInvalid,
+    /**
+     * The Message was rejected because it could not be verified.
+     */
+    BadSecurityChecksFailed: StatusCodes.BadSecurityChecksFailed,
+    /**
+     * The request could not be sent because of a network interruption.
+     */
+    BadRequestInterrupted: StatusCodes.BadRequestInterrupted,
+    /**
+     * Timeout occurred while processing the request.
+     */
+    BadRequestTimeout: StatusCodes.BadRequestTimeout,
+    /**
+     * The secure channel has been closed.
+     */
+    BadSecureChannelClosed: StatusCodes.BadSecureChannelClosed,
+    /**
+     * The SecurityToken has expired or is not recognized. BadSecureChannelTokenUnknown
+     */
+    BadSecureChannelTokenUnknown: StatusCodes.BadSecureChannelTokenUnknown,
+    /**
+     * The sender Certificate is not trusted by the receiver.
+     */
+    BadCertificateUntrusted: StatusCodes.BadCertificateUntrusted,
+    /**
+     * The sender Certificate has expired or is not yet valid.
+     */
+    BadCertificateTimeInvalid: StatusCodes.BadCertificateTimeInvalid,
+    /**
+     * The issuer for the sender Certificate has expired or is not yet valid.
+     */
+    BadCertificateIssuerTimeInvalid: StatusCodes.BadCertificateIssuerTimeInvalid,
+    /**
+     * The sender’s Certificate may not be used for establishing a secure channel.
+     */
+    BadCertificateUseNotAllowed: StatusCodes.BadCertificateUseNotAllowed,
+    /**
+     * The issuer Certificate may not be used as a Certificate Authority.
+     */
+    BadCertificateIssuerUseNotAllowed: StatusCodes.BadCertificateIssuerUseNotAllowed,
+    /**
+     * Could not verify the revocation status of the sender’s Certificate.
+     */
+    BadCertificateRevocationUnknown: StatusCodes.BadCertificateRevocationUnknown,
+    /**
+     * Could not verify the revocation status of the issuer Certificate.
+     */
+    BadCertificateIssuerRevocationUnknown: StatusCodes.BadCertificateIssuerRevocationUnknown,
+    /**
+     * The sender Certificate has been revoked by the issuer.
+     */
+    BadCertificateRevoked: StatusCodes.BadCertificateRevoked
+    /**The issuer Certificate has been revoked by its issuer.
+     */
+    // todo  BadIssuerCertificateRevoked: StatusCodes.BadIssuerCertificateRevoked,
+    /** The receiver Certificate thumbprint is not recognized by the receiver.
+     */
+    // todo .. BadCertificateUnknown: StatusCodes.BadCertificateUnknown
+};