@blamejs/core 0.9.1 → 0.9.3
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 +2 -0
- package/lib/auth/aal.js +29 -16
- package/lib/auth/ciba.js +60 -7
- package/lib/auth/fido-mds3.js +51 -10
- package/lib/auth/oauth.js +34 -9
- package/lib/auth/oid4vci.js +14 -6
- package/lib/auth/oid4vp.js +19 -2
- package/lib/auth/passkey.js +64 -5
- package/package.json +1 -1
- package/sbom.cdx.json +6 -6
package/CHANGELOG.md
CHANGED
|
@@ -8,6 +8,8 @@ upgrading across more than a few patches at a time.
|
|
|
8
8
|
|
|
9
9
|
## v0.9.x
|
|
10
10
|
|
|
11
|
+
- v0.9.3 (2026-05-11) — **Audit hardening slice 3: OAuth + OID4VCI + OID4VP + CIBA + constant-time-compare migrations**. Continues the 2026-05-11 auth audit follow-through. **`b.auth.oauth.refreshAccessToken` atomic check-and-insert** — new `ropts.checkAndInsert(token, expireAtMs)` callback contract replaces the previous `ropts.seen(token)` check-then-act race. Two concurrent refresh requests on the same event-loop tick could both see `seen === false` and both POST to the token endpoint, neither flagging the replay; the new contract requires an atomic test-and-set (Redis SETNX, DB INSERT ON CONFLICT) and is the OAuth 2.1 §6.1 / RFC 9700 §4.13 one-time-use defense surfacing the actual race window. Legacy `seen` callback continues to work for backwards-compat with operator code; the docstring documents the race + recommends migration to `checkAndInsert`. **`b.auth.oid4vci` constant-time compares** — pre-auth `tx_code` hash compare (was `!==` on sha3 hex) and proof-JWT `c_nonce` compare (was `!==` on attacker-supplied wallet payload) both route through `b.crypto.timingSafeEqual`. **`b.auth.oid4vp` per-presentation `vct` enforcement** — DCQL filters with 2+ `vct_values` entries previously bypassed vct validation entirely (the framework only set `expectedVct` when the filter pinned to a single value). Verifier now validates the presented vct against the DCQL filter list manually when length > 1; refuses with `vp_token['<id>'][<n>] vct '<presented>' is not in DCQL vct_values [...]` on over-disclosure. **`b.auth.ciba` slow_down honoring** — CIBA §11.3 requires the client to increase its polling interval by at least 5s on every `slow_down` response. Pre-v0.9.3 the framework client never bumped, leaving operators to do their own interval bookkeeping. Now `pollToken()` tracks per-`authReqId` interval state internally (Map keyed by authReqId, seeded from `startAuthentication`'s response, cleared on token issuance), bumps by `max(5s, IdP-suggested interval) <= MAX_INTERVAL_SEC` on every slow_down, and attaches the next-suggested interval to the thrown `auth-ciba/slow_down` error as `err.nextIntervalSec` so operators read a spec-correct back-off without manual bookkeeping. **`b.auth.ciba` notification-token entropy** — `clientNotificationToken` now refuses < 32 chars per CIBA §7.1.2's opaque-hard-to-guess requirement. Pre-v0.9.3 a 4-char token passed. **`b.auth.ciba.parseNotification` constant-time compare** — bearer-token hash compare migrated from `!==` to `b.crypto.timingSafeEqual` (both sides are fixed-width sha3-512 hex strings; defense-in-depth even though equal-length JS string compare is already widely understood as constant-time on V8).
|
|
12
|
+
- v0.9.2 (2026-05-11) — **Audit hardening slice 2: WebAuthn + FIDO MDS3 + NIST AAL**. Continues the 2026-05-11 auth surface audit follow-through. **`b.auth.passkey.verifyAuthentication` counter-regression bypass fix** — pre-v0.9.2 the wrapper coerced `opts.credential.counter || 0`, silently zeroing an `undefined` / `null` / `NaN` counter and defeating CTAP 2.1 clone-detection on credentials whose stored counter was > 0. An operator deserializing the credential from a column that lost the counter would unknowingly accept a cloned authenticator. The wrapper now refuses `undefined` / `null` (operators MUST persist whatever the vendor returned at registration; first-time-stored credentials carry counter:0 explicitly) and rejects any non-integer / non-finite / negative value with `auth-passkey/bad-counter`. **`b.auth.passkey` multi-origin support** — `expectedOrigin` now accepts `string` OR `string[]` on both `verifyRegistration` and `verifyAuthentication`. Pre-v0.9.2 the wrapper enforced a single string only, blocking multi-origin deployments (web + admin-subdomain) from sharing one verifier; SimpleWebAuthn natively supports arrays. **`b.auth.passkey` prototype-pollution fix** — `ALLOWED_MEDIATION` lookup changed from `{...}[opts.mediation]` to `hasOwnProperty.call(ALLOWED_MEDIATION, opts.mediation)` with a null-prototype map. Pre-v0.9.2 a caller passing `mediation: "__proto__"` / `"constructor"` truthy-matched an inherited Object.prototype property and slipped past the allowlist into `generateAuthenticationOptions`. **`b.auth.aal.fromMethods` UV requirement for AAL3** — per NIST SP 800-63-4 §5.1.7, WebAuthn / passkey satisfies AAL3 (MF-CRYPT) only when user verification was performed on the assertion. Pre-v0.9.2 `fromMethods({ webauthn: true })` returned `AAL3` unconditionally; operators using `userVerification: "preferred"` whose authenticator skipped UV landed in AAL3 despite not satisfying the spec's MF requirement. The caller now passes `uv: true` (sourced from the vendor's authData UV bit) to claim AAL3 with webauthn alone; without `uv`, webauthn alone caps at AAL2 (SF-CRYPT). Combination paths (`webauthn + password` / `webauthn + pin`) reach AAL3 regardless of UV (the memorized secret provides the second factor independently). **`b.auth.fidoMds3.verifyAuthenticator` fail-closed default + new opts** — pre-v0.9.2 unknown AAGUIDs returned `{ ok: true, reason: "aaguid-not-in-blob" }`, silently trusting any authenticator the metadata service hadn't yet listed (rogue / pre-certification / fake hardware). Now fails closed by default; operators wanting the legacy fail-open behavior (test fixtures, pre-certification pilot rollouts) pass `opts.allowUnknownAaguid: true` explicitly. **`b.auth.fidoMds3.parseBlob` stale-BLOB refusal** — refuses BLOB payloads whose `nextUpdate` is already in the past (FIDO MDS3 §3.1.7). Pre-v0.9.2 the staleness was floored to `MIN_CACHE_TTL_MS` but the BLOB was still served, letting an attacker pin operators to a revoked-authenticator-list-frozen-at-X by serving an ancient signed-but-expired BLOB. **`b.auth.fidoMds3.REFUSE_STATUS`** — added `ATTESTATION_KEY_COMPROMISE` per FIDO MDS3 §3.1.4. Pre-v0.9.2 this status was silently accepted; manufacturer batch-signing-key compromise affects every credential attested under that key.
|
|
11
13
|
- v0.9.1 (2026-05-11) — **Audit hardening, slice 1 of N: federation auth + OAuth ID-token verifier**. Audit run after v0.9.0 across OAuth / SAML / OIDC federation / SD-JWT VC / WebAuthn / FIDO MDS3 / constant-time-compare surfaces; this slice closes the highest-severity SAML + OIDC + SD-JWT + OAuth findings. **SAML SP** (`b.auth.saml.sp.create({...}).buildAuthnRequest` + `.metadata`): every operator-supplied URL / ID interpolated into the emitted XML now routes through `b.xmlC14n.escapeAttrValue` / `escapeText`. Pre-v0.9.1 a `"` or `<` in `idpSsoUrl` / `assertionConsumerServiceUrl` / `entityId` / `nameIdFormat` broke out of attribute / element context and produced unsigned-XML breakout into the IdP redirect. Newly exported `b.xmlC14n.escapeAttrValue(s)` and `b.xmlC14n.escapeText(s)` — RFC 3741 §1.3.x compliant; available for any operator emitting XML alongside the framework's own SAML / canonicalization paths. **SAML SP `verifyResponse`**: digest compare on `Reference DigestValue` and `SubjectConfirmation InResponseTo` now use `b.crypto.timingSafeEqual` instead of `Buffer.compare` / `!==`. **SAML XSW defense**: refuses Response payloads that carry duplicate `<Status>`, `<StatusCode>`, `<Assertion>`, `<Subject>`, or `<NameID>` children — XML signature wrapping attacks ferry an unsigned sibling next to a signed element and exploit first-match parsers; the verifier now asserts single-child cardinality on every security-critical element via `_findAllChildren(...).length === 1`. **OIDC federation** (`b.auth.openidFederation.verifyEntityStatement`): JWK key-type cross-check against the JWS `alg` header BEFORE `nodeCrypto.createPublicKey` — an attacker-controlled entity-config declaring `alg: "ES256"` while supplying an RSA JWK would previously load through Node's silent algorithm-vs-key coercion path. Now refuses with `auth-openid-federation/alg-kty-mismatch` for any `alg=ES*` not paired with `kty=EC`, `alg=PS*`/`RS*` not paired with `RSA`, or `alg=EdDSA` not paired with `OKP`. **`b.auth.openidFederation.buildTrustChain` error-masking**: trust-chain ascent previously swallowed every per-authority failure via `catch (_e) {}` and continued to the next `authority_hint`; signature-failure errors from one authority no longer mask, the chain now refuses on cryptographic refusal (`bad-jwk`, `alg-kty-mismatch`, `bad-signature`, `signature-failed`). Network / 404 / iss-sub-mismatch errors still continue to the next hint but are collected and surfaced in the `no-ascent` failure shape. **SD-JWT VC verify** (`b.auth.sdJwtVc.verify`): three correctness fixes. (1) `_sd_alg` default switched from `sha3-512` to `sha-256` per IETF draft §4.1.1 — prior default broke verification against spec-conformant issuers when the issuer omitted `_sd_alg`. (2) Disclosure-replay defense: every disclosure digest tracked in a Set; second occurrence of the same digest refuses with `auth-sd-jwt-vc/disclosure-replay`. (3) Claim-shadowing defense: holder-supplied disclosures whose name collides with an issuer-signed top-level claim (`iss`, `sub`, `aud`, `iat`, `nbf`, `exp`, `jti`, `vct`, `cnf`, `_sd`, `_sd_alg`, `status`) refuse with `auth-sd-jwt-vc/protected-claim-shadow` instead of silently overwriting the signed value. (4) KB-JWT `sd_hash` now uses the credential's declared `_sd_alg` (was hardcoded sha256, breaking against issuers using sha3-512); `sd_hash` compare routed through `b.crypto.timingSafeEqual`. **OAuth `verifyIdToken`** (`b.auth.oauth.verifyIdToken`): three hardening fixes that bring the verifier to parity with the framework's other JWS verifiers. (1) `nodeCrypto.verify` wrapped in try/catch — previously panicked on key/sig shape mismatch (e.g. ES256 sig against an RS256 key returned by a buggy IdP with duplicate kids), bubbling a raw `Error` to the operator's handler instead of an `OAuthError`. (2) RFC 7515 §4.1.11 `crit` header refusal — every sibling verifier (`b.auth.jwt`, `b.auth.jwt.verifyExternal`, `b.auth.dpop`) refuses; verifyIdToken previously silently ignored, letting an attacker-controlled OP ship critical extensions the verifier doesn't understand. (3) `state` and `nonce` claim compares routed through `b.crypto.timingSafeEqual` — these are CSRF / replay tokens compared against attacker-controlled callback / payload data.
|
|
12
14
|
- v0.9.0 (2026-05-11) — **Minor: 3 new RFC primitives + `b.structuredFields` shared substrate + full audit-derived hardening sweep + 5 new bug-class detectors**. **`b.structuredFields`** consolidates the quote-aware top-level splitter (`splitTopLevel(s, sep)`), the raw-value control-byte refusal scan (`refuseControlBytes` / `containsControlBytes`), and the sf-string unquote (`unquoteSfString`) used by every RFC 8941 / RFC 9110 / RFC 9111 / RFC 9213 / RFC 9421 / RFC 6266 / RFC 6265 / RFC 6455 parser in the framework — replaces the per-file open-coded copies that were drifting site-by-site. **8 audit-surfaced bug-class sites fixed in this same patch** (no deferred follow-up): (1) `b.middleware.bodyParser._contentType` + `_parseHeaderParams` — RFC 9110 Content-Type / RFC 6266 Content-Disposition parameters can carry quoted-string values (`boundary="foo;bar"` / `filename="weird;name.txt"`); bare `.split(";")` previously sliced through quoted semicolons and corrupted multipart boundaries. (2) `b.requestHelpers.parseListHeader({ strictToken: true })` — control-byte scan now runs on the RAW value before `.trim()` so a leading `\n<token>` no longer slips past `RFC_9110_TOKEN_RE` (same v0.8.90 bug class; used by webhook signature parsing and WS subprotocol negotiation). (3) `b.middleware.tusUpload._parseChecksumHeader` — same trim-before-validate fix. (4) `b.httpClient.cache._parseCacheControl` — quote-aware `,` splitter (RFC 9111 §5.2 + RFC 9110 §5.6.4 directive values may be quoted-string). (5) `b.httpClient.cookieJar._parseSetCookie` — quote-aware `;` splitter defends RFC 7230 quoted-string attribute values (`SameSite="Strict"` from interop-imperfect upstreams). (6) `b.websocket._parseExtensionHeader` — quote-aware `;` and `,` splitters defend RFC 6455 §9.1 + RFC 7230 token-or-quoted-string parameter values against forward-compat extensions shipping quoted params. (7) `b.aiPref.parseHeader` — control-byte refusal added on the RAW value before split + trim. (8) `b.auth.stepUp.parseChallenge` — same trim-before-validate fix (returns `null` per defensive-reader contract instead of throwing). Also: `b.logStream.init({ minLevel })` now validates the level vocabulary at config time so a typo'd `"infos"` (which previously produced `LEVEL_PRIORITY["infos"] === undefined` and silently dropped every log record) throws at boot. `b.crypto.httpSig`'s RFC 9421 Signature-Input parameter parser uses the quote-aware `;` splitter (RFC 8941 §3.1.2 sf-string parameter values). `b.security.assertProductionPosture({ minTlsVersion })` validates `minTlsVersion` against the canonical TLS vocabulary BEFORE the rank comparison (a typo previously silently passed because `indexOf` returned `-1` — same bug class as v0.8.88 `b.auth.fal.meets`). **3 new RFC primitives**: **`b.cdnCacheControl`** ships an RFC 9213 directive list builder + parser shared across `Cache-Control`, `CDN-Cache-Control`, `Surrogate-Control`, and the operator-specific `Cloudflare-` / `Vercel-` / `Fastly-` / `Akamai-` / `Netlify-CDN-Cache-Control` variants. `build({...})` emits the directive list string (numeric directives non-negative-integer-only; refuses Infinity / NaN / floats / negatives; full RFC 9111 boolean directive set; `extensions` for non-standard directives with RFC 7234 §5.2 token-shape enforcement); refuses `public + private` conflict per RFC 9111 §5.2.2.5/§5.2.2.6. `parse(headerValue)` decodes any targeted header into `{ public, private, noStore, maxAge, sMaxAge, ..., directives, fields }` with **qualified-form support** (`private="Authorization"` flag stays enabled, field-name list under `.fields[camel]` per RFC 9111 §5.2.2.4 / §5.2.2.6 — presence == enabled, the argument narrows scope), **bare `max-stale` parses as `Infinity`** per RFC 9111 §5.2.1.2 (instead of buggy `Number(true) === 1`), and a quote-aware top-level `,` splitter so a `, ` inside a quoted directive value doesn't fake-split. `isTargetedHeader(name)` + curated `TARGETED_HEADERS` allowlist. **`b.clientHints`** ships a Sec-CH-UA-* request-header family parser per W3C UA Client Hints + IETF draft-davidben-http-client-hint-reliability. `parse(req.headers)` returns `{ brands, mobile, platform, platformVersion, arch, bitness, model, fullVersionList, wow64, formFactors, raw }`; quote-aware splitters at brand-list and brand-member-parameter level (RFC 8941 §4.1.1.4 parameter values may be sf-string); refuses control characters in any Sec-CH-* value. `acceptList(hintNames)` builds `Accept-CH` with typo-defense (unknown hint name throws `client-hints/unknown-hint`); dedupes case-variant duplicates; canonicalizes to W3C mixed-case spelling. `KNOWN_HINTS` exports the well-known 22-name list. **`b.network.dns.classifyDnskeyAlgorithm(algorithm)` / `classifyDsDigestType(digestType)`** — RFC 9905 DNSSEC SHA-1 deprecation classifier. Covers every IANA-assigned DNSKEY algorithm (including PRIVATEDNS/PRIVATEOID/INDIRECT/Reserved entries) and RFC 9558 §3 DS digest types 5 (GOST R 34.11-2012) + 6 (SM3); operators auditing inbound DNSSEC chain-of-trust evidence refuse validations where `deprecated === true`. Inline `allow:bare-split-on-quoted-header` markers added across `mail-auth.js` (DKIM / DMARC / ARC tag-list grammar — token-only), `network-smtp-policy.js` (TLS-RPT — token-only), `middleware/scim-server.js` (RFC 7644 §3.9 SCIM attribute paths), `http-client-cache.js` (RFC 9110 §12.5.5 Vary field-names), `http-message-signature.js` (RFC 9421 component-id covered list), `middleware/body-parser.js` (RFC 9112 §6.1 Transfer-Encoding token-only), each citing the controlling RFC clause showing why quoted-string is not a legal value in that grammar. **5 new bug-class detectors** in `test/layer-0-primitives/codebase-patterns.test.js` so the same shapes can't drift back in: (a) `trim-before-validate` — control-byte refusal scans must run on the RAW header value BEFORE `.trim()` strips leading/trailing C0/DEL bytes; detector now catches BOTH the `charCodeAt` codepoint-loop shape AND the `<NAME>_RE.test(<trimmed>)` grammar-regex shape; (b) `enum-rank-without-validation` — `_rankFn(X) >= _rankFn(Y)` arithmetic comparisons must have a preceding `isValid*` / `KNOWN_*` membership check on both inputs (catches `b.auth.fal.meets` bug shape); (c) `bool-string-coerce-shape` — boolean directive parsing must NOT use `val === "" || val === "true"` coercion (catches `b.cdnCacheControl.parse` qualified-form bug shape); (d) `bare-split-on-quoted-header` — RFC structured-fields parsers in files that ALSO handle sf-string unquote regex must use the shared quote-aware `b.structuredFields.splitTopLevel`, not bare `.split(",") / .split(";")`; (e) `scoped-context-binding-unused` — scope-named factory bindings (`forwarderDomain` / `realm` / `origin` / `audience` / `issuer`) captured in the factory must be compared against the inbound value's embedded scope in the `verify` / `reverse` / `decode` path (catches the v0.8.89 SRS forwarder-domain bug shape). Operators upgrade `0.8.90` → `0.9.0`; v0.8.91 was never tagged (its surface is folded into v0.9.0 with the audit-derived hardening).
|
|
13
15
|
- v0.8.90 (2026-05-11) — **RFC 8689 REQUIRETLS support** (`b.mail.requireTls`). Per-message TLS-requirement signaling between sender and receiver MTAs. Complements MTA-STS / DANE (policy-side, domain-scoped) with a per-message knob that overrides policy when the operator wants stricter-than-policy delivery — message bounces instead of falling back to cleartext if no downstream MTA can deliver under TLS. **`peerSupports(ehloLines)`**: walks a parsed EHLO response and returns `true` when the peer advertised the `REQUIRETLS` keyword; case-insensitive per RFC 5321 §2.4; refuses substring matches (`FOO-REQUIRETLS-BAR` does NOT match); empty / non-array input returns `false`. **`mailFromExtension({ requireTls })`**: builds the trailing `" REQUIRETLS"` token to append to a MAIL FROM line; refuses non-boolean flag value (a truthy-but-wrong-shape value like `"yes"` throws instead of silently succeeding). **`parseTlsRequiredHeader(headerValue)`**: parses the RFC 8689 §5 `TLS-Required` header — returns `"no"` only when the value is the literal token `no` (case-insensitive, ignoring whitespace) per spec; any other non-empty value returns `"yes"` (RFC 8689 §5: "any value other than 'No' MUST be treated as if the field had been absent" — conservative strict path); returns `null` for absent / empty / non-string input; refuses control characters on the **raw** header value before `trim()` runs so a leading `\n` / trailing `\r` / NUL / DEL byte can no longer slip past as the literal token `no` (ASCII HT remains permitted as structural folding whitespace).
|
package/lib/auth/aal.js
CHANGED
|
@@ -70,38 +70,51 @@ function _bandRank(band) {
|
|
|
70
70
|
var KNOWN_METHODS = [
|
|
71
71
|
"password", "pin", "totp", "sms", "webauthn", "passkey",
|
|
72
72
|
"hardware", "mtls",
|
|
73
|
+
// `uv` is a webauthn-side qualifier: when true, the
|
|
74
|
+
// authenticator-data UV bit was set on the assertion. Required
|
|
75
|
+
// for AAL3 paired with `webauthn` / `passkey` per SP 800-63-4
|
|
76
|
+
// §5.1.7.
|
|
77
|
+
"uv",
|
|
73
78
|
];
|
|
74
79
|
|
|
75
80
|
function fromMethods(methods) {
|
|
76
81
|
if (!methods || typeof methods !== "object") {
|
|
77
82
|
throw new AuthError("auth-aal/bad-methods",
|
|
78
|
-
"fromMethods: methods must be an object like { password: true, webauthn: true }");
|
|
83
|
+
"fromMethods: methods must be an object like { password: true, webauthn: true, uv: true }");
|
|
79
84
|
}
|
|
80
85
|
var has = function (m) { return methods[m] === true; };
|
|
81
|
-
|
|
82
|
-
//
|
|
83
|
-
// -
|
|
84
|
-
//
|
|
85
|
-
//
|
|
86
|
-
//
|
|
87
|
-
//
|
|
88
|
-
|
|
86
|
+
// SP 800-63-4 §5.1.7 — WebAuthn / passkey satisfies AAL3 only when
|
|
87
|
+
// user verification (UV) was actually performed on the assertion
|
|
88
|
+
// (MF-CRYPT requires the verifier to confirm the user authorized
|
|
89
|
+
// the operation). Pre-v0.9.2 this returned AAL3 unconditionally
|
|
90
|
+
// for any webauthn:true assertion; an operator using
|
|
91
|
+
// `userVerification: "preferred"` whose authenticator skipped UV
|
|
92
|
+
// landed in AAL3 despite not satisfying the spec's MF requirement.
|
|
93
|
+
//
|
|
94
|
+
// The operator passes `methods.uv: true` when verifyAuthentication's
|
|
95
|
+
// result confirmed UV on the authenticator data (vendor's
|
|
96
|
+
// `userVerified` flag). When `uv` is omitted or false, webauthn
|
|
97
|
+
// alone caps at AAL2 (SF-CRYPT — the cryptographic authenticator
|
|
98
|
+
// is verified, but user-verification proof is missing).
|
|
99
|
+
// Operators wanting the legacy optimistic path can pass
|
|
100
|
+
// `methods.uv: true` based on their startAuthentication
|
|
101
|
+
// `userVerification: "required"` setting having forced UV.
|
|
102
|
+
if ((has("webauthn") || has("passkey")) && has("uv")) return AAL3;
|
|
103
|
+
if ((has("webauthn") || has("passkey")) && !has("uv")) {
|
|
104
|
+
// SF-CRYPT (cryptographic but no UV-bound MF). Combine with a
|
|
105
|
+
// memorized secret to satisfy MF.
|
|
106
|
+
if (has("password") || has("pin")) return AAL3;
|
|
107
|
+
return AAL2;
|
|
108
|
+
}
|
|
89
109
|
if (has("hardware") && (has("password") || has("pin"))) return AAL3;
|
|
90
110
|
|
|
91
|
-
// Multi-factor → AAL2.
|
|
92
|
-
// - password + totp / sms / hardware single-factor / mtls.
|
|
93
|
-
// - SP 800-63-4 §5.1.3.3 — SMS is RESTRICTED but still satisfies
|
|
94
|
-
// AAL2 with the operator's documented risk acceptance.
|
|
95
111
|
if (has("password") || has("pin")) {
|
|
96
112
|
if (has("totp") || has("sms") || has("hardware") || has("mtls")) return AAL2;
|
|
97
113
|
return AAL1; // memorized secret alone
|
|
98
114
|
}
|
|
99
115
|
|
|
100
|
-
// Single-factor cryptographic — mtls / hardware on its own → AAL1
|
|
101
|
-
// unless paired with a memorized secret (handled above).
|
|
102
116
|
if (has("hardware") || has("mtls")) return AAL1;
|
|
103
117
|
|
|
104
|
-
// No recognized method asserted.
|
|
105
118
|
throw new AuthError("auth-aal/no-methods",
|
|
106
119
|
"fromMethods: methods object did not assert any known authenticator " +
|
|
107
120
|
"(known: " + KNOWN_METHODS.join(", ") + ")");
|
package/lib/auth/ciba.js
CHANGED
|
@@ -55,7 +55,7 @@ var lazyRequire = require("../lazy-require");
|
|
|
55
55
|
var validateOpts = require("../validate-opts");
|
|
56
56
|
var safeJson = require("../safe-json");
|
|
57
57
|
var safeUrl = require("../safe-url");
|
|
58
|
-
var { generateToken, sha3Hash } = require("../crypto");
|
|
58
|
+
var { generateToken, sha3Hash, timingSafeEqual } = require("../crypto");
|
|
59
59
|
var { AuthError } = require("../framework-error");
|
|
60
60
|
|
|
61
61
|
var httpClient = lazyRequire(function () { return require("../http-client"); });
|
|
@@ -169,6 +169,16 @@ function create(opts) {
|
|
|
169
169
|
throw new AuthError("auth-ciba/no-notification-token",
|
|
170
170
|
"auth.ciba.client.create: clientNotificationToken required for ping/push delivery modes");
|
|
171
171
|
}
|
|
172
|
+
// Minimum-entropy guard on the client_notification_token (audit
|
|
173
|
+
// 2026-05-11). CIBA §7.1.2 requires the token be opaque + hard to
|
|
174
|
+
// guess; the framework's other token-shaped primitives enforce 32
|
|
175
|
+
// chars minimum. A 4-char token was previously accepted; refuse.
|
|
176
|
+
if (clientNotificationToken !== null && clientNotificationToken.length < 32) { // allow:raw-byte-literal — RFC 9700 §7.1.2 token char-length minimum, not bytes
|
|
177
|
+
throw new AuthError("auth-ciba/notification-token-too-short",
|
|
178
|
+
"auth.ciba.client.create: clientNotificationToken must be >= 32 chars " +
|
|
179
|
+
"(generate via b.crypto.generateToken(32) or stronger; CIBA §7.1.2 " +
|
|
180
|
+
"requires opaque hard-to-guess token).");
|
|
181
|
+
}
|
|
172
182
|
|
|
173
183
|
// Each backchannel-authentication request mints a fresh
|
|
174
184
|
// `client_notification_token` per the spec? No — the RP registers
|
|
@@ -368,6 +378,10 @@ function create(opts) {
|
|
|
368
378
|
? rv.interval : DEFAULT_INTERVAL_SEC;
|
|
369
379
|
var expiresIn = typeof rv.expires_in === "number" && rv.expires_in > 0
|
|
370
380
|
? rv.expires_in : DEFAULT_EXPIRES_SEC;
|
|
381
|
+
// Seed the per-authReqId interval tracker so pollToken's
|
|
382
|
+
// slow_down handler bumps from the IdP-supplied starting point
|
|
383
|
+
// (CIBA §11.3 minimum-5s bump on every slow_down response).
|
|
384
|
+
_registerInitialInterval(rv.auth_req_id, interval);
|
|
371
385
|
|
|
372
386
|
_emitAudit("start", "success", {
|
|
373
387
|
authReqIdHash: sha3Hash("auth-ciba:" + rv.auth_req_id),
|
|
@@ -399,6 +413,17 @@ function create(opts) {
|
|
|
399
413
|
* var tokens = await ciba.pollToken({ authReqId: ticket.authReqId });
|
|
400
414
|
* // → { accessToken, idToken, refreshToken, tokenType, scope, expiresIn, raw }
|
|
401
415
|
*/
|
|
416
|
+
// Per-authReqId interval tracking — CIBA §11.3 requires the client
|
|
417
|
+
// to increase its polling interval by at least 5s on every
|
|
418
|
+
// `slow_down` response. The framework client now maintains this
|
|
419
|
+
// state internally so operators reading `err.nextIntervalSec` get
|
|
420
|
+
// a spec-correct back-off without rolling their own counter.
|
|
421
|
+
var _intervalState = new Map(); // authReqId → current interval sec
|
|
422
|
+
|
|
423
|
+
function _registerInitialInterval(authReqId, intervalSec) {
|
|
424
|
+
_intervalState.set(authReqId, intervalSec);
|
|
425
|
+
}
|
|
426
|
+
|
|
402
427
|
async function pollToken(popts) {
|
|
403
428
|
popts = popts || {};
|
|
404
429
|
if (typeof popts.authReqId !== "string" || popts.authReqId.length === 0) {
|
|
@@ -421,7 +446,33 @@ function create(opts) {
|
|
|
421
446
|
body.set("client_id", opts.clientId);
|
|
422
447
|
}
|
|
423
448
|
if (clientAuth === "mtls") body.set("client_id", opts.clientId);
|
|
424
|
-
var rv
|
|
449
|
+
var rv;
|
|
450
|
+
try {
|
|
451
|
+
rv = await _postForm(endpoint, body);
|
|
452
|
+
} catch (err) {
|
|
453
|
+
// CIBA §11.3 — on slow_down response, increase polling
|
|
454
|
+
// interval by at least 5s. Attach the next-suggested-interval
|
|
455
|
+
// to the error so the operator's poll loop reads a spec-
|
|
456
|
+
// correct back-off without manual bookkeeping. The IdP MAY
|
|
457
|
+
// optionally return its own `interval` value in the 400 body;
|
|
458
|
+
// honor that when >= current + 5, otherwise enforce the
|
|
459
|
+
// minimum 5s bump.
|
|
460
|
+
if (err && err.code === "auth-ciba/slow_down") {
|
|
461
|
+
var current = _intervalState.get(popts.authReqId) || DEFAULT_INTERVAL_SEC;
|
|
462
|
+
var idpSuggested = err.cibaError && typeof err.cibaError.interval === "number"
|
|
463
|
+
? err.cibaError.interval : null;
|
|
464
|
+
var next = current + 5; // allow:raw-time-literal — §11.3 mandates +5s minimum
|
|
465
|
+
if (idpSuggested !== null && idpSuggested > next && idpSuggested <= MAX_INTERVAL_SEC) {
|
|
466
|
+
next = idpSuggested;
|
|
467
|
+
}
|
|
468
|
+
if (next > MAX_INTERVAL_SEC) next = MAX_INTERVAL_SEC;
|
|
469
|
+
_intervalState.set(popts.authReqId, next);
|
|
470
|
+
err.nextIntervalSec = next;
|
|
471
|
+
}
|
|
472
|
+
throw err;
|
|
473
|
+
}
|
|
474
|
+
// Token issued — clear interval tracking for this authReqId.
|
|
475
|
+
_intervalState.delete(popts.authReqId);
|
|
425
476
|
_emitAudit("token_received", "success", {
|
|
426
477
|
authReqIdHash: sha3Hash("auth-ciba:" + popts.authReqId),
|
|
427
478
|
});
|
|
@@ -479,13 +530,15 @@ function create(opts) {
|
|
|
479
530
|
throw new AuthError("auth-ciba/bad-bearer",
|
|
480
531
|
"ciba.parseNotification: empty bearer or no expected token configured");
|
|
481
532
|
}
|
|
482
|
-
// Constant-time compare
|
|
483
|
-
//
|
|
484
|
-
//
|
|
485
|
-
//
|
|
533
|
+
// Constant-time compare on the SHA3 hash of both tokens —
|
|
534
|
+
// matches the project-wide discipline (audit 2026-05-11). Both
|
|
535
|
+
// sides are fixed-width sha3-512 hex strings; timingSafeEqual
|
|
536
|
+
// adds explicit defense-in-depth over `!==` even though equal-
|
|
537
|
+
// length JS string compare is already broadly understood as
|
|
538
|
+
// constant-time on V8.
|
|
486
539
|
var presentedHash = sha3Hash(presented);
|
|
487
540
|
var expectedHash = sha3Hash(clientNotificationToken);
|
|
488
|
-
if (presentedHash
|
|
541
|
+
if (!timingSafeEqual(presentedHash, expectedHash)) {
|
|
489
542
|
_emitAudit("notification_token_mismatch", "failure", {});
|
|
490
543
|
throw new AuthError("auth-ciba/wrong-bearer",
|
|
491
544
|
"ciba.parseNotification: client_notification_token does not match");
|
package/lib/auth/fido-mds3.js
CHANGED
|
@@ -70,6 +70,11 @@ var REFUSE_STATUS = {
|
|
|
70
70
|
REVOKED: 1,
|
|
71
71
|
USER_KEY_PHYSICAL_COMPROMISE: 1,
|
|
72
72
|
USER_KEY_REMOTE_COMPROMISE: 1,
|
|
73
|
+
// FIDO MDS3 §3.1.4 — attestation-key compromise means the
|
|
74
|
+
// manufacturer's batch-signing key is suspect; every credential
|
|
75
|
+
// attested under that key MUST be refused. Pre-v0.9.2 this token
|
|
76
|
+
// was missing from the refuse-list (audit 2026-05-11).
|
|
77
|
+
ATTESTATION_KEY_COMPROMISE: 1,
|
|
73
78
|
};
|
|
74
79
|
|
|
75
80
|
// FIDO Certified levels that surface as certifiedLevel. The spec uses
|
|
@@ -333,6 +338,21 @@ function _verifyAndParseBlob(token) {
|
|
|
333
338
|
throw new FidoMds3Error("fido-mds3/bad-payload",
|
|
334
339
|
"BLOB payload 'nextUpdate' missing or not YYYY-MM-DD: " + payload.nextUpdate);
|
|
335
340
|
}
|
|
341
|
+
// Stale-BLOB refusal — FIDO MDS3 §3.1.7 says clients SHOULD refresh
|
|
342
|
+
// by nextUpdate; a BLOB whose nextUpdate is already in the past is
|
|
343
|
+
// not safe to trust even though its cert chain still validates.
|
|
344
|
+
// Pre-v0.9.2 the staleness was floored to MIN_CACHE_TTL_MS in
|
|
345
|
+
// _ttlFromNextUpdate but the BLOB itself was still served from
|
|
346
|
+
// cache; an attacker serving an ancient signed-but-expired BLOB
|
|
347
|
+
// could keep operators on a revoked-authenticator-list-frozen-at-X.
|
|
348
|
+
// Refuse at parse time so neither fetch nor cache lookup honors it.
|
|
349
|
+
// (Audit 2026-05-11.)
|
|
350
|
+
if (nextUpdate.getTime() < Date.now()) {
|
|
351
|
+
throw new FidoMds3Error("fido-mds3/blob-stale",
|
|
352
|
+
"BLOB payload nextUpdate \"" + payload.nextUpdate +
|
|
353
|
+
"\" is in the past — refusing to trust a stale metadata BLOB " +
|
|
354
|
+
"(FIDO MDS3 §3.1.7)");
|
|
355
|
+
}
|
|
336
356
|
return {
|
|
337
357
|
entries: payload.entries,
|
|
338
358
|
no: payload.no,
|
|
@@ -539,7 +559,7 @@ function _certifiedLevel(statusReports) {
|
|
|
539
559
|
|
|
540
560
|
/**
|
|
541
561
|
* @primitive b.auth.fidoMds3.verifyAuthenticator
|
|
542
|
-
* @signature b.auth.fidoMds3.verifyAuthenticator(blob, registrationInfo)
|
|
562
|
+
* @signature b.auth.fidoMds3.verifyAuthenticator(blob, registrationInfo, opts)
|
|
543
563
|
* @since 0.8.53
|
|
544
564
|
* @status stable
|
|
545
565
|
* @related b.auth.fidoMds3.fetch, b.auth.fidoMds3.lookupAaguid
|
|
@@ -549,21 +569,31 @@ function _certifiedLevel(statusReports) {
|
|
|
549
569
|
* `{ ok, statement, statusReports, certifiedLevel, reason? }`. Refuses
|
|
550
570
|
* (ok: false) when the authenticator's status reports include any of
|
|
551
571
|
* REVOKED / USER_KEY_PHYSICAL_COMPROMISE / USER_KEY_REMOTE_COMPROMISE
|
|
552
|
-
* (FIDO MDS3 section 3.1.4 compromise
|
|
553
|
-
*
|
|
554
|
-
*
|
|
555
|
-
*
|
|
572
|
+
* / ATTESTATION_KEY_COMPROMISE (FIDO MDS3 section 3.1.4 compromise
|
|
573
|
+
* bucket).
|
|
574
|
+
*
|
|
575
|
+
* AAGUIDs not present in the BLOB **fail closed by default** in
|
|
576
|
+
* v0.9.2+ (pre-v0.9.2 returned `ok: true, statement: null`, silently
|
|
577
|
+
* trusting any authenticator not yet in the metadata service). To
|
|
578
|
+
* accept unknown AAGUIDs (test fixtures, pre-certification rollouts),
|
|
579
|
+
* pass `opts.allowUnknownAaguid: true`; the `reason` field then notes
|
|
580
|
+
* the operator opt-in.
|
|
556
581
|
*
|
|
557
582
|
* Audits auth.fido_mds3.verify.refused (drop-silent) on compromise.
|
|
558
583
|
*
|
|
584
|
+
* @opts
|
|
585
|
+
* allowUnknownAaguid: boolean, // default false (fail-closed)
|
|
586
|
+
*
|
|
559
587
|
* @example
|
|
560
588
|
* var blob = { entries: [] };
|
|
561
589
|
* var reg = { aaguid: "00000000-0000-0000-0000-000000000000" };
|
|
562
|
-
* var rv = b.auth.fidoMds3.verifyAuthenticator(blob, reg
|
|
590
|
+
* var rv = b.auth.fidoMds3.verifyAuthenticator(blob, reg,
|
|
591
|
+
* { allowUnknownAaguid: true });
|
|
563
592
|
* rv.ok === true && rv.statement === null;
|
|
564
|
-
* // → true
|
|
593
|
+
* // → true (with operator opt-in)
|
|
565
594
|
*/
|
|
566
|
-
function verifyAuthenticator(blob, registrationInfo) {
|
|
595
|
+
function verifyAuthenticator(blob, registrationInfo, vopts) {
|
|
596
|
+
vopts = vopts || {};
|
|
567
597
|
if (!blob) {
|
|
568
598
|
throw new FidoMds3Error("fido-mds3/bad-blob", "blob is required");
|
|
569
599
|
}
|
|
@@ -573,12 +603,23 @@ function verifyAuthenticator(blob, registrationInfo) {
|
|
|
573
603
|
}
|
|
574
604
|
var entry = lookupAaguid(blob, registrationInfo.aaguid);
|
|
575
605
|
if (!entry) {
|
|
606
|
+
// Fail-CLOSED default for unknown AAGUIDs (audit 2026-05-11).
|
|
607
|
+
// Pre-v0.9.2 default was `ok: true, reason: "aaguid-not-in-blob"`
|
|
608
|
+
// — an attacker registering a credential with an AAGUID not in
|
|
609
|
+
// the BLOB (rogue authenticator, fake hardware) silently passed.
|
|
610
|
+
// The framework's primitive now refuses by default; operators
|
|
611
|
+
// who genuinely want to accept unknown authenticators (test
|
|
612
|
+
// fixtures, pre-certification pilot rollouts) pass
|
|
613
|
+
// `vopts.allowUnknownAaguid: true` explicitly.
|
|
614
|
+
var unknownOk = vopts.allowUnknownAaguid === true;
|
|
576
615
|
return {
|
|
577
|
-
ok:
|
|
616
|
+
ok: unknownOk,
|
|
578
617
|
statement: null,
|
|
579
618
|
statusReports: [],
|
|
580
619
|
certifiedLevel: { level: 0, plus: false },
|
|
581
|
-
reason:
|
|
620
|
+
reason: unknownOk
|
|
621
|
+
? "aaguid-not-in-blob (operator opted in via allowUnknownAaguid)"
|
|
622
|
+
: "aaguid-not-in-blob",
|
|
582
623
|
};
|
|
583
624
|
}
|
|
584
625
|
var statusReports = Array.isArray(entry.statusReports) ? entry.statusReports : [];
|
package/lib/auth/oauth.js
CHANGED
|
@@ -561,20 +561,45 @@ function create(opts) {
|
|
|
561
561
|
// constrained confidential clients. Operators with sender-
|
|
562
562
|
// constrained tokens (DPoP / mTLS) can opt out by NOT supplying
|
|
563
563
|
// a seen callback.
|
|
564
|
-
|
|
565
|
-
|
|
564
|
+
//
|
|
565
|
+
// Atomic check-and-insert (audit 2026-05-11) — pre-v0.9.3 the
|
|
566
|
+
// check ran via `ropts.seen(token)` which was a check-then-act
|
|
567
|
+
// race: two concurrent refresh requests landed on the same
|
|
568
|
+
// event-loop tick could both see `seen === false` and both POST
|
|
569
|
+
// to the token endpoint, neither flagging the replay. The new
|
|
570
|
+
// contract is `ropts.checkAndInsert(token, expireAtMs)` which
|
|
571
|
+
// MUST atomically test-and-set: returns true if the token was
|
|
572
|
+
// ALREADY in the store (replay) and false if it just inserted
|
|
573
|
+
// the token. The legacy `seen` callback continues to work for
|
|
574
|
+
// backward compatibility but emits a deprecation warning.
|
|
575
|
+
var alreadySeen = false;
|
|
576
|
+
if (typeof ropts.checkAndInsert === "function") {
|
|
577
|
+
var nowMs = Date.now();
|
|
578
|
+
// 24h max refresh-token TTL — operators with shorter TTLs
|
|
579
|
+
// should configure their store's own expiry policy.
|
|
580
|
+
var expireAtMs = nowMs + C.TIME.hours(24);
|
|
581
|
+
try { alreadySeen = await ropts.checkAndInsert(refreshToken, expireAtMs); }
|
|
582
|
+
catch (e) {
|
|
583
|
+
throw new OAuthError("auth-oauth/seen-callback-failed",
|
|
584
|
+
"refreshAccessToken: checkAndInsert() callback threw: " + ((e && e.message) || String(e)));
|
|
585
|
+
}
|
|
586
|
+
} else if (typeof ropts.seen === "function") {
|
|
587
|
+
// Legacy non-atomic path. Documented as a check-then-act race;
|
|
588
|
+
// operators sharing a single-writer store (Redis SETNX, DB
|
|
589
|
+
// INSERT ON CONFLICT) MUST migrate to checkAndInsert. Stays
|
|
590
|
+
// here for backwards-compat with existing operator code.
|
|
566
591
|
try { alreadySeen = await ropts.seen(refreshToken); }
|
|
567
592
|
catch (e) {
|
|
568
593
|
throw new OAuthError("auth-oauth/seen-callback-failed",
|
|
569
594
|
"refreshAccessToken: seen() callback threw: " + ((e && e.message) || String(e)));
|
|
570
595
|
}
|
|
571
|
-
|
|
572
|
-
|
|
573
|
-
|
|
574
|
-
|
|
575
|
-
|
|
576
|
-
|
|
577
|
-
|
|
596
|
+
}
|
|
597
|
+
if (alreadySeen === true) {
|
|
598
|
+
throw new OAuthError("auth-oauth/refresh-token-replay",
|
|
599
|
+
"refreshAccessToken: refresh token has been presented before — refused " +
|
|
600
|
+
"(OAuth 2.1 §6.1 / RFC 9700 §4.13 one-time-use defense). The operator MUST " +
|
|
601
|
+
"treat this as a token-theft signal: revoke the refresh-token family + force " +
|
|
602
|
+
"the user to re-authenticate.");
|
|
578
603
|
}
|
|
579
604
|
var endpoint = await _resolveEndpoint("tokenEndpoint");
|
|
580
605
|
var body = new URLSearchParams();
|
package/lib/auth/oid4vci.js
CHANGED
|
@@ -54,7 +54,7 @@ var lazyRequire = require("../lazy-require");
|
|
|
54
54
|
var validateOpts = require("../validate-opts");
|
|
55
55
|
var safeJson = require("../safe-json");
|
|
56
56
|
var nodeCrypto = require("node:crypto");
|
|
57
|
-
var { generateToken, sha3Hash } = require("../crypto");
|
|
57
|
+
var { generateToken, sha3Hash, timingSafeEqual } = require("../crypto");
|
|
58
58
|
var { AuthError } = require("../framework-error");
|
|
59
59
|
|
|
60
60
|
var cache = lazyRequire(function () { return require("../cache"); });
|
|
@@ -116,9 +116,14 @@ function _verifyProofJwt(proofJwt, expectedAud, expectedCNonce, expectedClientId
|
|
|
116
116
|
throw new AuthError("auth-oid4vci/wrong-proof-aud",
|
|
117
117
|
"credential issuance: proof JWT aud \"" + payload.aud + "\" mismatch (expected \"" + expectedAud + "\")");
|
|
118
118
|
}
|
|
119
|
-
if (expectedCNonce !== null
|
|
120
|
-
|
|
121
|
-
|
|
119
|
+
if (expectedCNonce !== null) {
|
|
120
|
+
// Constant-time c_nonce compare — secret-shaped value vs
|
|
121
|
+
// attacker-controlled wallet payload. (Audit 2026-05-11.)
|
|
122
|
+
if (typeof payload.nonce !== "string" ||
|
|
123
|
+
!timingSafeEqual(payload.nonce, expectedCNonce)) {
|
|
124
|
+
throw new AuthError("auth-oid4vci/wrong-proof-nonce",
|
|
125
|
+
"credential issuance: proof JWT nonce mismatch (replay defense — wallet must use the c_nonce from the most recent issuer response)");
|
|
126
|
+
}
|
|
122
127
|
}
|
|
123
128
|
if (typeof payload.iat !== "number") {
|
|
124
129
|
throw new AuthError("auth-oid4vci/no-proof-iat",
|
|
@@ -400,8 +405,11 @@ function create(opts) {
|
|
|
400
405
|
"exchangePreAuthorizedCode: tx_code required (offer mandates it)");
|
|
401
406
|
}
|
|
402
407
|
var txHash = sha3Hash("oid4vci-tx:" + eopts.txCode);
|
|
403
|
-
// Constant-time
|
|
404
|
-
|
|
408
|
+
// Constant-time compare on the hashed tx_code (audit 2026-05-11
|
|
409
|
+
// — was `!==` on fixed-width sha3 hex; per CLAUDE.md rule §5
|
|
410
|
+
// every framework-internal compare against attacker-controlled
|
|
411
|
+
// input routes through timingSafeEqual).
|
|
412
|
+
if (!timingSafeEqual(txHash, entry.txCodeHash)) {
|
|
405
413
|
// Don't consume on failure — wallet may be retrying. Operator
|
|
406
414
|
// attaches their own attempt counter / lockout via b.auth.lockout.
|
|
407
415
|
_emitAudit("tx_code_mismatch", "failure", {
|
package/lib/auth/oid4vp.js
CHANGED
|
@@ -452,6 +452,16 @@ function create(opts) {
|
|
|
452
452
|
continue;
|
|
453
453
|
}
|
|
454
454
|
try {
|
|
455
|
+
// Per-presentation vct enforcement (audit 2026-05-11): when
|
|
456
|
+
// DCQL's `vct_values` has 1 entry, `expectedVct` pins it.
|
|
457
|
+
// With 2+ entries the verifier's expectedVct opt can't hold
|
|
458
|
+
// a list, so we verify-without-expected and then validate
|
|
459
|
+
// the actual vct against the DCQL list manually — over-
|
|
460
|
+
// disclosure defense (a holder presenting a vct outside
|
|
461
|
+
// the DCQL filter would previously slip through).
|
|
462
|
+
var dcqlVctValues = cq.meta && Array.isArray(cq.meta.vct_values) ? cq.meta.vct_values : null;
|
|
463
|
+
var expectedVct = dcqlVctValues && dcqlVctValues.length === 1
|
|
464
|
+
? dcqlVctValues[0] : undefined;
|
|
455
465
|
var verified = await sdJwtVcCore().verify(t, {
|
|
456
466
|
issuerKeyResolver: opts.issuerKeyResolver,
|
|
457
467
|
audience: audience,
|
|
@@ -459,9 +469,16 @@ function create(opts) {
|
|
|
459
469
|
requireKeyBinding: true,
|
|
460
470
|
requireKeyAttestation: vopts.requireKeyAttestation === true,
|
|
461
471
|
keyAttestationVerifier: opts.keyAttestationVerifier || null,
|
|
462
|
-
expectedVct:
|
|
463
|
-
? cq.meta.vct_values[0] : undefined,
|
|
472
|
+
expectedVct: expectedVct,
|
|
464
473
|
});
|
|
474
|
+
if (dcqlVctValues && dcqlVctValues.length > 1) {
|
|
475
|
+
if (!verified.claims || dcqlVctValues.indexOf(verified.claims.vct) === -1) {
|
|
476
|
+
verifyErrors.push("vp_token['" + id + "'][" + ti + "] vct '" +
|
|
477
|
+
((verified.claims && verified.claims.vct) || "<missing>") +
|
|
478
|
+
"' is not in DCQL vct_values [" + dcqlVctValues.join(", ") + "]");
|
|
479
|
+
continue;
|
|
480
|
+
}
|
|
481
|
+
}
|
|
465
482
|
presentations.push({
|
|
466
483
|
id: id,
|
|
467
484
|
format: cq.format,
|
package/lib/auth/passkey.js
CHANGED
|
@@ -112,13 +112,39 @@ async function startRegistration(opts) {
|
|
|
112
112
|
return options;
|
|
113
113
|
}
|
|
114
114
|
|
|
115
|
+
function _validateExpectedOrigin(value) {
|
|
116
|
+
if (typeof value === "string") {
|
|
117
|
+
if (value.length === 0) {
|
|
118
|
+
throw new AuthError("auth-passkey/missing-expectedOrigin",
|
|
119
|
+
"expectedOrigin must be a non-empty string or array of strings");
|
|
120
|
+
}
|
|
121
|
+
return;
|
|
122
|
+
}
|
|
123
|
+
if (Array.isArray(value)) {
|
|
124
|
+
if (value.length === 0) {
|
|
125
|
+
throw new AuthError("auth-passkey/missing-expectedOrigin",
|
|
126
|
+
"expectedOrigin array must contain at least one non-empty string");
|
|
127
|
+
}
|
|
128
|
+
for (var i = 0; i < value.length; i += 1) {
|
|
129
|
+
if (typeof value[i] !== "string" || value[i].length === 0) {
|
|
130
|
+
throw new AuthError("auth-passkey/missing-expectedOrigin",
|
|
131
|
+
"expectedOrigin[" + i + "] must be a non-empty string");
|
|
132
|
+
}
|
|
133
|
+
}
|
|
134
|
+
return;
|
|
135
|
+
}
|
|
136
|
+
throw new AuthError("auth-passkey/missing-expectedOrigin",
|
|
137
|
+
"expectedOrigin must be a non-empty string or array of strings");
|
|
138
|
+
}
|
|
139
|
+
|
|
115
140
|
async function verifyRegistration(opts) {
|
|
116
141
|
if (!opts) throw new AuthError("auth-passkey/missing-opts", "opts is required");
|
|
117
142
|
if (!opts.response) {
|
|
118
143
|
throw new AuthError("auth-passkey/missing-response", "opts.response is required");
|
|
119
144
|
}
|
|
120
145
|
_requireString(opts.expectedChallenge, "expectedChallenge");
|
|
121
|
-
|
|
146
|
+
// Multi-origin deployments (web + admin subdomain) need string[].
|
|
147
|
+
_validateExpectedOrigin(opts.expectedOrigin);
|
|
122
148
|
_requireString(opts.expectedRPID, "expectedRPID");
|
|
123
149
|
|
|
124
150
|
var rv = await _vendor().verifyRegistrationResponse({
|
|
@@ -161,12 +187,17 @@ async function verifyRegistration(opts) {
|
|
|
161
187
|
// Credential Management spec: "silent" / "optional" / "required" /
|
|
162
188
|
// "conditional". "conditional" enables passkey autofill on
|
|
163
189
|
// <input autocomplete="webauthn">.
|
|
164
|
-
|
|
190
|
+
// Null-prototype map so `opts.mediation === "__proto__"` /
|
|
191
|
+
// `"constructor"` can't truthy-match an inherited property and slip
|
|
192
|
+
// past the allowlist (audit 2026-05-11).
|
|
193
|
+
var ALLOWED_MEDIATION = Object.assign(Object.create(null),
|
|
194
|
+
{ silent: 1, optional: 1, required: 1, conditional: 1 });
|
|
165
195
|
|
|
166
196
|
async function startAuthentication(opts) {
|
|
167
197
|
if (!opts) throw new AuthError("auth-passkey/missing-opts", "opts is required");
|
|
168
198
|
_requireString(opts.rpId, "rpId");
|
|
169
|
-
if (opts.mediation !== undefined &&
|
|
199
|
+
if (opts.mediation !== undefined &&
|
|
200
|
+
!Object.prototype.hasOwnProperty.call(ALLOWED_MEDIATION, opts.mediation)) {
|
|
170
201
|
throw new AuthError("auth-passkey/bad-mediation",
|
|
171
202
|
"mediation must be one of silent/optional/required/conditional");
|
|
172
203
|
}
|
|
@@ -352,12 +383,40 @@ async function verifyAuthentication(opts) {
|
|
|
352
383
|
throw new AuthError("auth-passkey/missing-response", "opts.response is required");
|
|
353
384
|
}
|
|
354
385
|
_requireString(opts.expectedChallenge, "expectedChallenge");
|
|
355
|
-
|
|
386
|
+
_validateExpectedOrigin(opts.expectedOrigin);
|
|
356
387
|
_requireString(opts.expectedRPID, "expectedRPID");
|
|
357
388
|
if (!opts.credential || !opts.credential.id || !opts.credential.publicKey) {
|
|
358
389
|
throw new AuthError("auth-passkey/missing-credential",
|
|
359
390
|
"opts.credential { id, publicKey, counter? } is required");
|
|
360
391
|
}
|
|
392
|
+
// Counter regression bypass fix (audit 2026-05-11) — pre-v0.9.2
|
|
393
|
+
// shape `opts.credential.counter || 0` silently zeroed an
|
|
394
|
+
// undefined / null / NaN counter, defeating CTAP 2.1 clone-
|
|
395
|
+
// detection on credentials whose stored counter is > 0. An
|
|
396
|
+
// operator who deserialized the credential from a column that
|
|
397
|
+
// dropped the counter would unknowingly accept a cloned
|
|
398
|
+
// authenticator. Require an explicit non-negative integer.
|
|
399
|
+
var counter;
|
|
400
|
+
if (opts.credential.counter === undefined || opts.credential.counter === null) {
|
|
401
|
+
// First-time-stored credentials legitimately have no counter
|
|
402
|
+
// yet (registration ran on a vendor returning 0). Operators
|
|
403
|
+
// MUST persist whatever the vendor returned; if they didn't,
|
|
404
|
+
// refuse rather than silently coerce.
|
|
405
|
+
throw new AuthError("auth-passkey/missing-counter",
|
|
406
|
+
"opts.credential.counter is required (set to 0 at registration; " +
|
|
407
|
+
"store the newCounter returned by verifyAuthentication on every " +
|
|
408
|
+
"successful auth). undefined / null is refused to prevent clone-" +
|
|
409
|
+
"detection bypass when the persisted column is missing.");
|
|
410
|
+
}
|
|
411
|
+
if (typeof opts.credential.counter !== "number" ||
|
|
412
|
+
!isFinite(opts.credential.counter) ||
|
|
413
|
+
opts.credential.counter < 0 ||
|
|
414
|
+
Math.floor(opts.credential.counter) !== opts.credential.counter) {
|
|
415
|
+
throw new AuthError("auth-passkey/bad-counter",
|
|
416
|
+
"opts.credential.counter must be a non-negative integer (got " +
|
|
417
|
+
typeof opts.credential.counter + ")");
|
|
418
|
+
}
|
|
419
|
+
counter = opts.credential.counter;
|
|
361
420
|
|
|
362
421
|
var rv = await _vendor().verifyAuthenticationResponse({
|
|
363
422
|
response: opts.response,
|
|
@@ -367,7 +426,7 @@ async function verifyAuthentication(opts) {
|
|
|
367
426
|
credential: {
|
|
368
427
|
id: opts.credential.id,
|
|
369
428
|
publicKey: opts.credential.publicKey,
|
|
370
|
-
counter:
|
|
429
|
+
counter: counter,
|
|
371
430
|
transports: opts.credential.transports,
|
|
372
431
|
},
|
|
373
432
|
requireUserVerification: opts.requireUserVerification !== false,
|
package/package.json
CHANGED
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.6",
|
|
5
|
-
"serialNumber": "urn:uuid:
|
|
5
|
+
"serialNumber": "urn:uuid:f34cb19c-6c49-42cb-80d5-8727b4fb0f86",
|
|
6
6
|
"version": 1,
|
|
7
7
|
"metadata": {
|
|
8
|
-
"timestamp": "2026-05-
|
|
8
|
+
"timestamp": "2026-05-12T00:20:11.455Z",
|
|
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.9.
|
|
22
|
+
"bom-ref": "@blamejs/core@0.9.3",
|
|
23
23
|
"type": "library",
|
|
24
24
|
"name": "blamejs",
|
|
25
|
-
"version": "0.9.
|
|
25
|
+
"version": "0.9.3",
|
|
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.9.
|
|
29
|
+
"purl": "pkg:npm/%40blamejs/core@0.9.3",
|
|
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.9.
|
|
57
|
+
"ref": "@blamejs/core@0.9.3",
|
|
58
58
|
"dependsOn": []
|
|
59
59
|
}
|
|
60
60
|
]
|