@blamejs/core 0.17.2 → 0.17.4

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/CHANGELOG.md CHANGED
@@ -8,6 +8,10 @@ upgrading across more than a few patches at a time.
8
8
 
9
9
  ## v0.17.x
10
10
 
11
+ - v0.17.4 (2026-07-17) — **Three fail-open holes closed: a raw cross-border write could evade the data-residency gate by wedging a comment or eliding whitespace, safe decompression returned undecompressed bytes for an unknown algorithm name, and the shared anti-replay helper admitted a replay when its store signalled a duplicate with a non-boolean value.** A raw SQL write submitted through b.db.runSql / b.db.prepare().run() could slip past the cross-border data-residency gate: the gate's write-detection matched tokens on whitespace boundaries, but SQL lets tokens abut with no whitespace when a comment or a quoted identifier sits between them, so INSERT/**/INTO, an INSERT whose quoted table name abuts INTO, and UPDATE/**/table all executed while the gate never engaged -- landing or moving a row across a residency boundary under a regulated posture. b.safeDecompress resolved its algorithm allowlist with a truthiness lookup, so an algorithm name that collides with an inherited object member (constructor, toString) reached the dispatch and returned the input undecompressed instead of refusing the unknown algorithm -- a fail-open for any caller that maps a client Content-Encoding onto the algorithm. And b.nonceStore.enforceReplay -- the anti-replay helper JWT and DPoP verification use -- detected a replay only when its store returned a literal false, so a store signalling a duplicate with another falsy value (a Redis SETNX null, a SQL INSERT rowcount of 0) let the replayed token through. **Added:** *b.safeSql.normalizeForScan* — A parse-only SQL normalizer that produces a copy whose token boundaries are real whitespace, so a scanner built with whitespace-anchored patterns cannot be evaded by a comment or a quoted-identifier boundary that lets two tokens abut. It collapses line and block comments to a single space and inserts a separating space where a quoted string or identifier follows a word character, in the same quote- and comment-aware single pass as the existing b.safeSql placeholder scanners (a comment marker inside a string literal is preserved, never collapsed). The executed SQL is never derived from this copy -- it only feeds a scanner. **Security:** *Data-residency gate catches a raw write that hides behind a comment or elided whitespace* — The cross-border data-residency gate that guards raw SQL writes (b.db.runSql, b.db.prepare().run()) detected the write and extracted its target table with regexes that assume tokens are separated by whitespace. SQL does not require whitespace between tokens when a comment or a quoted-identifier boundary separates them, so three forms that every engine executes as ordinary writes slipped the detection entirely and the gate never engaged: an INSERT whose double-quoted table name abuts INTO with no space, an INSERT with a block comment wedged between INSERT and INTO, and the same for UPDATE and its table. Under a regulated residency posture a row could therefore be inserted or moved across a residency boundary with no tag check (CWE-863; a GDPR Chapter V transfer-restriction bypass). The gate now normalizes a parse-only copy of the statement through the new b.safeSql.normalizeForScan before matching -- collapsing comments to a space and inserting a boundary where a quoted token abuts a word character -- so every write is detected and gated regardless of how its tokens are spaced. The executed SQL is unchanged. · *Safe decompression refuses an unknown algorithm instead of returning the input undecompressed* — b.safeDecompress resolved opts.algorithm against its allowlist of supported codecs (gzip, deflate, deflate-raw, brotli) with a truthiness lookup on a plain object. An algorithm name that names an inherited object member -- constructor, toString, valueOf -- read back a truthy inherited value, slipped the unsupported-algorithm refusal, and reached the dispatch, where invoking the inherited member returned the raw input buffer. The primitive therefore silently returned undecompressed bytes with no error, contrary to its contract that any algorithm outside the allowlist is refused -- a fail-open for a consumer that maps a client-supplied Content-Encoding token onto the algorithm, which also sidesteps the decompression-bomb size and ratio caps. The allowlist is now resolved with an own-property check, so any unsupported name -- however it collides with a built-in member -- is refused. · *Shared anti-replay helper fails closed on a non-boolean store result* — b.nonceStore.enforceReplay is the anti-replay helper that b.auth.jwt and b.auth.dpop route their replay defense through; it calls the store's checkAndInsert and treated the result as a replay only when it was the literal false. The recommended stores signal a duplicate with a different falsy value -- a Redis SET ... NX returns null and a SQL INSERT ... ON CONFLICT returns a rowcount of 0 -- so a store using either would have its duplicate signal missed and the replayed nonce admitted (a replay-protection bypass for a JWT or DPoP proof). enforceReplay now admits a nonce only on a truthy first-seen confirmation and treats every other result as a replay, matching the fail-closed check the framework's inline anti-replay consumers already use.
12
+
13
+ - v0.17.3 (2026-07-17) — **A family of security guards that compared a token case-sensitively where the browser or HTTP stack matches it case-insensitively now normalize first, closing a CSP unsafe-keyword bypass and a cookie-prefix bypass and fixing several over-strict refusals of spec-compliant input.** Browsers, HTTP stacks, and URL parsers match a whole family of security-relevant tokens ASCII case-insensitively -- CSP source keywords and scheme sources, the __Secure-/__Host- cookie name prefixes, HTTP auth schemes, media types, and URL schemes. Several framework guards compared those tokens case-sensitively, so a mixed-case variant behaved differently in the guard than it does downstream. In the bypass direction this let a case-variant unsafe CSP keyword reach a header the browser still honors, and let a prefix-violating cookie past the invariant that keeps the browser from silently dropping it. In the over-strict direction it wrongly refused spec-compliant input (a lowercase auth scheme, a mixed-case media type, an uppercase URL scheme, a mixed-case operator allowlist entry). Every case now normalizes the token to ASCII lowercase before the membership, equality, or prefix test. Separately, two crypto algorithm-name lookups and a CBOR integer-boundary round-trip are corrected. **Fixed:** *Auth scheme, media type, and URL scheme comparisons accept spec-compliant case variants* — Three guards refused input that a case-insensitive downstream would accept. The CIBA notification endpoint (b.auth.ciba) matched the Authorization Bearer scheme case-sensitively, rejecting a spec-compliant 'bearer <token>' sender (RFC 7235 auth-schemes are case-insensitive); it now matches the scheme case-insensitively while comparing the token itself verbatim. The tus upload middleware's creation-with-upload path compared the application/offset+octet-stream media type as an exact string, so a compliant 'Application/Offset+Octet-Stream' (or a variant carrying a parameter) created the upload but silently dropped the body; it now compares the lowercased media type with parameters stripped (RFC 7231 §3.1.1.1). The NEL middleware required its collector URL to begin with a lowercase 'https://', rejecting an equivalent 'HTTPS://'; it now compares the scheme case-insensitively (RFC 3986). · *Redirect allowlist matches a mixed-case operator origin or host* — b.safeRedirect.resolve compared the parser-canonicalized (lowercased) origin and host of a candidate URL against the operator's allowedOrigins/allowedHosts entries verbatim. A mixed-case operator entry such as 'Example.COM' or 'HTTPS://Example.com' therefore silently never matched, and a legitimate redirect fell through to the fallback. The operator entries are now canonicalized to lowercase before the comparison. This only makes the operator's intended allowlist work; because the attacker-controlled target is already normalized by the parser, it never widens the allowlist. · *Streaming-hash and OPRF suite selection reject an unknown algorithm name cleanly* — b.crypto.hashStream (and b.crypto.hashFile, which composes it) and b.crypto.oprf.suite resolved a caller-supplied algorithm/suite name against a plain-object lookup table with a truthiness check. A name colliding with an inherited Object.prototype member -- 'constructor' or '__proto__' -- read back a truthy inherited value and slipped the guard: hashStream then threw a synchronous TypeError from a function documented to return a rejected Promise (an unhandled exception for a caller wired only to .catch()), and oprf.suite returned a malformed suite object instead of throwing its documented bad-suite error. Both now resolve the name through an own-property check, matching the guarding used for the SRI algorithm table, so any unknown name is refused with the documented error. · *Deterministic CBOR re-encodes a negative integer at the -2^53 boundary as an integer* — The CBOR decoder returned the negative integer at the -2^53 boundary as a plain Number, but that value is one below the safe-integer range, so the deterministic encoder's integer branch re-emitted it as a float -- breaking round-trip and falsely tripping the requireDeterministic check on a value that is canonically an integer. The decoder now promotes a negative integer that is not a safe integer to a BigInt, so the encoder re-emits the integer head (RFC 8949 §3). **Security:** *CSP builder refuses unsafe keywords and catch-all schemes in any case* — b.csp.build screened its source tokens ('unsafe-inline', 'unsafe-eval', 'unsafe-hashes', the catch-all '*'/'https:', and 'data:' in img/media/font) against its guard sets with a case-sensitive comparison. A User-Agent matches CSP keywords and scheme sources ASCII case-insensitively (CSP3 §2.3/§6.7.2), so a case-variant such as 'Unsafe-Inline', 'HTTPS:', or 'DATA:' slipped the guard and was emitted verbatim into a Content-Security-Policy header the browser still enforces -- reintroducing the exact XSS-defense hole the guard exists to refuse, without the required acknowledgement opt-in. The builder now lowercases each token before the membership test; the original case is still emitted for real hosts and paths, only the guard comparison is normalized. mergeDirectives routes added sources through the same guard, so a case-variant in a merged policy is refused too. · *Cookie name-prefix invariants hold for any case of __Secure-/__Host-* — Browsers apply the __Secure-/__Host- cookie name-prefix requirements (RFC 6265bis §4.1.3) case-insensitively -- they lowercase the cookie name before the prefix test. b.cookies.serialize and the CSRF middleware's cookie-name safety check compared the prefix case-sensitively, so a case-variant name like __host- or __SECURE- dodged the framework's Secure/Path=//no-Domain invariant while still being subject to the browser's enforcement -- the cookie would then be silently rejected by the browser (never set), defeating the middleware or session it belonged to. Both now compare a lowercased copy of the name, so a prefix-violating cookie is refused at the source for every case. The CSRF middleware, which builds its own Set-Cookie header rather than routing through serialize, additionally gained the __Secure- branch it was missing entirely.
14
+
11
15
  - v0.17.2 (2026-07-17) — **Recurrence expansion refuses to spin or crash on an out-of-range interval, the CIDR guard rejects the whole of the ULA and link-local IPv6 ranges, and a device-bound session assertion no longer accepts a far-future issued-at.** b.calendar.expandRecurrence accepted an unbounded, unvalidated recurrence interval; a large interval drove its date arithmetic past the representable range, which either spun the expansion loop forever at full CPU (a denial of service reachable from any JSCalendar event in a request body) or threw an uncaught error that crashed the caller. The expander now stops as soon as the date arithmetic overflows. b.guardCidr tested IPv6 reserved-range membership on whole hex nibbles, so it missed the parts of the ULA (fc00::/7) and link-local (fe80::/10) ranges that do not fall on a nibble boundary -- accepting fd00::/8 and several fe80::/10 sub-ranges as clean under the strict profile that is supposed to refuse them. And b.dbsc.verifyBindingAssertion bounded the assertion's issued-at only from below, so a far-future issued-at was accepted and never aged out of the replay window. **Security:** *Recurrence expansion refuses an out-of-range interval instead of hanging or crashing* — b.calendar.expandRecurrence validated a recurrence rule's frequency but not its interval, so a caller-supplied interval was used unbounded. A large interval drives the expander's date arithmetic past the representable ECMAScript date range and yields a non-finite date, with two consequences from the one root: with a by-set-position rule the outer loop computed a non-finite period whose inner day-enumeration ran zero times, so the shared step budget never decremented and the not-after break comparisons (which compare against a non-finite value) never fired -- an infinite loop pinning a CPU at 100%; without by-set-position, the loop advanced to a non-finite date and threw an uncaught error when serializing it. Because the rule passed validation, any attacker supplying a JSCalendar event (a JMAP request body, an imported calendar) could trigger it. The expander now stops stepping as soon as the date arithmetic overflows the representable range -- no further instances can exist -- so a hostile interval yields the finite instances that fit and returns, rather than hanging or crashing. · *CIDR guard rejects the full ULA and link-local IPv6 ranges under the strict profile* — b.guardCidr tested whether an IPv6 address falls in a reserved range by comparing hex-nibble prefixes with a string prefix match. Reserved membership is a bit-prefix relation, and the unique-local (fc00::/7, 7 bits) and link-local (fe80::/10, 10 bits) ranges end mid-nibble, so the nibble comparison covered only fc00-fcff and fe80-fe8f -- it missed fd00::/8 (the half of the ULA block that real deployments actually assign) and the fe90::/16 through febf::/16 sub-ranges of link-local. Under the strict profile, which refuses reserved ranges, those CIDRs were accepted as clean and sanitize normalized them into place instead of refusing them. The reserved-range check now compares the whole nibbles and then the remaining prefix bits of the boundary nibble under a mask -- the same bit-prefix relation the IPv4 path and the SSRF guard already use -- so the entire reserved range is caught; nibble-aligned ranges (documentation, multicast, loopback) are unaffected. · *Device-bound session assertions reject a far-future issued-at* — b.dbsc.verifyBindingAssertion enforced the assertion's issued-at only as a lower bound (refusing one older than the configured max age), with no upper bound. A forward-dated issued-at makes the age check compare a negative interval, which never trips, so an assertion carrying an issued-at far in the future was accepted and could not age out of the replay window on the default 300-second path. verifyBindingAssertion now also refuses an issued-at more than a small clock-skew allowance in the future, matching the future-issued-at bound the JWT, DPoP, and client-attestation verifiers already enforce.
12
16
 
13
17
  - v0.17.1 (2026-07-17) — **Profile, posture, and capability name lookups across the content-safety and mail-protocol guards reject a prototype-member name instead of running under it, the mTLS CA can generate a CRL after a fingerprint-only revocation, and a hostile MIME filename no longer crashes attachment extraction.** A second family of guards resolved a profile, posture, or capability name against a plain-object lookup table with a truthiness or in-operator guard, so a name that is an inherited Object.prototype member (constructor, __proto__, toString) slipped the guard and the gate ran under the inherited member instead of refusing the unknown name. The shared profile resolver (b.gateContract.makeProfileResolver, which the guard family composes), the iCal and vCard content guards, the dark-patterns posture check, the JMAP capability allowlist, and the IMAP/POP3/ManageSieve command guards now all resolve names through an own-property check. Separately, b.mtlsCa.generateCrl crashed with a null-serial error once any certificate had been revoked by fingerprint (the mode the require-mTLS gate pins on), dropping every serial-keyed revocation from the published CRL; and b.safeMime.extractAttachments threw an uncaught URIError on a hostile Content-Disposition filename with a malformed percent-escape, crashing the caller. **Fixed:** *mTLS CA generates a CRL after a fingerprint-only revocation* — b.mtlsCa.revoke accepts a certificate fingerprint (the value the require-mTLS gate pins on and generateClientCert surfaces for exactly this use), which is stored without a serial number. b.mtlsCa.generateCrl then mapped the whole revocation registry into CRL entries and handed the null serial to the CRL encoder, which threw -- aborting CRL generation entirely. Because one fingerprint-only revocation broke every subsequent CRL build, all serial-keyed revocations were silently dropped from the published CRL and it could never be regenerated, going stale for external CRL-based revocation checking. generateCrl now projects out fingerprint-only entries (which a standard X.509 CRL cannot represent) before encoding and reports how many were omitted; fingerprint-only revocations remain enforced through the mTLS gate, and the CRL publishes every serial-keyed revocation. · *A hostile MIME filename no longer crashes attachment extraction* — b.safeMime.extractAttachments decoded an RFC 2231 / RFC 5987 extended filename parameter (filename*=charset''percent-encoded) with an unguarded percent-decode, so a malformed escape -- a truncated %, a non-hex %ZZ, or a sequence that decodes to invalid UTF-8 -- threw an uncaught URIError that escaped the parser's typed-error contract and crashed the caller (for example a mail store extracting attachments). A one-line hostile Content-Disposition header was a trivial denial of service. The decode now degrades to the still-encoded filename on failure, matching the framework's handling of every other percent-decode site, so a hostile filename yields a best-effort name that downstream filename guards still vet rather than a crash. **Security:** *Guard profile, posture, and capability lookups reject a prototype-member name* — A group of content-safety and mail-protocol guards resolved a caller- or request-supplied profile / posture / capability name against a plain-object lookup table using a truthiness check (var caps = TABLE[name]; if (!caps)) or the prototype-chain-aware in operator. A name that names an inherited Object.prototype member -- constructor, __proto__, toString -- is truthy (or present via the prototype chain), so it slipped the guard and the gate ran under the inherited member instead of refusing the unknown name. The shared resolver b.gateContract.makeProfileResolver (composed by the idempotency-key, mail-compose, message-id and other guards) now guards both its posture and profile lookups with hasOwnProperty; b.safeIcal, b.safeVcard, b.darkPatterns, b.guardJmap (capability allowlist and profile/posture), and the b.guardImapCommand / b.guardPop3Command / b.guardManageSieveCommand command guards do the same. An unknown or prototype-member name is now rejected with the guard's typed error; supported names are unaffected. This extends the same own-property hardening applied to the crypto algorithm-table lookups in 0.17.0 to the guard-family profile resolvers.
package/lib/auth/ciba.js CHANGED
@@ -632,11 +632,16 @@ function create(opts) {
632
632
  // node:http's normalization could re-introduce the unreachable
633
633
  // branch). Read lowercase only.
634
634
  var authzHeader = req.headers["authorization"];
635
- if (!authzHeader || authzHeader.indexOf("Bearer ") !== 0) {
635
+ // RFC 7235 §2.1 — the auth-scheme token ("Bearer") is ASCII
636
+ // case-insensitive; match it case-insensitively so a spec-compliant
637
+ // `bearer <token>` sender isn't wrongly refused. The token that follows
638
+ // keeps its case (it is compared verbatim below).
639
+ var BEARER_PREFIX = "bearer ";
640
+ if (!authzHeader || authzHeader.slice(0, BEARER_PREFIX.length).toLowerCase() !== BEARER_PREFIX) {
636
641
  throw new AuthError("auth-ciba/missing-bearer",
637
642
  "ciba.parseNotification: Authorization: Bearer header missing");
638
643
  }
639
- var presented = authzHeader.substring("Bearer ".length).trim();
644
+ var presented = authzHeader.slice(BEARER_PREFIX.length).trim();
640
645
  if (presented.length === 0 || !clientNotificationToken) {
641
646
  throw new AuthError("auth-ciba/bad-bearer",
642
647
  "ciba.parseNotification: empty bearer or no expected token configured");
package/lib/cbor.js CHANGED
@@ -393,7 +393,18 @@ function _decodeItem(state, depth) {
393
393
  case 0: return _readArgument(state, ai); // unsigned int
394
394
  case 1: { // negative int
395
395
  var n = _readArgument(state, ai);
396
- return (typeof n === "bigint") ? (-1n - n) : (-1 - n);
396
+ if (typeof n === "bigint") return -1n - n;
397
+ // The argument n is a safe Number, but a negative CBOR value is
398
+ // -1 - n, which reaches one below the safe range: n === 2^53-1
399
+ // yields -2^53, NOT a safe integer. The deterministic encoder's
400
+ // integer branch is |v| <= 2^53-1, so it would re-emit that value
401
+ // as a float — breaking round-trip and falsely tripping
402
+ // requireDeterministic on a canonical integer. Promote to BigInt
403
+ // when the value isn't a safe integer so encode() re-emits the
404
+ // integer head (RFC 8949 §3 — a negative-int argument is an
405
+ // integer, never a float).
406
+ var neg = -1 - n;
407
+ return Number.isSafeInteger(neg) ? neg : (-1n - BigInt(n));
397
408
  }
398
409
  case 2: { // byte string
399
410
  var blen = _lenOf(_readArgument(state, ai));
package/lib/cookies.js CHANGED
@@ -214,15 +214,25 @@ function serialize(name, value, attrs) {
214
214
  // __Host-* — MUST be Secure, Path=/, NO Domain
215
215
  //
216
216
  // Caught at the source so every caller (csrf-protect / session /
217
- // operator) gets the same enforcement.
218
- if (name.indexOf("__Secure-") === 0) {
217
+ // operator) gets the same enforcement. RFC 6265bis §5.4 requires user
218
+ // agents to match these prefixes case-INSENSITIVELY (the server-side
219
+ // description in §4.1.3 reads "case-sensitive", but the UA is what
220
+ // actually enforces the cookie-drop): Chromium and Firefox both
221
+ // lowercase the name before the prefix test, so `__host-` / `__SECURE-`
222
+ // get the same browser enforcement -- Secure, Path=/, no Domain -- as
223
+ // `__Host-` / `__Secure-`. Case-sensitive matching was itself the
224
+ // vulnerability CVE-2024-5699 (httpwg/http-extensions#2231). Compare a
225
+ // lowercased copy so a case-variant name can't dodge the invariant here
226
+ // and then be silently dropped by the browser.
227
+ var lowerName = name.toLowerCase();
228
+ if (lowerName.indexOf("__secure-") === 0) {
219
229
  if (attrs.secure !== true) {
220
230
  throw new CookieError("cookies/prefix-secure-required",
221
231
  "__Secure-* cookies MUST set Secure (RFC 6265bis §4.1.3.1) — got '" +
222
232
  name + "' without secure: true");
223
233
  }
224
234
  }
225
- if (name.indexOf("__Host-") === 0) {
235
+ if (lowerName.indexOf("__host-") === 0) {
226
236
  if (attrs.secure !== true) {
227
237
  throw new CookieError("cookies/prefix-host-secure-required",
228
238
  "__Host-* cookies MUST set Secure (RFC 6265bis §4.1.3.2) — got '" +
@@ -98,8 +98,17 @@ var SUITES = Object.keys(SUITE_IMPL);
98
98
  * // out === s.oprf.evaluate(kp.secretKey, Buffer.from("user@example.com"))
99
99
  */
100
100
  function suite(name) {
101
- var impl = SUITE_IMPL[String(name).toLowerCase()];
102
- if (!impl) throw new OprfError("oprf/bad-suite", "crypto.oprf.suite: unknown suite '" + name + "'; expected one of " + SUITES.join(", "));
101
+ // Own-property guard, not a truthiness check: SUITE_IMPL is a plain object,
102
+ // so a bracket read with a prototype-chain name ("__proto__" Object.prototype,
103
+ // "constructor" → the Object constructor) returns a truthy inherited value that
104
+ // an `if (!impl)` guard would accept as a "suite". Match the hasOwnProperty
105
+ // shape b.crypto.sri uses for its algorithm table so an unknown name — however
106
+ // it collides with Object.prototype — refuses cleanly with OprfError.
107
+ var key = String(name).toLowerCase();
108
+ if (!Object.prototype.hasOwnProperty.call(SUITE_IMPL, key)) {
109
+ throw new OprfError("oprf/bad-suite", "crypto.oprf.suite: unknown suite '" + name + "'; expected one of " + SUITES.join(", "));
110
+ }
111
+ var impl = SUITE_IMPL[key];
103
112
  // Expose only the modes the vendored @noble/curves implements (base +
104
113
  // verifiable). poprf is omitted rather than surfaced as an empty stub.
105
114
  return { name: impl.name, oprf: impl.oprf, voprf: impl.voprf };
package/lib/crypto.js CHANGED
@@ -121,13 +121,20 @@ function hmac(key, data, algorithm) {
121
121
  */
122
122
  function hashStream(readable, algorithm) {
123
123
  var alg = (algorithm || STREAM_HASH_DEFAULT).toLowerCase();
124
- var entry = STREAM_HASH_ALGORITHMS[alg];
125
- if (!entry) {
124
+ // Own-property guard, not a truthiness check: STREAM_HASH_ALGORITHMS is a
125
+ // plain object, so a name colliding with an Object.prototype member
126
+ // ("constructor" → the Object constructor, "__proto__" → Object.prototype)
127
+ // would read back a truthy inherited value, slip past `if (!entry)`, and then
128
+ // throw ERR_INVALID_ARG_TYPE synchronously at createHash(entry.algorithm ===
129
+ // undefined) — a sync throw from a Promise-returning function. Refuse it here
130
+ // as the documented Promise.reject, matching b.crypto.sri's algorithm guard.
131
+ if (!Object.prototype.hasOwnProperty.call(STREAM_HASH_ALGORITHMS, alg)) {
126
132
  return Promise.reject(new TypeError(
127
133
  "crypto.hashStream: unsupported algorithm '" + algorithm +
128
134
  "' (allowed: " + Object.keys(STREAM_HASH_ALGORITHMS).join(", ") + ")"
129
135
  ));
130
136
  }
137
+ var entry = STREAM_HASH_ALGORITHMS[alg];
131
138
  if (!readable || typeof readable.pipe !== "function") {
132
139
  return Promise.reject(new TypeError(
133
140
  "crypto.hashStream: readable must be a Readable stream"
package/lib/csp.js CHANGED
@@ -221,20 +221,28 @@ function build(directives, opts) {
221
221
  "csp.build: source '" + src + "' contains whitespace or ';' — a CSP source " +
222
222
  "must be a single token (directive-injection defense)");
223
223
  }
224
+ // A UA matches CSP source keywords ('unsafe-inline' ...) and scheme
225
+ // sources (https: / data:) ASCII case-INSENSITIVELY (CSP3 §2.3 /
226
+ // §6.7.2). Compare a lowercased copy against the (lowercase) guard sets
227
+ // so a case-variant token ("'Unsafe-Inline'" / "HTTPS:" / "DATA:") can't
228
+ // slip past and be emitted verbatim into a header the browser still
229
+ // honors. The ORIGINAL src is what gets emitted (hosts/paths stay
230
+ // case-preserved); only the guard comparison is normalized.
231
+ var srcLower = src.toLowerCase();
224
232
  if (!acknowledgeUnsafe && SCRIPT_DIRECTIVES.indexOf(name) !== -1 &&
225
- UNSAFE_KEYWORDS.indexOf(src) !== -1) {
233
+ UNSAFE_KEYWORDS.indexOf(srcLower) !== -1) {
226
234
  throw new CspError("csp/unsafe-keyword",
227
235
  "csp.build: " + name + " contains " + src + "; pass acknowledgeUnsafe:true with a " +
228
236
  "documented justification to allow it (CSP3 §6.2.5.x — unsafe keywords are a " +
229
237
  "common XSS bypass surface)");
230
238
  }
231
- if (CATCH_ALL_SOURCES.indexOf(src) !== -1) {
239
+ if (CATCH_ALL_SOURCES.indexOf(srcLower) !== -1) {
232
240
  throw new CspError("csp/catch-all-source",
233
241
  "csp.build: " + name + " contains catch-all source '" + src + "'; CSP3 best " +
234
242
  "practice refuses these (use an explicit allowlist instead)");
235
243
  }
236
244
  if (!allowDataImages && (name === "img-src" || name === "media-src" || name === "font-src") &&
237
- src === "data:") {
245
+ srcLower === "data:") {
238
246
  throw new CspError("csp/data-source",
239
247
  "csp.build: " + name + " contains 'data:'; pass allowDataImages:true with a " +
240
248
  "documented reason (data: URLs sidestep most CSP defenses)");
package/lib/db-query.js CHANGED
@@ -1430,6 +1430,29 @@ function _stripLeadingSqlComments(sql) {
1430
1430
  return s;
1431
1431
  }
1432
1432
 
1433
+ // _normalizeForWriteParse — the single parse-copy normalizer the residency
1434
+ // write-detection regexes run against. The regexes hand-roll SQL tokenization
1435
+ // with `\s+` (whitespace-separated) token boundaries, but SQL lets two tokens
1436
+ // abut with NO whitespace whenever a comment or a quoted-identifier boundary
1437
+ // separates them — so a real write escaped the whitespace-strict detection
1438
+ // entirely (the gate was skipped → a cross-border write landed ungated):
1439
+ //
1440
+ // an INSERT whose quoted table name abuts INTO with no space
1441
+ // an INSERT with a slash-star comment wedged between INSERT and INTO
1442
+ // an UPDATE with a slash-star comment wedged between UPDATE and the table
1443
+ //
1444
+ // All are valid SQLite (the engine tokenizes the comment / quote as a
1445
+ // separator), so the executed statement writes rows while `_rawWriteTable`
1446
+ // returned null and the gate never engaged (CWE-863; GDPR Art. 44-46 transfer
1447
+ // restriction bypass). safeSql.normalizeForScan produces a parse-only copy
1448
+ // where every token boundary is real whitespace (comments collapsed to a space,
1449
+ // a separating space inserted where a quoted token abuts a word character);
1450
+ // leading comments then strip so the ^-anchored regexes see the statement head.
1451
+ // The executed SQL is unchanged; this copy only feeds the gate's parse.
1452
+ function _normalizeForWriteParse(sql) {
1453
+ return _stripLeadingSqlComments(safeSql.normalizeForScan(sql));
1454
+ }
1455
+
1433
1456
  // Non-anchored write-target scan for the writable-CTE / EXPLAIN-prefixed case.
1434
1457
  // A SQLite `WITH c AS (...) INSERT INTO residents ...` / `WITH ... UPDATE residents
1435
1458
  // SET ...` is a real write, but its effective verb is hidden behind the prefix so
@@ -1452,10 +1475,12 @@ function _firstResidencyWriteTarget(s) {
1452
1475
  }
1453
1476
 
1454
1477
  function _rawWriteTable(sql) {
1455
- // The ^-anchored regexes scan only the statement head (constant-time). Strip
1456
- // leading comments first so a commented-out head can't hide the write.
1478
+ // The ^-anchored regexes scan only the statement head (constant-time).
1479
+ // Normalize first so a leading OR internal comment, and a quoted table that
1480
+ // abuts the keyword with no whitespace, can't hide the write from the
1481
+ // whitespace-anchored detection (a gate-skip → cross-border-write bypass).
1457
1482
  if (typeof sql !== "string") return null;
1458
- var s = _stripLeadingSqlComments(sql);
1483
+ var s = _normalizeForWriteParse(sql);
1459
1484
  if (_RAW_WRITE_KEYWORD_RE.test(s)) { // allow:regex-no-length-cap
1460
1485
  var m = _RAW_TABLE_RE.exec(s); // allow:regex-no-length-cap
1461
1486
  return m ? _unquoteIdent(m[1] || m[2]) : null;
@@ -1546,11 +1571,13 @@ function _assertRawWriteResidency(sql, boundParams) {
1546
1571
  if (!cryptoField.getPerRowResidency(table) && !cryptoField.getColumnResidency(table)) return;
1547
1572
  boundParams = _flattenRunParams(boundParams);
1548
1573
 
1549
- // Parse the comment-stripped head: a leading "/* x */" / "-- x" comment must
1550
- // not hide the INSERT/UPDATE body from the ^-anchored regexes below (that would
1551
- // let a residency-restricted write through the gate). The executed SQL is
1574
+ // Normalize the parse copy the same way engagement did: a leading OR internal
1575
+ // comment, and a quoted table/column that abuts the keyword with no whitespace,
1576
+ // must not hide the INSERT/UPDATE body from the whitespace-anchored regexes
1577
+ // below (that would let a residency-restricted write parse as unmodelled and,
1578
+ // for the value it can't read, skip the tag check). The executed SQL is
1552
1579
  // unchanged; this normalized copy is only for residency parsing.
1553
- var norm = _stripLeadingSqlComments(sql);
1580
+ var norm = _normalizeForWriteParse(sql);
1554
1581
 
1555
1582
  // The INSERT/UPDATE body regexes below scan with [\s\S]+; bound the input
1556
1583
  // first and fail CLOSED on an over-long statement - a residency write the
@@ -430,16 +430,30 @@ function create(opts) {
430
430
  if (["Lax", "Strict", "None"].indexOf(cookieCfg.sameSite) === -1) {
431
431
  throw new Error("middleware.csrfProtect: opts.cookie.sameSite must be Lax|Strict|None");
432
432
  }
433
- // __Host- prefix safety: if operator picks a __Host- name, the
434
- // Path/Domain/Secure constraints must be compatible. Path must be "/",
435
- // no Domain (we never set one), Secure resolved per-request. Catch
436
- // operator-side typos (e.g. __Host-csrf with a custom path) at boot.
437
- if (cookieCfg.name && /^__Host-/.test(cookieCfg.name)) {
438
- if (cookieCfg.path !== "/") {
439
- throw new Error("middleware.csrfProtect: __Host-* cookie name requires path='/'");
440
- }
441
- if (cookieCfg.secure === false) {
442
- throw new Error("middleware.csrfProtect: __Host-* cookie name requires secure (cannot be explicit false)");
433
+ // Cookie name-prefix safety (RFC 6265bis §4.1.3). csrf-protect builds its
434
+ // own Set-Cookie header rather than routing through b.cookies.serialize, so
435
+ // this boot check is the only enforcement point. §5.4 requires user agents
436
+ // to apply the prefix test case-INSENSITIVELY (the server-side §4.1.3
437
+ // description reads "case-sensitive", but the UA is what drops the cookie),
438
+ // so `__host-`/`__SECURE-` get the same browser enforcement as
439
+ // `__Host-`/`__Secure-` -- case-sensitive matching was itself CVE-2024-5699.
440
+ // Compare a lowercased copy so a case-variant name can't dodge the invariant
441
+ // here and then be silently rejected by the browser. Catch typos at boot.
442
+ // __Host-* — Path must be "/", no Domain (we never set one), Secure.
443
+ // __Secure-* — Secure.
444
+ if (cookieCfg.name) {
445
+ var lowerCookieName = cookieCfg.name.toLowerCase();
446
+ if (lowerCookieName.indexOf("__host-") === 0) {
447
+ if (cookieCfg.path !== "/") {
448
+ throw new Error("middleware.csrfProtect: __Host-* cookie name requires path='/'");
449
+ }
450
+ if (cookieCfg.secure === false) {
451
+ throw new Error("middleware.csrfProtect: __Host-* cookie name requires secure (cannot be explicit false)");
452
+ }
453
+ } else if (lowerCookieName.indexOf("__secure-") === 0) {
454
+ if (cookieCfg.secure === false) {
455
+ throw new Error("middleware.csrfProtect: __Secure-* cookie name requires secure (cannot be explicit false)");
456
+ }
443
457
  }
444
458
  }
445
459
  }
@@ -151,7 +151,7 @@ function create(opts) {
151
151
  // honor secure-origin report endpoints. Refusing at config-time so
152
152
  // an operator typo (`http://`) surfaces at boot, not as silent
153
153
  // never-fires-in-production.
154
- if (opts.collectorUrl.slice(0, 8) !== "https://") { // string-prefix length, not bytes
154
+ if (opts.collectorUrl.slice(0, 8).toLowerCase() !== "https://") { // RFC 3986 scheme is case-insensitive; string-prefix length, not bytes
155
155
  throw new TypeError(
156
156
  "middleware.nel: opts.collectorUrl must be https:// (browsers " +
157
157
  "ignore non-secure NEL collectors); got " + opts.collectorUrl);
@@ -511,8 +511,12 @@ function create(opts) {
511
511
  if (expHdr) headers["Upload-Expires"] = expHdr;
512
512
 
513
513
  // creation-with-upload: append the body in the same request when
514
- // Content-Type is application/offset+octet-stream.
515
- var contentType = req.headers["content-type"];
514
+ // Content-Type is application/offset+octet-stream. RFC 7231 §3.1.1.1 —
515
+ // the media type is case-insensitive and may carry parameters, so compare
516
+ // the lowercased type/subtype (a compliant `Application/Offset+Octet-Stream`
517
+ // must still take the append path).
518
+ var rawContentType = req.headers["content-type"];
519
+ var contentType = rawContentType ? String(rawContentType).split(";")[0].trim().toLowerCase() : "";
516
520
  if (hasCreationWithBody && contentType === "application/offset+octet-stream") {
517
521
  var chunk;
518
522
  try { chunk = await _readChunk(req, maxChunkSize); }
@@ -291,7 +291,15 @@ async function enforceReplay(store, jti, expireAtMs, opts) {
291
291
  throw new opts.errorClass(opts.storeFailedCode,
292
292
  "replayStore.checkAndInsert threw: " + ((e && e.message) || String(e)));
293
293
  }
294
- if (inserted === false) {
294
+ // Fail CLOSED on ANY non-truthy result, not just a literal `false`. The
295
+ // recommended backends (Redis SETNX, SQL INSERT ... ON CONFLICT) signal a
296
+ // duplicate with a non-`false` falsy value — `SET ... NX` returns null, the
297
+ // SETNX command / an INSERT returns 0 — so an `=== false` compare would miss
298
+ // the replay and admit the token (fail open). Only a truthy result is a
299
+ // positive "first-seen" confirmation; anything else is treated as a replay.
300
+ // Mirrors the truthiness guard every inline consumer already uses
301
+ // (b.auth.oauth `!inserted`, api-encrypt `!freshNonce`, webhook `!fresh`).
302
+ if (!inserted) {
295
303
  throw new opts.errorClass(opts.replayCode,
296
304
  opts.tokenLabel + " jti='" + jti + "' has been seen before — replay refused");
297
305
  }
@@ -165,8 +165,14 @@ function safeDecompress(input, opts) {
165
165
  "windowBits", "audit", "ctx"],
166
166
  "safeDecompress");
167
167
 
168
- // Algorithm — required, must be in allowlist
169
- if (typeof opts.algorithm !== "string" || !_algorithms[opts.algorithm]) {
168
+ // Algorithm — required, must be in allowlist. Use an own-property check,
169
+ // not `!_algorithms[algo]`: a bare truthiness/`in` lookup inherits
170
+ // Object.prototype members, so a non-own key ("constructor", "toString",
171
+ // …) would resolve to a prototype function and get invoked below —
172
+ // `Object(buf)` returns the raw input, silently bypassing the allowlist
173
+ // and returning un-decompressed bytes (fail-open).
174
+ if (typeof opts.algorithm !== "string" ||
175
+ !Object.prototype.hasOwnProperty.call(_algorithms, opts.algorithm)) {
170
176
  throw new SafeDecompressError(
171
177
  "safe-decompress/unsupported-algorithm",
172
178
  "safeDecompress: algorithm must be one of " +
@@ -97,15 +97,23 @@ function resolve(rawTarget, opts) {
97
97
  try { parsed = safeUrl.parse(rawTarget, { allowedProtocols: safeUrl.ALLOW_HTTP_TLS }); }
98
98
  catch (_e) { return fallback; }
99
99
 
100
+ // parsed.origin / host / hostname come lowercased from the WHATWG URL
101
+ // parser (scheme + host are case-insensitive per RFC 3986). Canonicalize
102
+ // each operator allowlist entry the same way before comparing, so a
103
+ // mixed-case operator entry ("HTTPS://Example.com" / "Example.COM") still
104
+ // matches instead of silently never matching and falling through to the
105
+ // fallback. The attacker-controlled rawTarget is already normalized, so
106
+ // this only makes the operator's intended allowlist work — never widens it.
100
107
  if (baseOrigin !== null && parsed.origin === baseOrigin) return rawTarget;
101
108
  if (allowedOrigins) {
102
109
  for (var i = 0; i < allowedOrigins.length; i += 1) {
103
- if (parsed.origin === allowedOrigins[i]) return rawTarget;
110
+ if (parsed.origin === String(allowedOrigins[i]).toLowerCase()) return rawTarget;
104
111
  }
105
112
  }
106
113
  if (allowedHosts) {
107
114
  for (var j = 0; j < allowedHosts.length; j += 1) {
108
- if (parsed.host === allowedHosts[j] || parsed.hostname === allowedHosts[j]) {
115
+ var allowedHost = String(allowedHosts[j]).toLowerCase();
116
+ if (parsed.host === allowedHost || parsed.hostname === allowedHost) {
109
117
  return rawTarget;
110
118
  }
111
119
  }
package/lib/safe-sql.js CHANGED
@@ -494,6 +494,93 @@ function toPositional(sql, dialect) {
494
494
  return out;
495
495
  }
496
496
 
497
+ /**
498
+ * @primitive b.safeSql.normalizeForScan
499
+ * @signature b.safeSql.normalizeForScan(sql)
500
+ * @since 0.17.4
501
+ * @status stable
502
+ * @related b.safeSql.countPlaceholders, b.safeSql.toPositional, b.safeSql.assertSingleStatement
503
+ *
504
+ * Produce a parse-only copy of `sql` whose token boundaries are real
505
+ * whitespace, so a regex tokenizer that assumes whitespace-separated tokens
506
+ * cannot be evaded. SQL lets two tokens abut with NO whitespace whenever a
507
+ * comment OR a quoted-identifier boundary separates them (an `INSERT` whose
508
+ * quoted table name abuts `INTO`, or a slash-star comment wedged between a
509
+ * keyword and the table); a keyword/table detector hand-rolled with `\s+`
510
+ * boundaries silently misses those forms even though the engine executes them.
511
+ * This scan replaces every line (`--`) and slash-star block comment with a
512
+ * single space and inserts a separating space wherever a quoted string /
513
+ * identifier (`'...'` / `"..."` / a backtick-quoted name) abuts a word
514
+ * character on either side — a word char directly before the opening quote OR
515
+ * directly after the closing quote. The same quote- and comment-aware single
516
+ * pass as `countPlaceholders` / `toPositional` (doubled-quote escapes
517
+ * respected), so a comment marker inside a string literal is copied verbatim,
518
+ * never collapsed. The executed SQL is unchanged — this copy only feeds a
519
+ * scanner.
520
+ *
521
+ * @example
522
+ * var b = require("blamejs");
523
+ * b.safeSql.normalizeForScan('INSERT INTO"t"(a) VALUES(?)');
524
+ * // → 'INSERT INTO "t"(a) VALUES(?)'
525
+ *
526
+ * b.safeSql.normalizeForScan('UPDATE"residents"SET x=1');
527
+ * // → 'UPDATE "residents" SET x=1'
528
+ *
529
+ * b.safeSql.normalizeForScan("SELECT 1-- note");
530
+ * // → "SELECT 1 "
531
+ */
532
+ function normalizeForScan(sql) {
533
+ var s = String(sql);
534
+ var out = "";
535
+ var i = 0;
536
+ var len = s.length;
537
+ while (i < len) {
538
+ var c = s.charAt(i);
539
+ var nx = i + 1 < len ? s.charAt(i + 1) : "";
540
+ if (c === "'" || c === '"' || c === "`") {
541
+ // A quoted token abutting a word character gets a separating space so the
542
+ // downstream whitespace-anchored tokenizer sees the boundary — on BOTH
543
+ // sides: before the opening quote when a word char precedes it
544
+ // (`INTO"t"`), and after the closing quote when a word char follows it
545
+ // (`"t"SET` / `UPDATE"residents"SET`). A quoted identifier separates
546
+ // tokens with no whitespace in either direction, so a one-sided boundary
547
+ // still lets a write hide from the scan. Between, copy the whole quoted
548
+ // run verbatim (doubled-quote escapes preserved) so a comment marker
549
+ // inside the literal is never collapsed.
550
+ if (out.length > 0 && /\w/.test(out.charAt(out.length - 1))) out += " ";
551
+ out += c;
552
+ i += 1;
553
+ while (i < len) {
554
+ var q = s.charAt(i);
555
+ out += q;
556
+ if (q === c) {
557
+ if (s.charAt(i + 1) === c) { out += c; i += 2; continue; }
558
+ i += 1; break;
559
+ }
560
+ i += 1;
561
+ }
562
+ if (i < len && /\w/.test(s.charAt(i))) out += " ";
563
+ continue;
564
+ }
565
+ if (c === "-" && nx === "-") { // line comment → one space
566
+ i += 2;
567
+ while (i < len && s.charAt(i) !== "\n") i += 1;
568
+ out += " ";
569
+ continue;
570
+ }
571
+ if (c === "/" && nx === "*") { // block comment → one space
572
+ i += 2;
573
+ while (i < len && !(s.charAt(i) === "*" && s.charAt(i + 1) === "/")) i += 1;
574
+ i += 2;
575
+ out += " ";
576
+ continue;
577
+ }
578
+ out += c;
579
+ i += 1;
580
+ }
581
+ return out;
582
+ }
583
+
497
584
  /**
498
585
  * @primitive b.safeSql.DEFAULT_IDENTIFIER_RE
499
586
  * @signature b.safeSql.DEFAULT_IDENTIFIER_RE
@@ -693,6 +780,7 @@ module.exports = {
693
780
  assertOneOf: assertOneOf,
694
781
  countPlaceholders: countPlaceholders,
695
782
  toPositional: toPositional,
783
+ normalizeForScan: normalizeForScan,
696
784
  SafeSqlError: SafeSqlError,
697
785
  // Exposed so consumers can compose their own validators
698
786
  DEFAULT_IDENTIFIER_RE: DEFAULT_IDENTIFIER_RE,
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@blamejs/core",
3
- "version": "0.17.2",
3
+ "version": "0.17.4",
4
4
  "description": "The Node framework that owns its stack.",
5
5
  "license": "Apache-2.0",
6
6
  "author": "blamejs contributors",
package/sbom.cdx.json CHANGED
@@ -2,10 +2,10 @@
2
2
  "$schema": "http://cyclonedx.org/schema/bom-1.5.schema.json",
3
3
  "bomFormat": "CycloneDX",
4
4
  "specVersion": "1.5",
5
- "serialNumber": "urn:uuid:2d2edd9e-40b2-4d7d-8170-e9e03ba03685",
5
+ "serialNumber": "urn:uuid:52cadd01-ca88-4b82-9fe0-e10eaf4be085",
6
6
  "version": 1,
7
7
  "metadata": {
8
- "timestamp": "2026-07-17T09:06:52.771Z",
8
+ "timestamp": "2026-07-17T12:31:35.543Z",
9
9
  "lifecycles": [
10
10
  {
11
11
  "phase": "build"
@@ -19,14 +19,14 @@
19
19
  }
20
20
  ],
21
21
  "component": {
22
- "bom-ref": "@blamejs/core@0.17.2",
22
+ "bom-ref": "@blamejs/core@0.17.4",
23
23
  "type": "application",
24
24
  "name": "blamejs",
25
- "version": "0.17.2",
25
+ "version": "0.17.4",
26
26
  "scope": "required",
27
27
  "author": "blamejs contributors",
28
28
  "description": "The Node framework that owns its stack.",
29
- "purl": "pkg:npm/%40blamejs/core@0.17.2",
29
+ "purl": "pkg:npm/%40blamejs/core@0.17.4",
30
30
  "properties": [],
31
31
  "externalReferences": [
32
32
  {
@@ -54,7 +54,7 @@
54
54
  "components": [],
55
55
  "dependencies": [
56
56
  {
57
- "ref": "@blamejs/core@0.17.2",
57
+ "ref": "@blamejs/core@0.17.4",
58
58
  "dependsOn": []
59
59
  }
60
60
  ]