opcjs-base 0.1.38-alpha → 0.1.39-alpha

package/dist/index.cjs

@@ -17870,9 +17870,13 @@ var SecureChannelMessageDecoder = class extends TransformStream {
   }
   logger = getLogger("secureChannel.SecureChannelMessageDecoder");
   /**
-   * Validates that `sequenceNumber` is strictly increasing from the last
+   * Validates that `sequenceNumber` is monotonically increasing from the last
    * seen remote sequence.  Allows exactly one UInt32 wrap-around per token.
-   * Returns false and logs an error if the number is a duplicate or out of order.
+   *
+   * Forward gaps are tolerated with a warning because some server implementations
+   * may consume sequence numbers internally (e.g. cancelled responses) without
+   * sending them on the wire.  Backward jumps and duplicates are rejected as
+   * errors since they indicate replay or reordering.
    */
   validateSequenceNumber(sequenceNumber, msgType, controller) {
     const last = this.context.lastRemoteSequenceNumber;
@@ -17881,14 +17885,18 @@ var SecureChannelMessageDecoder = class extends TransformStream {
       this.logger.debug(`[${msgType}] Sequence number initialized to ${sequenceNumber}`);
       return true;
     }
-    const isIncrement = sequenceNumber === last + 1;
     const isWrap = last >= SEQ_WRAP_THRESHOLD2 && sequenceNumber < SEQ_WRAP_MAX;
-    if (!isIncrement && !isWrap) {
-      this.logger.error(`[${msgType}] Invalid remote sequence number: expected ${last + 1}, got ${sequenceNumber}`);
-      controller.error(new Error(`Invalid remote sequence number: expected ${last + 1}, got ${sequenceNumber}`));
+    const isForward = sequenceNumber > last;
+    if (!isForward && !isWrap) {
+      this.logger.error(`[${msgType}] Invalid remote sequence number: expected > ${last}, got ${sequenceNumber}`);
+      controller.error(new Error(`Invalid remote sequence number: expected > ${last}, got ${sequenceNumber}`));
       return false;
     }
-    this.logger.debug(`[${msgType}] Sequence number advanced: ${last} \u2192 ${sequenceNumber}`);
+    if (sequenceNumber !== last + 1 && !isWrap) {
+      this.logger.warn(`[${msgType}] Remote sequence number gap: expected ${last + 1}, got ${sequenceNumber} (skipped ${sequenceNumber - last - 1})`);
+    } else {
+      this.logger.debug(`[${msgType}] Sequence number advanced: ${last} \u2192 ${sequenceNumber}`);
+    }
     this.context.lastRemoteSequenceNumber = sequenceNumber;
     return true;
   }