@blamejs/core 0.15.63 → 0.15.65

package/CHANGELOG.md

@@ -8,6 +8,10 @@ upgrading across more than a few patches at a time.
 
 ## v0.15.x
 
+- v0.15.65 (2026-06-29) — **The account-takeover kill-switch now actually locks the account: b.auth.lockout gains a lock() method and the kill-switch engages it through the operator's lockout instance.** b.auth.atoKillSwitch.trigger is the incident-response path for a compromised account: it destroys the user's sessions and then locks them out of new logins. The lockout step called b.auth.lockout.lock(), but no such method existed (and the kill-switch invoked it on the lockout module, which has no store), so the call threw and was swallowed — the kill-switch reported success while never locking the account, leaving an attacker who still held the credentials free to log straight back in. b.auth.lockout instances now expose lock(key, { durationMs?, untilMs?, reason? }) to force an account into lockout immediately, atomically, independent of the failure counter, and atoKillSwitch.trigger engages it through the lockout instance the operator passes as opts.lockout. When no lockout instance is supplied the step is skipped and the result's lockoutApplied is false (with an audit row), rather than silently claiming the account was locked. The release also closes a lock-clear race in lockout and refuses a self-disabling window. **Added:** *b.auth.lockout instances expose lock()* — lock(key, { durationMs?, untilMs?, reason? }) forces an account into lockout immediately — the operator action behind an ATO kill-switch or incident response — independent of the failure counter, defaulting to a long admin lock. A forced lock is released only by an explicit unlock(): recordSuccess() does not clear it, so a successful login by someone who still holds the compromised password cannot release a kill-switch lock. It uses the same atomic compare-and-set as the failure counter and throws if the lock cannot be committed (the caller must know it did not lock). **Security:** *Account-takeover kill-switch engages the lockout instead of silently doing nothing* — b.auth.atoKillSwitch.trigger invoked b.auth.lockout.lock(), which did not exist; the call threw and was caught by a best-effort guard, so the kill-switch destroyed sessions but never locked the account — an attacker still holding valid credentials could immediately re-authenticate. b.auth.lockout instances now provide lock() (an atomic, compare-and-set forced lockout), and the kill-switch calls it on the lockout instance supplied as opts.lockout. Without a lockout instance the lockout step cannot run; the result's lockoutApplied is false and an audit row records that the account was not locked, so an operator is not misled into believing a compromised account was secured. opts.lockout is now the lockout instance (or false to skip), not a boolean toggle. · *lockout clears the failure counter atomically and refuses a zero window* — b.auth.lockout.recordSuccess and unlock cleared state with a read-then-delete, which could erase a lock a concurrent recordFailure had just engaged; they now clear under the same compare-and-set the failure path uses. create() also refuses windowMs: 0, which previously disabled lockout entirely (every failure decayed immediately and the zero-TTL state never persisted).
+
+- v0.15.64 (2026-06-29) — **Token and HTTP-message-signature verifiers now reject a non-finite clock-skew or tolerance value instead of letting it disable the expiry, not-before, and future-dating checks.** Several verifiers read an operator-supplied clock-skew or tolerance value with a bare type check that accepted Infinity and NaN. Because the resulting comparison (for example exp + skew < now, or age > tolerance) is always false when the skew or tolerance is Infinity, a misconfigured non-finite value silently disabled the time-window check — so an expired token, a not-yet-valid token, a future-dated signature, or a replayed (too-old) signature would verify. The affected paths are the shared external-JWT verifier b.auth.jwt.verifyExternal (which the JAR / signed-request-object path also uses), the SD-JWT-VC credential verifier, the OAuth ID-token and client-attestation verifiers, and the HTTP message signature verifier's freshness and clock-skew windows. Each now requires a present skew or tolerance to be a non-negative finite number: the throw-based JWT, SD-JWT-VC, and OAuth verifiers reject a malformed value, and the verdict-returning HTTP-message-signature verifier falls back to its safe default rather than disabling the window. A build check now fails if any verifier reads one of these options without a finiteness guard. **Added:** *verifyExternal and jar.parse accept an `now` clock override* — b.auth.jwt.verifyExternal and b.auth.jar.parse now accept an optional `now` (a finite epoch-milliseconds value) to evaluate a token's exp / nbf / iat as of a specific instant instead of the wall clock — consistent with b.auth.sdJwtVc.verify. A negative clockSkewMs is no longer accepted as a way to shift the evaluation time; use `now`. **Security:** *Non-finite clock-skew / tolerance no longer disables a verifier's time-window check* — b.auth.jwt.verifyExternal, b.auth.sdJwtVc.verify, b.auth.oauth (ID-token and client-attestation verification), and the HTTP message signature verifier read their clock-skew, max-clock-skew, max-PoP-age, and tolerance options with a bare `typeof === "number"` check that let Infinity and NaN through. With such a value the expiry / not-before / future-dating comparison is always false, so the check was silently skipped and an expired or not-yet-valid token — or a future-dated or replayed signature — would verify. The JWT, SD-JWT-VC, and OAuth verifiers now reject a non-finite or negative skew (the option is operator configuration, not attacker-controlled), and the HTTP message signature verifier falls back to its default tolerance and skew. The CWT, DPoP, SAML, and OpenID Federation verifiers already enforced this and are unchanged. **Detectors:** *Verifier clock-skew / tolerance options must be finite-guarded* — A build check fails if a verifier reads a clock-skew or tolerance option with a bare `typeof === "number"` without a finiteness guard (a non-negative-finite check, an inline isFinite fallback, or a create-time non-negative-finite schema), preventing a future verifier from reintroducing the disable-the-window class.
+
 - v0.15.63 (2026-06-29) — **OID4VCI now enforces single-use of a pre-authorized code and of a single-use access token under concurrency, so two simultaneous requests can no longer mint two credentials from one.** On the OID4VCI credential issuer, two single-use values were consumed by a delete whose result was ignored, so concurrent requests could each act on the same value. exchangePreAuthorizedCode read the pre-authorized code's entry, validated the transaction code, then deleted the code and minted an access token without checking that its own call had removed the entry — two simultaneous /token requests with the same code each saw the entry, each deleted it, and each minted a distinct access token, issuing two access tokens (and ultimately two credentials) from a code OID4VCI requires to be single-use. issueCredential had the same shape: with single-use access tokens (the default), it minted the credential first and deleted the access token afterward as best-effort cleanup, so two concurrent requests bearing the same token both read it and the same not-yet-rotated c_nonce, both proofs verified, and both minted a credential. Both paths now claim the value with an atomic delete and proceed only when that delete succeeded; the losing request is refused. The transaction-code and proof checks still run first, so a bad transaction code or proof does not consume the value (a wallet can retry). **Security:** *OID4VCI pre-authorized code and access token are single-use under concurrency* — exchangePreAuthorizedCode and issueCredential consumed their single-use value (the pre-authorized code, and the single-use access token) with a delete whose return was discarded, and in issueCredential's case after the credential was already minted. Two concurrent requests carrying the same value therefore each succeeded — minting two access tokens from one pre-authorized code, or two credentials from one single-use access token — defeating the single-use guarantee OID4VCI §3.5 requires (an authorization intended for one credential could yield two). Both paths now delete the value atomically and issue only if that delete removed it, refusing the request that lost the race; the transaction-code and proof verifications run before the claim, so an invalid attempt does not burn the value. If the operator's credential issuer throws after the access token has been claimed (a transient signer outage), the token is restored so the wallet can retry rather than being permanently consumed without a credential.
 
 - v0.15.62 (2026-06-29) — **ARC evaluation now reads each hop's instance with the same strict parser the signature checks use, so a crafted ARC-Authentication-Results header can no longer forge the upstream auth-results surfaced to downstream policy.** b.mail.arc.evaluate returns finalAr — the most recent hop's ARC-Authentication-Results, the receiver's view of the upstream authentication results — which operators may key downstream decisions on. The instance tag (i=) on each ARC header was parsed by three different regexes: the indexing pass that drives the AMS/AS signature checks required i= with no surrounding space and at most three digits, while the finalAr extraction and the AMS header-retention test accepted a looser form (a space around =, unbounded digits). When a sealer signs without covering ARC-Authentication-Results in its AMS h= (permitted by RFC 8617 and supported by the verifier), an attacker holding no key could append a second ARC-Authentication-Results written so the strict pass ignored it while the loose pass consumed it — forging finalAr on a chain that still verified as pass. All ARC instance reads now route through one strict parser, so the evaluation surfaces the same hop the signatures validated. The release also repairs the b.mail.arc.sign excludeAarFromAms option (it was read but rejected by option validation, so the documented opt-out was unreachable) and routes the ARC-Seal signature's b= stripping through the shared tag-aware helper. **Fixed:** *ARC finalAr is read from the strictly-indexed hop, not a looser rescan* — b.mail.arc.evaluate extracted finalAr (and validated the per-hop AMS header retention) with a regex that accepted ARC instance tags the signature-indexing pass rejected — a space around i= or more than three digits. A sealer that omits ARC-Authentication-Results from its AMS h= leaves the AAR outside signature coverage; an attacker could then inject a second ARC-Authentication-Results whose instance the strict crypto pass skipped but the finalAr pass accepted, presenting attacker-chosen upstream auth-results on a chain that still reported pass. Every ARC instance read now goes through a single strict parser, so finalAr is always the AAR the chain's signatures actually covered. · *b.mail.arc.sign accepts excludeAarFromAms again* — The excludeAarFromAms option was read when building the AMS h= list but was missing from the function's option allow-list, so passing it raised an unknown-option error — the documented opt-out could not be used. It is now accepted. · *ARC-Seal b= stripping uses the shared tag-aware helper* — The ARC-Seal verification stripped the signature's b= value with a regex that could mis-zero a value containing b= inside another tag; it now uses the same tag-aware stripper as DKIM, so canonicalization matches the signer in every case. **Detectors:** *ARC instance parsing must use the shared strict reader* — A check fails the build if any ARC instance (i=) parsing regex is added outside the single shared reader, preventing a future divergence between the signature-indexing pass and the finalAr / header-retention passes.

package/lib/auth/ato-kill-switch.js

@@ -10,17 +10,22 @@
  * cleanup path once the trigger fires:
  *
  *   1. destroy every session for the user across the cluster
- *   2. lock the user out of new logins (b.auth.lockout)
+ *   2. lock the user out of new logins via the supplied lockout instance
  *   3. emit an audit row with reason / actor for downstream forensics
  *
  *   await b.auth.atoKillSwitch.trigger({
  *     userId:    "u_42",
  *     reason:    "fraud-signal: 14 failed MFA from new geo",
  *     actor:     { id: req.user && req.user.id, role: req.user && req.user.role },
- *     lockout:   true,                  // default true
+ *     lockout:   appLockout,            // a b.auth.lockout instance (or false to skip)
  *     accessLock: "locked",             // optional — flip the global access-lock mode
  *   });
  *
+ * The lockout step needs the operator's configured b.auth.lockout instance
+ * (there is no module-level lockout store); pass it as `opts.lockout`. When it
+ * is omitted, the lockout step is skipped and the returned `lockoutApplied` is
+ * false (with an audit row) — the account is NOT locked.
+ *
  * Returns `{ sessionsDestroyed, lockoutApplied, accessLockMode }`.
  */
 
@@ -29,7 +34,6 @@ var validateOpts = require("../validate-opts");
 var { defineClass } = require("../framework-error");
 
 var session = lazyRequire(function () { return require("../session"); });
-var lockout = lazyRequire(function () { return require("./lockout"); });
 var accessLock = lazyRequire(function () { return require("./access-lock"); });
 var audit = lazyRequire(function () { return require("../audit"); });
 
@@ -43,8 +47,15 @@ async function trigger(opts) {
 
   validateOpts.requireNonEmptyString(opts.userId, "userId", AtoKillSwitchError, "auth-ato-kill-switch/missing-user-id");
   validateOpts.requireNonEmptyString(opts.reason, "reason", AtoKillSwitchError, "auth-ato-kill-switch/missing-reason");
-  var doLockout       = opts.lockout !== false;
-  var accessLockMode  = typeof opts.accessLock === "string" ? opts.accessLock : null;
+  // opts.lockout is the operator's configured b.auth.lockout INSTANCE to lock
+  // the account in (or false to skip the lockout step). The kill-switch needs
+  // that instance's store to engage the lock — there is no module-level lockout
+  // singleton — so without an instance the lockout step cannot run and is
+  // reported (rather than silently claiming success while never locking).
+  var skipLockout    = opts.lockout === false;
+  var lockoutInst    = (opts.lockout && typeof opts.lockout === "object" &&
+                        typeof opts.lockout.lock === "function") ? opts.lockout : null;
+  var accessLockMode = typeof opts.accessLock === "string" ? opts.accessLock : null;
 
   var sessionsDestroyed = 0;
   try {
@@ -63,13 +74,37 @@ async function trigger(opts) {
   }
 
   var lockoutApplied = false;
-  if (doLockout) {
-    try {
-      await lockout().lock(opts.userId, {
-        reason: "ato-kill-switch:" + opts.reason,
+  if (!skipLockout) {
+    if (lockoutInst) {
+      try {
+        await lockoutInst.lock(opts.userId, {
+          reason: "ato-kill-switch:" + opts.reason,
+        });
+        lockoutApplied = true;
+      } catch (e) {
+        // The lockout step failed (e.g. a cache outage) AFTER sessions were
+        // destroyed. Don't fail the whole kill-switch — but surface it so the
+        // operator knows the account was NOT locked, rather than swallowing.
+        audit().safeEmit({
+          action: "auth.ato_kill_switch.partial",
+          outcome: "failure",
+          metadata: { userId: opts.userId, step: "lockout", reason: e && e.message },
+        });
+      }
+    } else {
+      // No lockout instance supplied: the kill-switch cannot lock the account.
+      // Report it (the result's lockoutApplied stays false) so the operator
+      // doesn't believe the user was locked out of new logins.
+      audit().safeEmit({
+        action: "auth.ato_kill_switch.partial",
+        outcome: "failure",
+        metadata: {
+          userId: opts.userId,
+          step:   "lockout",
+          reason: "no lockout instance supplied (pass opts.lockout = b.auth.lockout.create({ cache, namespace }))",
+        },
       });
-      lockoutApplied = true;
-    } catch (_e) { /* lockout is best-effort; sessions already destroyed */ }
+    }
   }
 
   var modeApplied = null;

package/lib/auth/jar.js

@@ -139,7 +139,7 @@ async function parse(jar, opts) {
   }
   validateOpts.requireObject(opts, "jar.parse", AuthJarError);
   validateOpts(opts, [
-    "clientId", "audience", "algorithms", "jwks", "jwksUri", "keyResolver", "clockSkewMs",
+    "clientId", "audience", "algorithms", "jwks", "jwksUri", "keyResolver", "clockSkewMs", "now",
   ], "jar.parse");
   validateOpts.requireNonEmptyString(opts.clientId, "jar.parse: clientId", AuthJarError, "auth-jar/bad-client-id");
   validateOpts.requireNonEmptyString(opts.audience, "jar.parse: audience", AuthJarError, "auth-jar/bad-audience");
@@ -156,6 +156,7 @@ async function parse(jar, opts) {
     issuer:      opts.clientId,
     audience:    opts.audience,
     clockSkewMs: opts.clockSkewMs,
+    now:         opts.now,
   });
   // RFC 9101 §10.8 — the request object MUST be explicitly typed so a JWT
   // minted for another purpose (id_token / access-token / logout-token)

package/lib/auth/jwt-external.js

@@ -56,6 +56,7 @@ var lazyRequire = require("../lazy-require");
 var validateOpts = require("../validate-opts");
 var C = require("../constants");
 var bCrypto = require("../crypto");
+var numericBounds = require("../numeric-bounds");
 var { AuthError } = require("../framework-error");
 
 var httpClient = lazyRequire(function () { return require("../http-client"); });
@@ -421,7 +422,7 @@ async function verifyExternal(token, opts) {
   opts = opts || {};
   validateOpts(opts, [
     "algorithms", "jwks", "jwksUri", "jwksCacheMs", "keyResolver",
-    "audience", "issuer", "subject", "clockSkewMs",
+    "audience", "issuer", "subject", "clockSkewMs", "now",
     // v0.9.4 — opt-out for the kid-less-token JWKS-of-one refusal
     // (default refuses; non-conforming IdPs that emit kid-less tokens
     // set this true).
@@ -564,9 +565,19 @@ async function verifyExternal(token, opts) {
       "signature verification failed");
   }
 
-  // Claim validation.
+  // Claim validation. A present clockSkewMs must be a non-negative finite
+  // integer: a bare `typeof === "number"` lets Infinity / NaN through, and
+  // `exp + Infinity < now` (or NaN) is always false — silently disabling the
+  // expiry/nbf/iat gates so an expired or not-yet-valid token verifies. Reject
+  // a malformed skew loudly (it is operator config, not token-controlled).
+  numericBounds.requireNonNegativeFiniteIntIfPresent(opts.clockSkewMs,
+    "verifyExternal: opts.clockSkewMs", AuthError, "auth-jwt-external/bad-clock-skew");
   var clockSkewMs = typeof opts.clockSkewMs === "number" ? opts.clockSkewMs : DEFAULT_CLOCK_SKEW_MS;
-  var nowSec   = Math.floor(Date.now() / C.TIME.seconds(1));
+  // opts.now (finite ms) overrides the wall clock for the time checks —
+  // consistent with sdJwtVc.verify; lets a caller evaluate the token as of a
+  // specific instant without abusing a (now-rejected) negative skew.
+  var nowMs    = (typeof opts.now === "number" && isFinite(opts.now)) ? opts.now : Date.now();
+  var nowSec   = Math.floor(nowMs / C.TIME.seconds(1));
   var skewSec  = Math.floor(clockSkewMs / C.TIME.seconds(1));
 
   if (typeof payload.exp !== "number") {

package/lib/auth/lockout.js

@@ -90,6 +90,11 @@ var observability = lazyRequire(function () { return require("../observability")
 
 var _err = LockoutError.factory;
 
+// Default duration for an operator-forced lock() (ATO kill-switch / incident
+// response) when neither untilMs nor durationMs is supplied — long enough to
+// require an explicit admin unlock() during an active incident.
+var DEFAULT_ADMIN_LOCK_MS = C.TIME.hours(24);
+
 var DEFAULTS = Object.freeze({
   maxAttempts:     5,
   windowMs:        C.TIME.minutes(15),
@@ -173,7 +178,11 @@ function create(opts) {
   _requirePositiveInt("maxAttempts", maxAttempts);
 
   var windowMs = opts.windowMs !== undefined ? opts.windowMs : DEFAULTS.windowMs;
-  _requireNonNegFinite("windowMs", windowMs);
+  // windowMs must be POSITIVE: a 0 window makes every failure "decay" on the
+  // next request (the `now - lastFailureAt > windowMs` reset fires for any
+  // elapsed time) so the counter never reaches maxAttempts, AND the cache TTL
+  // of 0 makes the state non-persistent — together silently disabling lockout.
+  _requirePositiveInt("windowMs", windowMs);
 
   var lockoutDurations = opts.lockoutDurations !== undefined
                             ? opts.lockoutDurations : DEFAULTS.lockoutDurations;
@@ -242,6 +251,49 @@ function create(opts) {
     }
   }
 
+  // Atomically clear the lockout state under a compare-and-set, so a clear
+  // (recordSuccess / unlock) cannot race a concurrent recordFailure that just
+  // engaged a lock — a read-then-del would erase that fresh lock. onState is
+  // invoked with the pre-clear state inside the CAS mutator (it may re-run on a
+  // retry; the last invocation reflects the committed state) to capture audit
+  // detail. On a backend that can't do an atomic update at runtime it falls
+  // back to read-then-del (a lost clear leaves a lock in place — fail-safe).
+  // preserveIf(state) → true to KEEP the state (abort the clear) rather than
+  // delete it. recordSuccess passes a predicate that preserves an active forced
+  // (admin / ATO kill-switch) lock, so a successful login by someone who still
+  // holds the compromised password cannot clear it — only an explicit unlock()
+  // releases a forced lock.
+  async function _atomicClear(key, onState, preserveIf) {
+    try {
+      await cache.update(_scopedKey(key), function (state) {
+        state = state || null;
+        onState(state);
+        if (!state) return { abort: true };
+        if (preserveIf && preserveIf(state)) return { abort: true };
+        return { delete: true };
+      }, { ttlMs: windowMs });
+    } catch (e) {
+      if (e && e.code === "UNSUPPORTED") {
+        var st = await _readState(key);
+        onState(st);
+        if (st && !(preserveIf && preserveIf(st))) await _deleteState(key);
+        return;
+      }
+      _signalCacheError("update");
+      // Best-effort fallback so a transient cache error doesn't leave the
+      // counter un-cleared on the happy path.
+      var st2 = await _readState(key);
+      onState(st2);
+      if (st2 && !(preserveIf && preserveIf(st2))) await _deleteState(key);
+    }
+  }
+
+  // An active forced/admin lock (set by lock()) must survive recordSuccess.
+  function _isActiveForcedLock(state) {
+    return !!(state && state.forced === true &&
+              typeof state.lockedUntil === "number" && state.lockedUntil > clock());
+  }
+
   function _verdictFromState(state, now) {
     if (!state) return { locked: false, attempts: 0 };
     if (state.lockedUntil && state.lockedUntil > now) {
@@ -380,13 +432,19 @@ function create(opts) {
   async function recordSuccess(key, callOpts) {
     _requireKey(key);
     callOpts = callOpts || {};
-    var state = await _readState(key);
-    var hadCounter = !!(state && (state.attempts > 0 || state.lockedUntil));
-    if (state) await _deleteState(key);
+    var hadCounter = false;
+    var clearedAttempts = 0;
+    // Preserve an active forced (admin / ATO) lock: a successful credential
+    // verification must not release a kill-switch lock — the password may still
+    // be the compromised one. A forced lock yields only to unlock().
+    await _atomicClear(key, function (state) {
+      hadCounter = !!(state && (state.attempts > 0 || state.lockedUntil));
+      clearedAttempts = (state && state.attempts) || 0;
+    }, _isActiveForcedLock);
     _emitObs("auth.lockout.success", { namespace: namespace });
     if (auditSuccess) {
       _emitAudit("auth.lockout.success", key, "success",
-        { attemptsCleared: (state && state.attempts) || 0,
+        { attemptsCleared: clearedAttempts,
           hadCounter:      hadCounter },
         callOpts.req);
     }
@@ -401,26 +459,93 @@ function create(opts) {
   async function unlock(key, callOpts) {
     _requireKey(key);
     callOpts = callOpts || {};
-    var state = await _readState(key);
     var now = clock();
-    var hadLock = !!(state && (
-      (state.lockedUntil && state.lockedUntil > now) ||
-      (state.attempts || 0) > 0
-    ));
-    if (state) await _deleteState(key);
+    var hadLock = false;
+    var prior = { attempts: 0, lockedUntil: null, lockNumber: 0 };
+    await _atomicClear(key, function (state) {
+      hadLock = !!(state && (
+        (state.lockedUntil && state.lockedUntil > now) ||
+        (state.attempts || 0) > 0
+      ));
+      prior = {
+        attempts:    (state && state.attempts) || 0,
+        lockedUntil: (state && state.lockedUntil) || null,
+        lockNumber:  (state && state.lockNumber) || 0,
+      };
+    });
     _emitObs("auth.lockout.unlock", { namespace: namespace });
     if (auditUnlock) {
       _emitAudit("auth.lockout.unlock", key, "success",
         { hadLock:           hadLock,
-          priorAttempts:    (state && state.attempts) || 0,
-          priorLockedUntil: (state && state.lockedUntil) || null,
-          priorLockNumber:  (state && state.lockNumber) || 0,
+          priorAttempts:    prior.attempts,
+          priorLockedUntil: prior.lockedUntil,
+          priorLockNumber:  prior.lockNumber,
           reason:           callOpts.reason || null },
         callOpts.req);
     }
     return hadLock;
   }
 
+  // Force an account into lockout immediately — the operator action behind an
+  // ATO kill-switch / incident response, independent of the failure counter.
+  // Sets lockedUntil to `untilMs` (absolute) or now+`durationMs`, defaulting to
+  // a long admin lock; bumps lockNumber so a subsequent failure ladders from
+  // here. Uses the same compare-and-set as recordFailure (atomic, retried). An
+  // admin lock that cannot be committed THROWS (the caller — e.g. the kill-
+  // switch — must know it did not lock), rather than the hot-path fail-open.
+  async function lock(key, callOpts) {
+    _requireKey(key);
+    callOpts = callOpts || {};
+    var now = clock();
+    var lockedUntil;
+    if (typeof callOpts.untilMs === "number" && isFinite(callOpts.untilMs)) {
+      lockedUntil = callOpts.untilMs;
+    } else if (typeof callOpts.durationMs === "number" && isFinite(callOpts.durationMs) && callOpts.durationMs > 0) {
+      lockedUntil = now + callOpts.durationMs;
+    } else {
+      lockedUntil = now + DEFAULT_ADMIN_LOCK_MS;
+    }
+    if (lockedUntil <= now) {
+      throw _err("BAD_OPT", "lock: resolved lockedUntil is not in the future " +
+        "(untilMs/durationMs) — use unlock() to clear a lock");
+    }
+    var ttl = lockedUntil - now + windowMs;
+    var lockNumber = 0;
+    try {
+      await cache.update(_scopedKey(key), function (state) {
+        lockNumber = ((state && state.lockNumber) || 0) + 1;
+        return {
+          value: {
+            attempts:       0,
+            lockNumber:     lockNumber,
+            firstFailureAt: (state && state.firstFailureAt) || now,
+            lastFailureAt:  now,
+            lockedUntil:    lockedUntil,
+            // Marks this as an operator-forced lock so recordSuccess won't clear
+            // it — it is released only by an explicit unlock().
+            forced:         true,
+          },
+          ttlMs: ttl,
+        };
+      }, { ttlMs: ttl });
+    } catch (e) {
+      if (e && e.code === "UNSUPPORTED") {
+        throw _err("CACHE_NO_ATOMIC_UPDATE",
+          "auth.lockout: the cache backend does not support atomic update() — " +
+          "lock() cannot enforce the lockout across nodes on a get/set-only backend.");
+      }
+      throw e;
+    }
+    _emitObs("auth.lockout.engaged", { namespace: namespace, lockNumber: String(lockNumber) });
+    if (auditEngaged) {
+      _emitAudit("auth.lockout.engaged", key, "denied",
+        { lockNumber: lockNumber, lockedUntil: lockedUntil, durationMs: lockedUntil - now,
+          forced: true, reason: callOpts.reason || null },
+        callOpts.req);
+    }
+    return { locked: true, lockedUntil: lockedUntil, lockNumber: lockNumber };
+  }
+
   async function attempts(key) {
     _requireKey(key);
     var state = await _readState(key);
@@ -437,6 +562,7 @@ function create(opts) {
     recordFailure: recordFailure,
     recordSuccess: recordSuccess,
     check:         check,
+    lock:          lock,
     unlock:        unlock,
     attempts:      attempts,
     close:         close,

package/lib/auth/oauth.js

@@ -107,6 +107,7 @@
 var nodeCrypto = require("node:crypto");
 var cache = require("../cache");
 var C = require("../constants");
+var numericBounds = require("../numeric-bounds");
 var safeAsync = require("../safe-async");
 var bCrypto = require("../crypto");
 var { generateBytes, timingSafeEqual: cryptoTimingSafeEqual } = bCrypto;
@@ -886,6 +887,11 @@ async function verifyClientAttestation(attestationJwt, popJwt, vopts) {
       "client-attestation: missing 'cnf.jwk' confirmation key (RFC 7800)");
   }
   var nowSec  = Math.floor(Date.now() / C.TIME.seconds(1));
+  // A present skew/maxAge must be a non-negative finite integer; a bare typeof
+  // check lets Infinity/NaN through, and `exp + Infinity < now` (or NaN) is
+  // always false — disabling the attestation/PoP expiry gates.
+  numericBounds.requireNonNegativeFiniteIntIfPresent(vopts.clockSkewSec,
+    "verifyClientAttestation: opts.clockSkewSec", OAuthError, "auth-oauth/bad-clock-skew");
   var skewSec = typeof vopts.clockSkewSec === "number" ? vopts.clockSkewSec : (C.TIME.minutes(1) / C.TIME.seconds(1));
   if (typeof ap.exp !== "number" || ap.exp + skewSec < nowSec) {
     throw new OAuthError("auth-oauth/attestation-expired",
@@ -918,6 +924,8 @@ async function verifyClientAttestation(attestationJwt, popJwt, vopts) {
   if (typeof pp.iat !== "number") {
     throw new OAuthError("auth-oauth/attestation-pop-no-iat", "client-attestation-pop: missing 'iat'");
   }
+  numericBounds.requireNonNegativeFiniteIntIfPresent(vopts.maxPopAgeSec,
+    "verifyClientAttestation: opts.maxPopAgeSec", OAuthError, "auth-oauth/bad-pop-max-age");
   var maxAge = typeof vopts.maxPopAgeSec === "number" ? vopts.maxPopAgeSec : DEFAULT_POP_MAX_AGE_SEC;
   if (pp.iat - skewSec > nowSec) {
     throw new OAuthError("auth-oauth/attestation-pop-iat-future", "client-attestation-pop: iat in the future");
@@ -992,6 +1000,12 @@ function create(opts) {
       "requires PKCE for all clients. Remove the opt or upgrade the IdP.");
   }
   var pkce = true;
+  // A present clockSkewMs must be a non-negative finite integer; Infinity/NaN
+  // would disable verifyIdToken's exp gate (`exp + Infinity < now` is always
+  // false → an expired ID token verifies). Reject a malformed skew at config
+  // time so the operator catches it.
+  numericBounds.requireNonNegativeFiniteIntIfPresent(opts.clockSkewMs,
+    "oauth.create: opts.clockSkewMs", OAuthError, "auth-oauth/bad-clock-skew");
   var clockSkewMs  = typeof opts.clockSkewMs === "number" ? opts.clockSkewMs : DEFAULT_CLOCK_SKEW_MS;
   var discoveryCacheMs = typeof opts.discoveryCacheMs === "number"
                            ? opts.discoveryCacheMs : DEFAULT_DISCOVERY_CACHE_MS;

package/lib/auth/sd-jwt-vc.js

@@ -62,6 +62,7 @@
 
 var nodeCrypto = require("node:crypto");
 var bCrypto = require("../crypto");
+var numericBounds = require("../numeric-bounds");
 var safeBuffer = require("../safe-buffer");
 var safeJson = require("../safe-json");
 var validateOpts = require("../validate-opts");
@@ -523,6 +524,12 @@ async function verify(presentation, opts) {
   // 2. Validate iss / iat / exp / vct
   var nowSec = (typeof opts.now === "number" && isFinite(opts.now))
     ? Math.floor(opts.now / 1000) : Math.floor(Date.now() / 1000);                  // ms→s conversion factor
+  // A present maxClockSkewSec must be a non-negative finite integer; a bare
+  // typeof check lets Infinity/NaN through, and `exp < nowSec - Infinity` (or
+  // NaN) is always false — silently accepting an expired credential. Reject a
+  // malformed skew (operator config).
+  numericBounds.requireNonNegativeFiniteIntIfPresent(opts.maxClockSkewSec,
+    "verify: opts.maxClockSkewSec", AuthError, "auth-sd-jwt-vc/bad-clock-skew");
   var skew = (typeof opts.maxClockSkewSec === "number") ? opts.maxClockSkewSec : 60; // allow:raw-time-literal — default 60s clock-skew tolerance
   if (typeof jwtParsed.payload.iat === "number" && jwtParsed.payload.iat > nowSec + skew) {
     throw new AuthError("auth-sd-jwt-vc/iat-future",

package/lib/http-message-signature.js

@@ -561,8 +561,16 @@ function verify(msg, opts) {
   // itself still has to verify — only the floor is waived).
   validateOpts.optionalNonEmptyStringArray(opts.requiredComponents,
     "httpSig.verify: requiredComponents", HttpSigError, "BAD_OPT");
-  var toleranceMs = typeof opts.toleranceMs === "number" ? opts.toleranceMs : DEFAULT_TOLERANCE_MS;
-  var clockSkewMs = typeof opts.clockSkewMs === "number" ? opts.clockSkewMs : DEFAULT_CLOCK_SKEW_MS;
+  // A present toleranceMs / clockSkewMs must be a non-negative finite number;
+  // a bare typeof check lets Infinity/NaN through, and `ageMs > Infinity` (the
+  // expiry gate) or `-ageMs > Infinity` (the future-dating gate) is always
+  // false — silently accepting a stale (replayed) or future-dated signature.
+  // verify() returns verdicts rather than throwing, so a malformed value falls
+  // back to the safe default instead of disabling the window.
+  var toleranceMs = (typeof opts.toleranceMs === "number" && isFinite(opts.toleranceMs) && opts.toleranceMs >= 0)
+    ? opts.toleranceMs : DEFAULT_TOLERANCE_MS;
+  var clockSkewMs = (typeof opts.clockSkewMs === "number" && isFinite(opts.clockSkewMs) && opts.clockSkewMs >= 0)
+    ? opts.clockSkewMs : DEFAULT_CLOCK_SKEW_MS;
   var nowMs = opts.now ? opts.now() : Date.now();
 
   var sigInput = _resolveHeader(m.headers, "signature-input");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blamejs/core",
-  "version": "0.15.63",
+  "version": "0.15.65",
   "description": "The Node framework that owns its stack.",
   "license": "Apache-2.0",
   "author": "blamejs contributors",

package/sbom.cdx.json

@@ -2,10 +2,10 @@
   "$schema": "http://cyclonedx.org/schema/bom-1.5.schema.json",
   "bomFormat": "CycloneDX",
   "specVersion": "1.5",
-  "serialNumber": "urn:uuid:c128c8b9-913b-4784-824b-3955b67b7bda",
+  "serialNumber": "urn:uuid:107d84f4-f93e-4798-846a-d948f93d854d",
   "version": 1,
   "metadata": {
-    "timestamp": "2026-06-29T20:57:16.088Z",
+    "timestamp": "2026-06-29T23:25:12.730Z",
     "lifecycles": [
       {
         "phase": "build"
@@ -19,14 +19,14 @@
       }
     ],
     "component": {
-      "bom-ref": "@blamejs/core@0.15.63",
+      "bom-ref": "@blamejs/core@0.15.65",
       "type": "application",
       "name": "blamejs",
-      "version": "0.15.63",
+      "version": "0.15.65",
       "scope": "required",
       "author": "blamejs contributors",
       "description": "The Node framework that owns its stack.",
-      "purl": "pkg:npm/%40blamejs/core@0.15.63",
+      "purl": "pkg:npm/%40blamejs/core@0.15.65",
       "properties": [],
       "externalReferences": [
         {
@@ -54,7 +54,7 @@
   "components": [],
   "dependencies": [
     {
-      "ref": "@blamejs/core@0.15.63",
+      "ref": "@blamejs/core@0.15.65",
       "dependsOn": []
     }
   ]