@blamejs/core 0.16.10 → 0.16.12

package/CHANGELOG.md

@@ -8,6 +8,10 @@ upgrading across more than a few patches at a time.
 
 ## v0.16.x
 
+- v0.16.12 (2026-07-11) — **Restore the static server's drive-by-execution defense, stop the retention sweep from aborting on subject-scoped rules, make wss:// connections to IP addresses work, and return a verdict instead of throwing on a malformed OpenPGP signature — four defects found by covering previously-untested error and adversarial branches.** Writing behavioral tests for the uncovered error and adversarial branches of the framework's most under-tested files surfaced four genuine defects, now fixed at the root. The static file server's safeAttachmentForRiskyMimes option — the drive-by-execution defense that forces Content-Disposition: attachment for risky inline types (text/html, image/svg+xml, application/javascript) — was silently inert: it was read from a value bag that only carries known-default keys, and this option has no default entry, so it always evaluated to false and the whole defense was dead code. The retention sweep threw and aborted entirely whenever a retention rule with a subjectField met a row with a subject value, because it called a lazily-required module as if it were already resolved. A wss:// connection to an IP-address host threw synchronously and was unusable, because the client set the TLS SNI to the IP literal, which the TLS stack forbids. And b.mail.crypto.pgp.verify threw on a truncated or malformed signature integer instead of returning its documented { ok: false } verdict, so a consumer iterating over untrusted signatures crashed rather than getting a negative result. A static gate now flags any option read from a defaults-applied value bag when that option has no default entry, so the static-server class cannot recur. **Fixed:** *Retention sweep no longer aborts on subject-scoped rules* — A retention rule declared with a subjectField consults the subject-level legal-hold registry when a candidate row carries a subject value. That path called the legal-hold module as though it were already resolved, but it is a lazily-required getter that must be invoked to load the module — so the call threw a TypeError, which the sweep converted into a SWEEP_FAILED and aborted the entire run. The getter is now invoked correctly, so subject-scoped retention sweeps proceed. · *wss:// connections to an IP-address host work* — Connecting the WebSocket client to a wss:// URL whose host is an IP literal threw ERR_INVALID_ARG_VALUE synchronously, because the client set the TLS SNI servername to the IP literal, which RFC 6066 and the Node TLS stack forbid. SNI is now sent only for hostname targets and omitted for IP literals; certificate identity is still verified against the address. **Security:** *Static server's safeAttachmentForRiskyMimes drive-by-execution defense works* — b.staticServe's documented safeAttachmentForRiskyMimes option forces Content-Disposition: attachment for risky inline MIME types (text/html, image/svg+xml, application/javascript) so a browser can't execute an operator-hosted file inline. It was read from the option bag produced by applyDefaults, which strips every key absent from the defaults table — and this option is not a defaults key — so it was always undefined and the defense never engaged. It is now read directly from the caller's options, so enabling it forces the attachment disposition as documented. Operators who set it were relying on a defense that did nothing; it is now active. · *b.mail.crypto.pgp.verify returns a verdict on a malformed signature instead of throwing* — verify() threw MailCryptoError('mail-crypto/pgp/bad-mpi') on a truncated or malformed signature integer, unlike every other malformed-signature path (bad armor, packet parse, hash mismatch), which returns { ok: false, code, reason }. A consumer looping over untrusted signatures and branching on .ok would crash on a crafted input rather than receive a negative verdict. Malformed signature integers now route through the same failure path and return { ok: false }. **Detectors:** *applydefaults-dropped-opt* — A static gate flags any option read from the result of applyDefaults(opts, DEFAULTS) when that option is absent from the DEFAULTS table — the exact shape that silently disabled the static server's attachment defense. The result of applyDefaults carries only the defaults' keys, so such a read is always undefined. The fix is to read the option from the caller's opts directly, or to add it to the defaults table.
+
+- v0.16.11 (2026-07-11) — **Fix a TOML parser bug where every double-quoted key decoded to the empty string — which also let a quoted "__proto__" key slip past the prototype-pollution guard — and return a real 201 from SCIM resource creation, both surfaced by covering previously-untested error and adversarial branches.** Writing behavioral tests for the uncovered error, adversarial, and defensive branches of the framework's most under-tested files surfaced two genuine defects, now fixed at the root. b.parsers.toml decoded every double-quoted (basic) key to the empty string: a document like "host" = "db1" parsed to { "": "db1" }, multiple distinct quoted keys collided onto one empty key, and — the security consequence — a quoted "__proto__" or "constructor" key decoded to the empty string and so slipped past the poisoned-key guard that rejects those names. TOML keys are usually written bare, so the bug was latent since the parser shipped. It is fixed to decode quoted keys correctly, and a quoted "__proto__" / "constructor" is now rejected with toml/poisoned-key. Separately, the SCIM server returned an undefined HTTP status instead of 201 Created when creating a User or Group singleton, because it referenced a status constant that did not exist; it now returns 201 with the created representation. **Fixed:** *b.parsers.toml decodes double-quoted keys correctly* — A basic (double-quoted) key with no escape sequence decoded to the empty string — "host" = "db1" became { "": "db1" }, and multiple distinct quoted keys in one table collided onto a single empty key. The decoder now appends the verbatim key run for both quoted-key kinds, so quoted, literal, dotted, and escaped keys all decode to their intended names. · *SCIM resource creation returns 201 Created* — Creating a User or Group via POST returned an undefined HTTP status because the handler referenced a CREATED status constant that was not defined, so the created representation was written with no valid status code. It now returns 201 with the created resource. **Security:** *b.parsers.toml rejects a quoted __proto__ / constructor key* — The double-quoted-key decoder never appended the key's characters for a basic key, so every double-quoted key resolved to the empty string. A quoted "__proto__" = ... or "constructor" = ... therefore decoded to "", which the poisoned-key guard does not flag, letting those names through. Quoted keys now decode to their real value, and a quoted __proto__ / constructor is rejected with toml/poisoned-key. Bare-key documents were unaffected; only double-quoted keys were mis-decoded.
+
 - v0.16.10 (2026-07-11) — **Restore five APIs that silently didn't work — certificate-fingerprint hashing from PEM, the tus-upload close handler, the mail-delivery factory, the log-stream local-sink env boot, and the CIBA internal-IdP opt — and make single-base guard profile composition work, all surfaced by writing a test for every documented primitive.** Writing a behavioral test for every documented primitive that previously had none surfaced a batch of APIs that were advertised but silently non-working, now fixed at the root. b.crypto.hashCertFingerprint and b.crypto.isCertRevoked threw a TypeError on the PEM-string input their own examples pass (a helper was called on its lazy-require getter instead of the module), so only the raw-DER path worked; b.middleware.tusUpload.close and b.mail.send.deliver.create were documented but never attached to their export, so calling the documented form threw 'not a function'; b.logStream.bootFromEnv advertised a 'local' protocol that threw at init because it wired the sink with the wrong option; and the CIBA client dropped the allowInternal opt on its own backchannel and token POSTs, so an operator who explicitly opted into an internal or loopback identity provider was still SSRF-blocked. Separately, gate-contract profile composition now honors its documented extends: string | string[] contract — a single base profile name passed as a bare string was silently ignored, which made the buildProfile example every guard shares a no-op. The EU AI Act fundamental-rights-impact-assessment and GPAI training-data-summary builders now raise typed ComplianceError refusals with descriptive messages instead of a bare Error whose message was only the error code. Five documentation examples are corrected to their real emitted values (the SRI integrity attribute, two guard rule-ids, a guard issue kind, and the guard-archive profile-composition call). The static gate that requires every documented primitive to be referenced by a test now has an empty backlog register: every documented primitive is exercised. **Added:** *A test for every documented primitive* — The primitive-without-test static gate's backlog register, which listed documented primitives that no test referenced, is now empty: every documented primitive is exercised by a behavioral test through its real consumer path. A new primitive must land with its own test reference; the register accepts an entry only with a specific documented reason coverage is indirect. **Changed:** *Guard profile composition accepts a single base profile as a bare string* — gate-contract profile composition documents extends: string | string[], but a bare-string extends was silently ignored — only the array form composed — which made the buildProfile example every guard shares (buildProfile({ extends: "strict" })) a no-op that returned an empty caps object. A string extends is now normalized to a one-element list, so single-base composition works across every b.guard*.buildProfile. The array form is unchanged. **Fixed:** *b.crypto.hashCertFingerprint / isCertRevoked accept PEM input again* — Both threw 'TypeError: safeBuffer.byteLengthOf is not a function' on a PEM-string certificate — the internal PEM-to-DER step called the byte-length helper on its lazy-require getter rather than the resolved module, so only the pre-parsed DER/Buffer path (which returns before that line) worked. Their documented examples pass a PEM string, so the advertised behavior was silently broken. Fixed at the call site. · *b.middleware.tusUpload.close is reachable* — The middleware export wired tusUpload.create and .memoryStore but never attached the documented .close handler, so b.middleware.tusUpload.close(middleware) threw 'not a function'. It is now attached to the export. · *b.mail.send.deliver.create is reachable* — b.mail.send.deliver was bound directly to the factory function but never carried the documented .create property, so the documented b.mail.send.deliver.create(opts) form resolved to undefined. The .create factory is now attached; the existing collapsed b.mail.send.deliver(opts) callable is unchanged. · *b.logStream.bootFromEnv local protocol works* — BLAMEJS_LOG_STREAM_PROTOCOL=local threw 'log-stream local requires { dir }' at init: bootFromEnv wired the local sink with a path option, but the directory-based local sink requires dir. BLAMEJS_LOG_STREAM_PATH is now mapped to the sink directory, so the advertised local protocol boots and writes end-to-end. · *CIBA client honors allowInternal on its own requests* — The backchannel-authentication and token POSTs of the CIBA client dropped the allowInternal option that create() accepts and threads to the OAuth client for discovery/JWKS, so an operator who explicitly set allowInternal:true for an internal or loopback identity provider was still refused with ssrf-guard/blocked-loopback on exactly those two endpoints. The option is now threaded through per request. The SSRF guard stays on by default; nothing changes unless the operator opts in. · *EU AI Act FRIA / training-data-summary raise typed refusals with real messages* — b.compliance.aiAct.fundamentalRightsImpactAssessment and b.compliance.aiAct.gpai.trainingDataSummary validated required fields by passing the built-in Error as the error class, so the descriptive message was dropped (its slot became the Error options bag) and the thrown message was only the error code. Both now raise a typed ComplianceError with the descriptive message and a stable code, matching the sibling adherenceForm builder. · *Five documentation examples corrected to real values* — The b.crypto.sri example carried a fabricated SHA-384 value (an operator copy-pasting the integrity attribute would get a mismatch); it now shows the correct digest. The b.guardTime.validate example named rule-id time.year-out-of-range (actual time.year-window); b.guardUuid named uuid.nil / uuid.max (actual uuid.nil-uuid / uuid.max-uuid); b.htmlBalance.checkSafe named issue kind event-handler-attribute (actual event-handler). The b.guardArchive.buildProfile example passed { profile: "strict" }, which resolves no caps; it now uses the correct { extends: "strict" } form.
 
 - v0.16.9 (2026-07-10) — **`b.safeBuffer.quoteString` — one RFC quoted-string serializer behind eight header and protocol emitters — plus a consolidated per-primitive test suite and a static gate that refuses a documented primitive no test references.** Eight places in the framework serialized an RFC quoted-string by hand — escape backslash and DQUOTE, wrap in DQUOTEs — for Cache-Status and Signature-Input sf-strings, Link header parameters, Authentication-Results reason, IMAP quoted strings and METADATA values, ManageSieve responses, and Server-Timing descriptions. Those copies are consolidated into one primitive, b.safeBuffer.quoteString, so an unescaped quote can never terminate a string early and smuggle extra parameters into a protocol line, and the escaping cannot drift between emitters. Output is byte-identical to before. The test suite is reorganized so every primitive's tests live in a canonical per-primitive file (thirty-eight auxiliary files folded or renamed; every assertion preserved — the full suite passes with the exact same check count). On top of that reorganization, the comment-block validator gains a primitive-without-test check: a documented @primitive that no test references — neither the full dotted form nor its namespace plus a method invocation — now fails the static gate. One hundred six existing primitives currently lack any test reference; they are recorded in an explicit shrink-only register in the validator, each entry an open test-backfill item, and the gate refuses any new primitive shipping untested. **Added:** *b.safeBuffer.quoteString — RFC quoted-string serialization* — Coerces to string, escapes every backslash and DQUOTE with a leading backslash, and wraps the result in DQUOTEs — the quoted-string grammar shared by RFC 8941 §3.3.3 Structured Fields sf-string, RFC 8288 Link header parameters, RFC 8601 §2.2 Authentication-Results reason, RFC 3501 §4.3 IMAP quoted strings, and RFC 5804 §1.2 ManageSieve strings. Escaping only: a grammar with a restricted character range (sf-string is printable-ASCII; IMAP quoted strings cannot carry CR/LF) enforces its range check before calling it. · *Static gate: a documented primitive must be referenced by a test* — The comment-block validator now scans the test corpus for every @primitive: the full dotted form (b.namespace.method) anywhere under test/, or the namespace / owning module referenced together with the method invoked in the same test file — the same-file rule stops a ubiquitous method name like .create( in an unrelated test from counting as coverage. A primitive with no reference fails the pre-push static gate. One hundred six existing primitives currently have no reference; they are listed in a shrink-only register inside the validator — each entry an open test-backfill item that is deleted when its test lands — and a new primitive cannot ship untested. **Changed:** *Eight quoted-string emitters route through the one serializer* — Cache-Status key/detail, Signature-Input sf-strings, Link header parameter values, Authentication-Results reason, IMAP METADATA values and quoted strings, ManageSieve OK/NO/BYE/LISTSCRIPTS responses, and Server-Timing desc now compose b.safeBuffer.quoteString instead of hand-rolling the escape. Emitted bytes are unchanged. · *Per-primitive canonical test files* — Thirty-eight auxiliary test files are folded into (or renamed to) their primitive's canonical test file — one file per primitive domain, concern-named splits kept (e.g. http-client-cache / -stream / -throttle). No assertion was changed, added, or removed by the reorganization; the full suite passes with the exact same check count before and after. The b.*-surface reference gate that previously ran inside the smoke suite is superseded by the stronger validator check above. **Detectors:** *rfc-quoted-string-escape-owned-by-safeBuffer* — Flags an inline backslash-then-DQUOTE escape chain in lib/ outside the primitive's home — the quoted-string serializer re-implemented, whose escaping can drift from the canonical one. While being proven against the pre-consolidation tree it caught an eighth emitter the manual sweep had missed (an IMAP response-literal helper), which is consolidated in this release too.

package/lib/mail-crypto-pgp.js

@@ -866,7 +866,13 @@ function verify(opts) {
 
   var ok;
   if (parsed.pubAlg === PUB_ALG_RSA) {
-    var rsaMpi = _readMpi(parsed.sigMpisBytes, 0);
+    // A malformed / truncated signature MPI is untrusted input like any
+    // other packet field: return a verdict, don't throw. Every other
+    // bad-signature path here routes through _fail; the raw _readMpi
+    // reads did not, so a truncated MPI escaped as an exception.
+    var rsaMpi;
+    try { rsaMpi = _readMpi(parsed.sigMpisBytes, 0); }
+    catch (mpiErr) { return _fail(mpiErr.code || "mail-crypto/pgp/bad-mpi", mpiErr.message); }
     // The signature is an integer in [0, n); when its value has one or more
     // high zero bytes (~1/256 of signatures) the OpenPGP MPI encoding strips
     // them (RFC 9580 §3.2), but node's RSA verify requires a signature exactly
@@ -889,9 +895,13 @@ function verify(opts) {
     }
   } else {
     // Ed25519Legacy — two MPIs (R, S) reassemble into the 64-byte raw
-    // EdDSA signature.
-    var rMpi = _readMpi(parsed.sigMpisBytes, 0);
-    var sMpi = _readMpi(parsed.sigMpisBytes, rMpi.next);
+    // EdDSA signature. Truncated/malformed MPIs are untrusted input:
+    // return a verdict via _fail rather than throwing out of verify().
+    var rMpi, sMpi;
+    try {
+      rMpi = _readMpi(parsed.sigMpisBytes, 0);
+      sMpi = _readMpi(parsed.sigMpisBytes, rMpi.next);
+    } catch (mpiErrE) { return _fail(mpiErrE.code || "mail-crypto/pgp/bad-mpi", mpiErrE.message); }
     function _padTo32(buf) {
       if (buf.length === 32) return buf;
       if (buf.length > 32) return buf.slice(buf.length - 32);

package/lib/middleware/scim-server.js

@@ -262,7 +262,7 @@ async function _dispatch(req, res, basePath, bearer, opts, maxPageSize, bulkCfg)
   if (req.method === "POST" && !resourceId) {
     _assertSchema(body, resourceType === "Users" ? SCIM_CORE_SCHEMA_USER : SCIM_CORE_SCHEMA_GROUP);
     var created = await impl.create(body, ctx);
-    _writeJson(res, H.CREATED, created);
+    _writeJson(res, 201, created);   // 201 Created (HTTP_STATUS has no CREATED key)
     return;
   }
 

package/lib/parsers/safe-toml.js

@@ -201,7 +201,16 @@ function parse(input, opts) {
     while (!_eof()) {
       var c = _peek();
       if (c === quote) {
-        if (!basic) out = input.substring(start, pos);
+        // Append the pending verbatim run for BOTH kinds. A basic key
+        // accumulates decoded escapes into `out` and resets `start` past
+        // each; the trailing literal run (or the whole key when it has no
+        // escapes) must still be appended here. A literal key never
+        // touches `out` before this point, so the append yields its full
+        // verbatim content. Assigning only for the literal branch dropped
+        // every escape-free basic key to "" — collapsing distinct quoted
+        // keys together and letting a quoted `"__proto__"` slip past the
+        // poisoned-key guard by decoding to the empty string.
+        out += input.substring(start, pos);
         _advance();
         return out;
       }

package/lib/retention.js

@@ -492,7 +492,7 @@ function create(opts) {
           // tables), the central registry is authoritative. Honors
           // the same skip semantics as the per-row field.
           if (rule.subjectField && row[rule.subjectField]) {
-            var holdsRegistry = legalHold._getSingleton();
+            var holdsRegistry = legalHold()._getSingleton();
             if (holdsRegistry && holdsRegistry.isHeld(row[rule.subjectField])) {
               summary.legalHoldsHonored++;
               _emit("retention.row.legal_hold_skipped",

package/lib/static.js

@@ -845,7 +845,11 @@ function create(opts) {
   var auditSuccess    = cfg.auditSuccess;
   var auditFailures   = cfg.auditFailures;
   var acceptRanges    = cfg.acceptRanges;
-  var safeAttachment  = !!cfg.safeAttachmentForRiskyMimes;
+  // Read straight from opts: safeAttachmentForRiskyMimes is not a DEFAULTS
+  // key, so applyDefaults(opts, DEFAULTS) never carries it into cfg — reading
+  // it off cfg silently dropped the opt and left the drive-by-execution
+  // defense (Content-Disposition: attachment for risky inline MIMEs) inert.
+  var safeAttachment  = !!opts.safeAttachmentForRiskyMimes;
   // forceAttachmentForNonText default follows mountType (v0.15.0): a
   // mount TYPED "user-content" forces risky inline MIMEs to download by
   // default (stored-XSS defense for untrusted uploads); a "curated" mount

package/lib/ws-client.js

@@ -406,10 +406,24 @@ class WsClient extends EventEmitter {
       var tlsOpts = Object.assign({
         host:         host,
         port:         port,
-        servername:   host,
         rejectUnauthorized: true,
         minVersion:   "TLSv1.3",
       }, dialTlsOpts || {});
+      // RFC 6066 §3: the TLS SNI ServerName MUST be a hostname, never an IP
+      // literal — Node throws ERR_INVALID_ARG_VALUE when servername is an IP,
+      // which made wss:// to any IP-literal target unusable. Send SNI only for
+      // hostname targets; for an IP-literal host omit it (cert identity is still
+      // verified against the address' iPAddress SAN by the default
+      // checkServerIdentity). An operator servername in tlsOpts still wins.
+      // Decide via ssrfGuard.canonicalizeHost so an IPv6 literal is recognized:
+      // parsed.hostname keeps the brackets (`[::1]`), and net.isIP only accepts
+      // the bare form — canonicalizeHost strips the brackets (and normalizes the
+      // address) so the IP test is correct. Without it an IPv6 literal would set
+      // servername=`[::1]`, a malformed SNI a strict server rejects. Hostnames
+      // canonicalize to a non-IP form, so they still get their SNI.
+      if (!tlsOpts.servername && !net.isIP(ssrfGuard.canonicalizeHost(host))) {
+        tlsOpts.servername = host;
+      }
       if (lookup) tlsOpts.lookup = lookup;
       try {
         var pqcShares = networkTls().pqc.getKeyShares();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blamejs/core",
-  "version": "0.16.10",
+  "version": "0.16.12",
   "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:1c234c55-1297-48c3-bbd2-7b7b1f134fa5",
+  "serialNumber": "urn:uuid:6bed96f9-e949-4536-b328-26e4e3b8964d",
   "version": 1,
   "metadata": {
-    "timestamp": "2026-07-11T03:08:14.929Z",
+    "timestamp": "2026-07-11T10:33:50.471Z",
     "lifecycles": [
       {
         "phase": "build"
@@ -19,14 +19,14 @@
       }
     ],
     "component": {
-      "bom-ref": "@blamejs/core@0.16.10",
+      "bom-ref": "@blamejs/core@0.16.12",
       "type": "application",
       "name": "blamejs",
-      "version": "0.16.10",
+      "version": "0.16.12",
       "scope": "required",
       "author": "blamejs contributors",
       "description": "The Node framework that owns its stack.",
-      "purl": "pkg:npm/%40blamejs/core@0.16.10",
+      "purl": "pkg:npm/%40blamejs/core@0.16.12",
       "properties": [],
       "externalReferences": [
         {
@@ -54,7 +54,7 @@
   "components": [],
   "dependencies": [
     {
-      "ref": "@blamejs/core@0.16.10",
+      "ref": "@blamejs/core@0.16.12",
       "dependsOn": []
     }
   ]