@blamejs/core 0.18.47 → 0.18.48
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 +26 -0
- package/lib/gate-contract.js +146 -0
- package/lib/guard-archive.js +18 -0
- package/lib/guard-auth.js +28 -0
- package/lib/guard-cidr.js +19 -0
- package/lib/guard-country.js +4 -4
- package/lib/guard-csv.js +31 -2
- package/lib/guard-domain.js +22 -0
- package/lib/guard-email.js +17 -0
- package/lib/guard-filename.js +45 -11
- package/lib/guard-graphql.js +16 -0
- package/lib/guard-html.js +17 -0
- package/lib/guard-image.js +19 -0
- package/lib/guard-json.js +23 -0
- package/lib/guard-jsonpath.js +14 -1
- package/lib/guard-jwt.js +20 -2
- package/lib/guard-markdown.js +25 -0
- package/lib/guard-mime.js +15 -5
- package/lib/guard-oauth.js +23 -0
- package/lib/guard-pdf.js +19 -1
- package/lib/guard-regex.js +16 -1
- package/lib/guard-shell.js +14 -1
- package/lib/guard-svg.js +17 -0
- package/lib/guard-template.js +14 -1
- package/lib/guard-text.js +16 -1
- package/lib/guard-time.js +20 -4
- package/lib/guard-uuid.js +28 -4
- package/lib/guard-xml.js +19 -0
- package/lib/guard-yaml.js +22 -2
- package/package.json +1 -1
- package/sbom.cdx.json +6 -6
package/CHANGELOG.md
CHANGED
|
@@ -8,6 +8,32 @@ upgrading across more than a few patches at a time.
|
|
|
8
8
|
|
|
9
9
|
## v0.18.x
|
|
10
10
|
|
|
11
|
+
- v0.18.48 (2026-08-22) — **A guard option you set and a guard option that exists were not the same set.** Every `*Policy` option across the guard family now names the values it takes, and refuses everything else where options are resolved rather than at the first request that trips the scan. Holding the options to their vocabularies surfaced four places where what a guard advertised and what it did had come apart: an option documented under a name nothing read, a value that disabled a critical check without appearing in any opts block, two policies whose settings the gate's floor overrode anyway, and one declared in all three profiles and read nowhere. **Fixed:** *A guard policy refuses a value outside its vocabulary* — The two hundred and sixty-two `*Policy` options across the guard family each declare the values they accept, checked where options are resolved — boot, not the first hostile input — and on every entry point, including the `gate()` and `validate()` of guards that bind their own resolver.
|
|
12
|
+
|
|
13
|
+
The failure it ends is quiet, because policies are read leniently. `duplicateKeyPolicy: "rejct"` is not `allow`, so the check still runs; it is not `reject` either, so the finding drops from critical to a warning. The operator asked to refuse a duplicate key and silently got an audit line. Nothing was let through that a stricter reading would have caught, so this was never a hole — but a setting that means something other than what it says is worth a restart to learn about.
|
|
14
|
+
|
|
15
|
+
`b.gateContract.policyVocabulary(names, values, overrides?)` builds these tables: most of a guard's policies answer the same question — refuse the construct, record it, or let it through — and one or two do not. Written out per option instead, a family of nine ends up with eight that accept `audit-only` and one that does not. Each guard also exports its `POLICY_VOCABULARY`, so a caller can ask what an option takes instead of reading the source. · *An option name the guard does not have is refused rather than ignored* — A `*Policy` key that no guard option matches was kept by the option merge, read by nothing, and left the profile default in force. `b.guardMime` had shipped that way: its opts block named `riskyTypesPolicy` while the code reads `riskyTypePolicy`, so an operator following the documentation and setting it to `reject` kept the balanced profile's `audit` and had risky content types reported rather than refused, with nothing to see but the documentation that was wrong to begin with.
|
|
16
|
+
|
|
17
|
+
A guard's policy set is fully described by its profiles, postures and defaults, so a near-miss is decidable and now refused by name. The rest of the option surface stays open — callbacks, caches and host wiring are not enumerable in advance.
|
|
18
|
+
|
|
19
|
+
The two statements a guard makes about an option — the vocabulary it enforces and the opts block an operator reads — are now compared to each other, which is what surfaced this one. · *`kidTraversalPolicy: "allow"` turned off JWT key-path traversal detection* — `b.guardJwt` documents the `kid` traversal check as unconditional, and the header field it inspects is the one that steers key lookup — a `kid` of `../../etc/passwd` reaching a file-backed resolver is the point of the check. The code consulted `kidTraversalPolicy` for an `allow` its opts block never offered, and that value skipped the check entirely.
|
|
20
|
+
|
|
21
|
+
The check no longer reads a policy, and the option accepts `reject` alone. `algNonePolicy` is likewise fixed at `reject`: `alg=none` was already refused unconditionally and the option had never been read. · *Two guardFilename policies were settings the floor already overrode* — `traversalPolicy` and `nullBytePolicy` accepted `allow` and `audit`, and both were overridden by the refusal floor the gate applies to a leaf filename. An operator setting either believed they had turned a class of finding down; the refusal stood regardless, and the two disagreed in the audit record rather than in the verdict. Both now accept `reject` alone, so the floor is visible in what the guard will take rather than only in what it does.
|
|
22
|
+
|
|
23
|
+
`adsPolicy` keeps `allow` for the one place that honours it — `verifyExtractionPath`, for an operator deliberately extracting stream-suffixed names — and no longer accepts an `audit` that did nothing. · *`svgzPolicy` was declared in every profile and read nowhere* — A gzip-compressed SVG is refused on its byte signature before any parse, and `b.guardSvg.sanitize` throws on one outright. `svgzPolicy` sat in all three profiles without being consulted, so an operator writing `svgzPolicy: "allow"` got a silent no-op and a reasonable belief that SVGZ now passed. The option takes `reject` alone, which is what the guard has always done. · *Three policies do not take `audit-only`, because their code reads the literal `audit`* — `audit-only` is the family's documented synonym for `audit`, and most policies treat them identically because the disposition mapper asks one predicate. Three conditions test for the literal spelling instead, and there the two words took different branches with nothing in the option's name to say which you had picked: `pkcePolicy: "audit-only"` refused every OAuth flow missing a PKCE challenge while `"audit"` served it with a finding; `pollutionPolicy: "audit-only"` removed `__proto__` from the parsed object while `"audit"` preserved it, a difference in the data rather than in a finding; and `pathSeparatorsPolicy: "audit-only"` failed a path-shaped name that `"audit"` accepted.
|
|
24
|
+
|
|
25
|
+
`b.guardOauth.pkcePolicy`, `b.guardJson.pollutionPolicy` and `b.guardFilename.pathSeparatorsPolicy` now refuse `audit-only` rather than accepting it and meaning something else by it — as `b.guardArchive.nestedArchivePolicy`, `b.guardDomain.trailingDotPolicy` and `b.guardYaml.tagPolicy` already did, for the same reason. Write `audit` on those six.
|
|
26
|
+
|
|
27
|
+
A detector holds the pairing: a policy compared against the literal `"audit"` may not also advertise the synonym, so the next one is a failing gate rather than two spellings that quietly disagree. · *Which guards refuse `strip` on a character policy, in full* — 0.18.47 made repair capability a declaration rather than something inferred from the presence of a `sanitize`, and named four guards the inference had been wrong about. Fourteen refuse `strip` on the shared character policies; the list is here because the boot error lands on a caller who had a working configuration, and four names is not the set.
|
|
28
|
+
|
|
29
|
+
Refuse it: `b.guardAuth`, `b.guardCidr`, `b.guardCountry`, `b.guardDomain`, `b.guardGraphql`, `b.guardJsonpath`, `b.guardJwt`, `b.guardMime`, `b.guardOauth`, `b.guardRegex`, `b.guardShell`, `b.guardTemplate`, `b.guardTime`, `b.guardUuid`. Accept it: `b.guardEmail`, `b.guardFilename`, `b.guardHtml`, `b.guardJson`, `b.guardMarkdown`, `b.guardSvg`, `b.guardText`, `b.guardXml`, `b.guardYaml`.
|
|
30
|
+
|
|
31
|
+
All fourteen refuse the character rather than removing it. Feed `b.guardUuid.sanitize` an identifier with a zero-width character in it and it throws `uuid.zero-width`, because a UUID with the character removed is a different identifier — and the same holds for a CIDR block, a country code, a domain, a JSONPath, a media type, a timestamp, a GraphQL document, an OAuth flow and an auth bundle. There is no repaired form to route a `strip` to.
|
|
32
|
+
|
|
33
|
+
`b.guardAuth`, `b.guardOauth` and `b.guardGraphql` are the clearest case for making capability a declaration rather than an inference: each carries a `sanitizeTransform` that is the identity function, because an auth bundle and an OAuth flow are composed of values the framework must not rewrite — forging a JWT `alg` or editing a `state` parameter would disarm the token the guard exists to catch. Inferring "can repair" from the presence of a `sanitize` read all three as repairing.
|
|
34
|
+
|
|
35
|
+
`b.guardCsv`, `b.guardImage` and `b.guardPdf` expose no shared character policy at all. guardCsv repairs characters through its own `bidiCharPolicy`, `controlCharPolicy` and `homoglyphPolicy`, and those still take `strip`. · *Twenty opts-block lines offered a repair the guard cannot perform* — `b.guardUuid`, `b.guardCountry`, `b.guardTime`, `b.guardMime`, `b.guardShell`, `b.guardRegex`, `b.guardJsonpath` and `b.guardTemplate` documented `strip` on their character policies. None of them repairs a character — a UUID with a zero-width character in it is not a UUID with the character removed, it is a different identifier — so the value was refused at construction while the documentation offered it. The lines now name what the guards accept.
|
|
36
|
+
|
|
11
37
|
- v0.18.47 (2026-08-22) — **A misspelled character policy was accepted at boot and quietly meant something else.** `bidiPolicy: "rejct"` was accepted by every door the framework offers — the guard's own option resolver and the gate built from it — and then read leniently at runtime: the scan runs because the value is not `allow`, an issue is raised, and the disposition falls through to refuse. Nothing is let through, so it was never a hole; the operator simply asked for one disposition, got another, and was told nothing at the moment a boot error would have cost them a restart. Ninety-three settings across the guard family behaved that way. **Fixed:** *A character policy refuses a value outside its vocabulary* — The five character-threat policies every guard shares — `bidiPolicy`, `nullBytePolicy`, `controlPolicy`, `zeroWidthPolicy`, `tagsPolicy` — now accept only `allow`, `audit`, `audit-only`, `reject`, and `strip` where the guard can carry a repair out. Anything else is refused where options are resolved, which is boot, not the first request that trips the scan. Every entry point takes the check: the generated resolver, and the `gate()` and `validate()` of the guards that bind their own.
|
|
12
38
|
|
|
13
39
|
The check is derived from each guard's own declared defaults rather than hand-listed per guard, for the same reason the size caps beside it are. Hand-listing is what produced the gap: `b.guardCountry` added this for its own three options after `{ reservedPolicy: "rejcet" }` served a reserved country code and reported success, and the other two hundred and fifty-nine policy options across the family never followed. A guard that declares its own vocabulary explicitly still wins, so that one is unchanged.
|
package/lib/gate-contract.js
CHANGED
|
@@ -2160,6 +2160,75 @@ var CHAR_POLICY_KEYS = Object.freeze([
|
|
|
2160
2160
|
]);
|
|
2161
2161
|
var CHAR_POLICY_BASE = Object.freeze(["allow", "audit", "audit-only", "reject"]);
|
|
2162
2162
|
|
|
2163
|
+
// The vocabularies a guard's own policies keep reaching for. Named because
|
|
2164
|
+
// most of a guard's policies answer the same question — refuse this construct,
|
|
2165
|
+
// record it, or let it through — and because a hand-written array per option
|
|
2166
|
+
// is where a family of nine drops `audit-only` from one of them.
|
|
2167
|
+
var POLICY_VALUES = Object.freeze({
|
|
2168
|
+
// Refuse the construct, record it, or let it through. `audit-only` is the
|
|
2169
|
+
// family's documented synonym for `audit`.
|
|
2170
|
+
rejectAuditAllow: Object.freeze(["reject", "audit", "audit-only", "allow"]),
|
|
2171
|
+
// The same, where the guard can also remove the construct and still produce
|
|
2172
|
+
// a usable document. Only for a finding the sanitizer genuinely repairs:
|
|
2173
|
+
// routing one to a repair that never happens serves the input as sanitized.
|
|
2174
|
+
rejectStripAuditAllow: Object.freeze(["reject", "strip", "audit", "audit-only", "allow"]),
|
|
2175
|
+
});
|
|
2176
|
+
|
|
2177
|
+
/**
|
|
2178
|
+
* @primitive b.gateContract.policyVocabulary
|
|
2179
|
+
* @signature b.gateContract.policyVocabulary(names, values, overrides?)
|
|
2180
|
+
* @since 0.18.48
|
|
2181
|
+
* @status stable
|
|
2182
|
+
* @related b.gateContract.charPolicyEnums, b.gateContract.defineGuard
|
|
2183
|
+
*
|
|
2184
|
+
* An `enumOpts` table where `names` all take `values`, with `overrides` merged
|
|
2185
|
+
* over the top for the options that differ. Most of a guard's policies share
|
|
2186
|
+
* one vocabulary and one or two do not, and writing the shared one out per
|
|
2187
|
+
* option is how a family of nine ends up with eight that accept `audit-only`
|
|
2188
|
+
* and one that does not.
|
|
2189
|
+
*
|
|
2190
|
+
* @example
|
|
2191
|
+
* var POLICY_ENUM = gateContract.policyVocabulary(
|
|
2192
|
+
* ["doctypePolicy", "entityPolicy", "cdataPolicy"],
|
|
2193
|
+
* gateContract.POLICY_VALUES.rejectAuditAllow,
|
|
2194
|
+
* { xmlDsigPolicy: ["audit", "audit-only", "allow"] });
|
|
2195
|
+
* POLICY_ENUM.entityPolicy; // → ["reject", "audit", "audit-only", "allow"]
|
|
2196
|
+
* POLICY_ENUM.xmlDsigPolicy; // → ["audit", "audit-only", "allow"]
|
|
2197
|
+
*/
|
|
2198
|
+
function policyVocabulary(names, values, overrides) {
|
|
2199
|
+
if (!Array.isArray(names) || !Array.isArray(values) || values.length === 0) {
|
|
2200
|
+
throw GateContractError.factory("gate-contract/bad-opt",
|
|
2201
|
+
"gateContract.policyVocabulary: names and values must both be arrays " +
|
|
2202
|
+
"and values must be non-empty");
|
|
2203
|
+
}
|
|
2204
|
+
// Frozen a level down, and from a copy. A guard publishes this table as its
|
|
2205
|
+
// `POLICY_VOCABULARY` while the resolver checks against the same arrays, so
|
|
2206
|
+
// an introspection caller that pushed onto one would be editing the rule, not
|
|
2207
|
+
// reading it: `POLICY_VOCABULARY.symlinkPolicy.push("bogus")` and the guard
|
|
2208
|
+
// accepts "bogus". Copying also keeps the caller's own array theirs.
|
|
2209
|
+
var out = {};
|
|
2210
|
+
var shared = Object.freeze(values.slice());
|
|
2211
|
+
names.forEach(function (name) {
|
|
2212
|
+
out[name] = shared;
|
|
2213
|
+
});
|
|
2214
|
+
if (overrides && typeof overrides === "object") {
|
|
2215
|
+
Object.keys(overrides).forEach(function (name) {
|
|
2216
|
+
// A non-array override would be dropped by the resolver's own
|
|
2217
|
+
// `Array.isArray` test and leave the option unconstrained — the option
|
|
2218
|
+
// would read as declared while accepting anything, which is the state
|
|
2219
|
+
// this whole mechanism exists to end. It is an author mistake, so it
|
|
2220
|
+
// belongs where the author is looking.
|
|
2221
|
+
if (!Array.isArray(overrides[name]) || overrides[name].length === 0) {
|
|
2222
|
+
throw GateContractError.factory("gate-contract/bad-opt",
|
|
2223
|
+
"gateContract.policyVocabulary: override for " + JSON.stringify(name) +
|
|
2224
|
+
" must be a non-empty array of values");
|
|
2225
|
+
}
|
|
2226
|
+
out[name] = Object.freeze(overrides[name].slice());
|
|
2227
|
+
});
|
|
2228
|
+
}
|
|
2229
|
+
return Object.freeze(out);
|
|
2230
|
+
}
|
|
2231
|
+
|
|
2163
2232
|
/**
|
|
2164
2233
|
* @primitive b.gateContract.charPolicyEnums
|
|
2165
2234
|
* @signature b.gateContract.charPolicyEnums(defaults, opts?)
|
|
@@ -2682,10 +2751,77 @@ function resolveProfileAndPosture(opts, cfg) {
|
|
|
2682
2751
|
}
|
|
2683
2752
|
});
|
|
2684
2753
|
}
|
|
2754
|
+
// The same failure with the NAME wrong instead of the value, which is
|
|
2755
|
+
// quieter because there is no value to inspect: the merge keeps the key,
|
|
2756
|
+
// nothing reads it, and the profile default stays in force. An operator who
|
|
2757
|
+
// sets `riskyTypesPolicy: "reject"` on a guard whose option is
|
|
2758
|
+
// `riskyTypePolicy` keeps the balanced profile's "audit" and gets no sign
|
|
2759
|
+
// the setting was dropped.
|
|
2760
|
+
//
|
|
2761
|
+
// Only `*Policy` keys are decidable this way: a guard's policy set is fully
|
|
2762
|
+
// described by its profiles, postures and defaults, so anything else with
|
|
2763
|
+
// that suffix is a near-miss. The rest of the opts surface stays open by
|
|
2764
|
+
// design — callbacks, caches and host wiring are not enumerable in advance.
|
|
2765
|
+
_refuseUnknownPolicyKeys(opts, resolved, cfg, ErrorClass, prefix);
|
|
2685
2766
|
return resolved;
|
|
2686
2767
|
}
|
|
2687
2768
|
|
|
2688
2769
|
|
|
2770
|
+
// A published copy of an enumOpts table, frozen a level down so a caller
|
|
2771
|
+
// reading it cannot reorder the guard's own vocabulary underneath it.
|
|
2772
|
+
function _freezeVocabulary(enumOpts) {
|
|
2773
|
+
var out = {};
|
|
2774
|
+
if (enumOpts && typeof enumOpts === "object") {
|
|
2775
|
+
Object.keys(enumOpts).forEach(function (k) {
|
|
2776
|
+
out[k] = Array.isArray(enumOpts[k]) ? Object.freeze(enumOpts[k].slice()) : enumOpts[k];
|
|
2777
|
+
});
|
|
2778
|
+
}
|
|
2779
|
+
return Object.freeze(out);
|
|
2780
|
+
}
|
|
2781
|
+
|
|
2782
|
+
// Every `*Policy` name this guard answers to: whatever its defaults carry, plus
|
|
2783
|
+
// whatever any profile or posture overlay can introduce, plus the vocabularies
|
|
2784
|
+
// declared for it. The union matters — a policy that only a strict profile sets
|
|
2785
|
+
// is still a real option, and `tagsPolicy` is governed through the family's
|
|
2786
|
+
// derivation without appearing in any defaults block.
|
|
2787
|
+
function _knownPolicyKeys(cfg) {
|
|
2788
|
+
var known = Object.create(null);
|
|
2789
|
+
function absorb(o) {
|
|
2790
|
+
if (!o || typeof o !== "object") return;
|
|
2791
|
+
Object.keys(o).forEach(function (k) {
|
|
2792
|
+
if (/Policy$/.test(k)) known[k] = true;
|
|
2793
|
+
});
|
|
2794
|
+
}
|
|
2795
|
+
absorb(cfg.defaults);
|
|
2796
|
+
absorb(cfg.enumOpts);
|
|
2797
|
+
[cfg.profiles, cfg.compliancePostures].forEach(function (table) {
|
|
2798
|
+
if (!table || typeof table !== "object") return;
|
|
2799
|
+
Object.keys(table).forEach(function (name) { absorb(table[name]); });
|
|
2800
|
+
});
|
|
2801
|
+
return known;
|
|
2802
|
+
}
|
|
2803
|
+
|
|
2804
|
+
// Refuses a caller-supplied `*Policy` the guard has no such option for. Reads
|
|
2805
|
+
// the caller's own opts rather than the merged result, so a guard's internal
|
|
2806
|
+
// wiring is never the thing that fails, and stays silent for a guard that has
|
|
2807
|
+
// no policies at all — there, the suffix carries no meaning to trade on.
|
|
2808
|
+
function _refuseUnknownPolicyKeys(opts, resolved, cfg, ErrorClass, prefix) {
|
|
2809
|
+
if (!opts || typeof opts !== "object") return;
|
|
2810
|
+
var supplied = Object.keys(opts).filter(function (k) {
|
|
2811
|
+
return /Policy$/.test(k) && opts[k] !== undefined;
|
|
2812
|
+
});
|
|
2813
|
+
if (!supplied.length) return;
|
|
2814
|
+
var known = _knownPolicyKeys(cfg);
|
|
2815
|
+
var names = Object.keys(known);
|
|
2816
|
+
if (!names.length) return;
|
|
2817
|
+
for (var i = 0; i < supplied.length; i += 1) {
|
|
2818
|
+
if (known[supplied[i]]) continue;
|
|
2819
|
+
throw ErrorClass.factory(prefix + ".bad-opt",
|
|
2820
|
+
prefix + ": no option named " + JSON.stringify(supplied[i]) +
|
|
2821
|
+
" — this guard's policies are " + names.sort().join(", "));
|
|
2822
|
+
}
|
|
2823
|
+
}
|
|
2824
|
+
|
|
2689
2825
|
// The own enumerable properties of `o` that were actually given a value. A key
|
|
2690
2826
|
// present with the value `undefined` is treated as absent, so it cannot erase
|
|
2691
2827
|
// a default it was never meant to touch.
|
|
@@ -3458,6 +3594,14 @@ function defineGuard(spec) {
|
|
|
3458
3594
|
// "non-negative integer", because derivation cannot tell a cap from a
|
|
3459
3595
|
// tolerance. Exposed so the family tests can tell the two apart.
|
|
3460
3596
|
INT_OPTS: Object.freeze(Array.isArray(spec.intOpts) ? spec.intOpts.slice() : []),
|
|
3597
|
+
// The value each policy option accepts, after the family's character
|
|
3598
|
+
// policies are folded in. Exposed for the same reason as INT_OPTS: the
|
|
3599
|
+
// family sweep has to ask each guard what it takes rather than assume one
|
|
3600
|
+
// vocabulary for all of them. Where a guard pins a policy to a single
|
|
3601
|
+
// value — guardFilename's traversal, guardJwt's alg=none — the answer is a
|
|
3602
|
+
// one-element list, and a sweep that assumed otherwise would report the
|
|
3603
|
+
// pinning as a regression.
|
|
3604
|
+
POLICY_VOCABULARY: _freezeVocabulary(_withCharPolicyEnums(defaults, spec)),
|
|
3461
3605
|
validate: spec.validate,
|
|
3462
3606
|
resolveOpts: _resolveGuardOpts,
|
|
3463
3607
|
buildProfile: buildProfileFn,
|
|
@@ -3821,6 +3965,8 @@ module.exports = {
|
|
|
3821
3965
|
// list cannot drift away from them.
|
|
3822
3966
|
capKeysOf: capKeysOf,
|
|
3823
3967
|
charPolicyEnums: charPolicyEnums,
|
|
3968
|
+
policyVocabulary: policyVocabulary,
|
|
3969
|
+
POLICY_VALUES: POLICY_VALUES,
|
|
3824
3970
|
identitySanitize: identitySanitize,
|
|
3825
3971
|
ctxValueFrom: ctxValueFrom,
|
|
3826
3972
|
runIssueValidator: runIssueValidator,
|
package/lib/guard-archive.js
CHANGED
|
@@ -197,6 +197,20 @@ var INT_OPTS = ["maxEntries", "maxTotalBytes", "maxEntryBytes",
|
|
|
197
197
|
// DERIVED from DEFAULTS rather than written out, because a hand-kept list is
|
|
198
198
|
// what drifted away from the defaults it was meant to mirror and left limits
|
|
199
199
|
// unchecked across the family.
|
|
200
|
+
// Each policy's vocabulary, so a misspelling is a boot error rather than a
|
|
201
|
+
// runtime surprise. Read leniently, a typo takes whichever branch is not the
|
|
202
|
+
// strict one: `symlinkPolicy: "rejct"` is not "allow", so the check runs, and
|
|
203
|
+
// it is not "reject" either, so the finding drops from critical to warn.
|
|
204
|
+
//
|
|
205
|
+
// `audit-only` is deliberately absent, unlike elsewhere in the family:
|
|
206
|
+
// nestedArchivePolicy tests for "audit" exactly, so the synonym would fall
|
|
207
|
+
// past both branches and record nothing at all.
|
|
208
|
+
var POLICY_ENUM = gateContract.policyVocabulary([
|
|
209
|
+
"traversalPolicy", "absolutePathPolicy", "symlinkPolicy", "hardlinkPolicy",
|
|
210
|
+
"nestedArchivePolicy", "duplicateNamePolicy", "caseInsensitiveCollisionPolicy",
|
|
211
|
+
"encryptionPolicy", "sparseEntryPolicy",
|
|
212
|
+
], ["reject", "audit", "allow"]);
|
|
213
|
+
|
|
200
214
|
function _resolveOpts(opts) {
|
|
201
215
|
return gateContract.resolveProfileAndPosture(opts, {
|
|
202
216
|
profiles: PROFILES,
|
|
@@ -206,6 +220,7 @@ function _resolveOpts(opts) {
|
|
|
206
220
|
errCodePrefix: "archive",
|
|
207
221
|
intOpts: INT_OPTS,
|
|
208
222
|
nonNegativeOpts: gateContract.capKeysOf(DEFAULTS),
|
|
223
|
+
enumOpts: POLICY_ENUM,
|
|
209
224
|
});
|
|
210
225
|
}
|
|
211
226
|
|
|
@@ -866,6 +881,9 @@ module.exports = {
|
|
|
866
881
|
// ---- guard-* family registry exports (consumed by b.guardAll) ----
|
|
867
882
|
NAME: "archive",
|
|
868
883
|
KIND: "entries", // archive-entries guard (consumes ctx.entries)
|
|
884
|
+
// The value each policy accepts. Named here rather than derived, because
|
|
885
|
+
// this guard binds its own resolver instead of going through defineGuard.
|
|
886
|
+
POLICY_VOCABULARY: POLICY_ENUM,
|
|
869
887
|
INTEGRATION_FIXTURES: Object.freeze({
|
|
870
888
|
kind: "entries",
|
|
871
889
|
contentType: "application/zip",
|
package/lib/guard-auth.js
CHANGED
|
@@ -393,7 +393,35 @@ var INTEGRATION_FIXTURES = Object.freeze({
|
|
|
393
393
|
// surface (validate / sanitize / bespoke gate) passed through verbatim.
|
|
394
394
|
// The custom KIND ("auth-bundle") is accepted because the bespoke gate
|
|
395
395
|
// reads its own ctx fields (ctx.authBundle / ctx.auth).
|
|
396
|
+
// The policies this wrapper accepts on the wrapped guard's behalf, with the
|
|
397
|
+
// wrapped guard's own vocabulary for each. Derived rather than restated: a
|
|
398
|
+
// second copy of guardOauth's values here would be free to drift from the ones
|
|
399
|
+
// the child actually enforces, and the operator would meet whichever of the
|
|
400
|
+
// two happened to be wrong.
|
|
401
|
+
//
|
|
402
|
+
// Without this, forwarding `codeReusePolicy` through the wrapper failed at the
|
|
403
|
+
// wrapper's own resolver, because the name belongs to the child and this
|
|
404
|
+
// guard's profiles never mention it.
|
|
405
|
+
//
|
|
406
|
+
// The character policies are excluded even though they are forwarded: their
|
|
407
|
+
// vocabulary is derived per guard from whether that guard can repair a
|
|
408
|
+
// character, and this guard cannot. Taking the child's answer would hand this
|
|
409
|
+
// one a `strip` it has no way to carry out the moment the two differ — the
|
|
410
|
+
// substitution that accepting-then-refusing was introduced to stop.
|
|
411
|
+
var CHILD_OWNED_POLICIES = Object.freeze([
|
|
412
|
+
"bidiPolicy", "controlPolicy", "nullBytePolicy", "zeroWidthPolicy", "tagsPolicy",
|
|
413
|
+
]);
|
|
414
|
+
var FORWARDED_POLICY_ENUM = Object.freeze(
|
|
415
|
+
OAUTH_FORWARDED_OPTS.reduce(function (acc, key) {
|
|
416
|
+
if (CHILD_OWNED_POLICIES.indexOf(key) !== -1) return acc;
|
|
417
|
+
var vocabulary = guardOauth.POLICY_VOCABULARY &&
|
|
418
|
+
guardOauth.POLICY_VOCABULARY[key];
|
|
419
|
+
if (Array.isArray(vocabulary)) acc[key] = vocabulary;
|
|
420
|
+
return acc;
|
|
421
|
+
}, {}));
|
|
422
|
+
|
|
396
423
|
var _guard = module.exports = gateContract.defineGuard({
|
|
424
|
+
enumOpts: FORWARDED_POLICY_ENUM,
|
|
397
425
|
name: "auth",
|
|
398
426
|
kind: "auth-bundle",
|
|
399
427
|
errorClass: GuardAuthError,
|
package/lib/guard-cidr.js
CHANGED
|
@@ -494,10 +494,29 @@ var INTEGRATION_FIXTURES = gateContract.identifierFixtures("8.8.8.0/24", "10.0.0
|
|
|
494
494
|
// standard serve -> audit-only -> refuse chain — reading ctx.identifier ||
|
|
495
495
|
// ctx.cidr via ctxFields. No sanitize action: an allowlist gate never
|
|
496
496
|
// rewrites the operator's stored network range.
|
|
497
|
+
// Each policy's vocabulary, so a misspelling is a boot error rather than a
|
|
498
|
+
// runtime surprise. Read leniently, a typo takes whichever branch is not the
|
|
499
|
+
// strict one: `reservedRangesPolicy: "rejct"` is not "allow", so the check
|
|
500
|
+
// runs, and it is not "reject" either, so the finding drops to a warning — the
|
|
501
|
+
// operator asked to refuse a reserved range and silently got an audit.
|
|
502
|
+
//
|
|
503
|
+
// requireMaskPolicy carries its own spellings because the question is about the
|
|
504
|
+
// mask rather than a threat disposition. `audit-only` rides with `audit` on the
|
|
505
|
+
// three that take one, matching how the framework treats the two everywhere
|
|
506
|
+
// else.
|
|
507
|
+
var POLICY_ENUM = gateContract.policyVocabulary([
|
|
508
|
+
"networkAlignmentPolicy", "reservedRangesPolicy", "ipv4MappedIpv6Policy",
|
|
509
|
+
], gateContract.POLICY_VALUES.rejectAuditAllow, {
|
|
510
|
+
// Names its own values because the finding is an absence — a bare address
|
|
511
|
+
// where a mask was expected — so "reject" alone would not say what of.
|
|
512
|
+
requireMaskPolicy: ["allow-bare-ip", "audit-bare-ip", "reject-bare-ip"],
|
|
513
|
+
});
|
|
514
|
+
|
|
497
515
|
module.exports = gateContract.defineGuard({
|
|
498
516
|
name: "cidr",
|
|
499
517
|
kind: "identifier",
|
|
500
518
|
errorClass: GuardCidrError,
|
|
519
|
+
enumOpts: POLICY_ENUM,
|
|
501
520
|
profiles: PROFILES,
|
|
502
521
|
base: 128,
|
|
503
522
|
integrationFixtures: INTEGRATION_FIXTURES,
|
package/lib/guard-country.js
CHANGED
|
@@ -348,10 +348,10 @@ function _detectIssues(input, opts) {
|
|
|
348
348
|
* @opts
|
|
349
349
|
* profile: "strict"|"balanced"|"permissive",
|
|
350
350
|
* compliancePosture: "hipaa"|"pci-dss"|"gdpr"|"soc2",
|
|
351
|
-
* bidiPolicy: "reject"|"
|
|
352
|
-
* controlPolicy: "reject"|"
|
|
353
|
-
* nullBytePolicy: "reject"|"
|
|
354
|
-
* zeroWidthPolicy: "reject"|"
|
|
351
|
+
* bidiPolicy: "reject"|"audit"|"allow",
|
|
352
|
+
* controlPolicy: "reject"|"audit"|"allow",
|
|
353
|
+
* nullBytePolicy: "reject"|"audit"|"allow",
|
|
354
|
+
* zeroWidthPolicy: "reject"|"audit"|"allow",
|
|
355
355
|
* reservedPolicy: "reject"|"audit"|"allow",
|
|
356
356
|
* userAssignedPolicy: "reject"|"audit"|"allow",
|
|
357
357
|
* formerlyUsedPolicy: "reject"|"audit"|"allow",
|
package/lib/guard-csv.js
CHANGED
|
@@ -639,7 +639,7 @@ function _stripIssues(text, opts) {
|
|
|
639
639
|
* twice does not stack prefixes.
|
|
640
640
|
*
|
|
641
641
|
* @opts
|
|
642
|
-
* formulaInjectionPolicy: "prefix-tab"|"prefix-quote"|"wrap-with-quotes-and-prefix"|"reject"|"allowlist",
|
|
642
|
+
* formulaInjectionPolicy: "prefix-tab"|"prefix-quote"|"wrap-with-quotes-and-prefix"|"reject"|"allowlist"|"audit-only"|"allow",
|
|
643
643
|
* formulasAllowlist: string[], // when policy === "allowlist"
|
|
644
644
|
* bidiCharPolicy: "reject"|"strip"|"audit"|"allow",
|
|
645
645
|
* controlCharPolicy: "reject"|"strip"|"allow",
|
|
@@ -1047,7 +1047,7 @@ function serialize(rows, opts) {
|
|
|
1047
1047
|
* controlCharPolicy: "reject"|"strip"|"allow",
|
|
1048
1048
|
* nullByteHandling: "reject"|"strip"|"allow",
|
|
1049
1049
|
* homoglyphPolicy: "audit"|"strip"|"allow",
|
|
1050
|
-
* formulaInjectionPolicy: "prefix-tab"|"prefix-quote"|"wrap-with-quotes-and-prefix"|"reject"|"audit-only"|"allow",
|
|
1050
|
+
* formulaInjectionPolicy: "prefix-tab"|"prefix-quote"|"wrap-with-quotes-and-prefix"|"reject"|"allowlist"|"audit-only"|"allow",
|
|
1051
1051
|
* dangerousFunctions: string[],
|
|
1052
1052
|
* dialectPolicy: "strict"|"permissive",
|
|
1053
1053
|
*
|
|
@@ -1409,7 +1409,36 @@ var INTEGRATION_FIXTURES = Object.freeze({
|
|
|
1409
1409
|
// extras (serialize / escapeCell / detect / schema / FORMULA_PREFIXES /
|
|
1410
1410
|
// DANGEROUS_FUNCTIONS) passed through verbatim. The bespoke `gate` carries
|
|
1411
1411
|
// CSV's sanitize-reparse-reserialize chain unchanged.
|
|
1412
|
+
// Each policy's vocabulary, so a misspelling is a boot error rather than a
|
|
1413
|
+
// runtime surprise. Read leniently, a typo takes whichever branch is not the
|
|
1414
|
+
// strict one: `formulaInjectionPolicy: "prefix-tabb"` is neither "allow" nor
|
|
1415
|
+
// "audit-only", so a mitigation is attempted, and it is not a spelling the
|
|
1416
|
+
// mitigation recognises — the operator asked for a tab prefix and got something
|
|
1417
|
+
// else without being told.
|
|
1418
|
+
//
|
|
1419
|
+
// The sets are the UNION of what the code compares, what the opts blocks
|
|
1420
|
+
// document and what the shipped profiles select, because no single source was
|
|
1421
|
+
// complete: the two opts blocks disagreed with each other, one listing
|
|
1422
|
+
// `allowlist` and the other `audit-only` / `allow`, while the code compares all
|
|
1423
|
+
// three. Both blocks now carry the full set.
|
|
1424
|
+
//
|
|
1425
|
+
// The last four are MODE settings rather than threat dispositions — a dialect,
|
|
1426
|
+
// a numeric rendering, whether whitespace is kept — so they share nothing with
|
|
1427
|
+
// the disposition vocabulary and are listed on their own terms.
|
|
1428
|
+
var POLICY_ENUM = Object.freeze({
|
|
1429
|
+
formulaInjectionPolicy: ["prefix-tab", "prefix-quote", "wrap-with-quotes-and-prefix",
|
|
1430
|
+
"reject", "allowlist", "audit-only", "allow"],
|
|
1431
|
+
bidiCharPolicy: ["reject", "strip", "audit", "allow"],
|
|
1432
|
+
controlCharPolicy: ["reject", "strip", "allow"],
|
|
1433
|
+
homoglyphPolicy: ["audit", "strip", "allow"],
|
|
1434
|
+
trailingWhitespacePolicy: ["trim", "preserve", "reject"],
|
|
1435
|
+
dialectPolicy: ["strict", "permissive"],
|
|
1436
|
+
numericPrecisionPolicy: ["decimal-string-above-safe-int", "scientific", "reject-bigint"],
|
|
1437
|
+
piiPolicy: ["preserve", "redact"],
|
|
1438
|
+
});
|
|
1439
|
+
|
|
1412
1440
|
module.exports = gateContract.defineGuard({
|
|
1441
|
+
enumOpts: POLICY_ENUM,
|
|
1413
1442
|
name: "csv",
|
|
1414
1443
|
kind: "content",
|
|
1415
1444
|
errorClass: GuardCsvError,
|
package/lib/guard-domain.js
CHANGED
|
@@ -657,7 +657,29 @@ var INTEGRATION_FIXTURES = gateContract.identifierFixtures("example.com", "192.1
|
|
|
657
657
|
// standard serve -> audit-only -> refuse chain — reading ctx.identifier ||
|
|
658
658
|
// ctx.domain via ctxFields. No sanitize action: an allowlist gate never
|
|
659
659
|
// rewrites the operator's stored allowlist key.
|
|
660
|
+
// Each policy's vocabulary, taken from this guard's own documented opts above,
|
|
661
|
+
// so a misspelling is a boot error rather than a runtime surprise. Read
|
|
662
|
+
// leniently, a typo takes whichever branch is not the strict one:
|
|
663
|
+
// `punycodePolicy: "rejct"` is not "allow", so the check runs, and it is not
|
|
664
|
+
// "reject" either, so the finding drops to a warning — the operator asked to
|
|
665
|
+
// refuse punycode and silently got an audit.
|
|
666
|
+
//
|
|
667
|
+
// trailingDotPolicy is deliberately different on both counts: it takes
|
|
668
|
+
// `normalize` rather than `allow`, and it does NOT take `audit-only`, because
|
|
669
|
+
// its check compares `=== "audit"` exactly and the synonym would not be
|
|
670
|
+
// honoured. A shared list would have advertised it.
|
|
671
|
+
var POLICY_ENUM = gateContract.policyVocabulary([
|
|
672
|
+
"ldhPolicy", "punycodePolicy", "mixedScriptPolicy", "specialUsePolicy",
|
|
673
|
+
"ipLiteralPolicy", "wildcardPolicy", "singleLabelPolicy", "underscorePolicy",
|
|
674
|
+
"dgaPolicy",
|
|
675
|
+
], gateContract.POLICY_VALUES.rejectAuditAllow, {
|
|
676
|
+
// No `allow`, and no `audit-only`: a trailing dot is either removed or
|
|
677
|
+
// reported, and the code compares for "audit" exactly.
|
|
678
|
+
trailingDotPolicy: ["normalize", "audit", "reject"],
|
|
679
|
+
});
|
|
680
|
+
|
|
660
681
|
module.exports = gateContract.defineGuard({
|
|
682
|
+
enumOpts: POLICY_ENUM,
|
|
661
683
|
name: "domain",
|
|
662
684
|
kind: "identifier",
|
|
663
685
|
errorClass: GuardDomainError,
|
package/lib/guard-email.js
CHANGED
|
@@ -1115,7 +1115,24 @@ var INTEGRATION_FIXTURES = Object.freeze({
|
|
|
1115
1115
|
// the address/message entries (validateAddress / validateMessage) passed
|
|
1116
1116
|
// through verbatim. The bespoke `gate` validates via validateMessage and
|
|
1117
1117
|
// carries the serve->audit-only->refuse chain unchanged.
|
|
1118
|
+
// Each policy's vocabulary, so a misspelling is a boot error rather than a
|
|
1119
|
+
// runtime surprise. Read leniently, a typo takes whichever branch is not the
|
|
1120
|
+
// strict one: `crlfHeaderInjectionPolicy: "rejct"` is not "allow", so the
|
|
1121
|
+
// check runs, and it is not "reject" either, so a header-injection attempt
|
|
1122
|
+
// drops to a warning.
|
|
1123
|
+
//
|
|
1124
|
+
// The character policies are absent on purpose — they are derived for the
|
|
1125
|
+
// whole guard family, and an entry here would shadow that derivation.
|
|
1126
|
+
var POLICY_ENUM = gateContract.policyVocabulary([
|
|
1127
|
+
"multiAtPolicy", "ipLiteralPolicy", "punycodePolicy", "mixedScriptPolicy",
|
|
1128
|
+
"addressCommentPolicy", "crlfHeaderInjectionPolicy", "bareCrPolicy",
|
|
1129
|
+
"bareLfPolicy", "smtpSmugglingPolicy", "displayNameSpoofPolicy",
|
|
1130
|
+
], gateContract.POLICY_VALUES.rejectAuditAllow, {
|
|
1131
|
+
bomPolicy: gateContract.POLICY_VALUES.rejectStripAuditAllow,
|
|
1132
|
+
});
|
|
1133
|
+
|
|
1118
1134
|
module.exports = gateContract.defineGuard({
|
|
1135
|
+
enumOpts: POLICY_ENUM,
|
|
1119
1136
|
name: "email",
|
|
1120
1137
|
kind: "content",
|
|
1121
1138
|
charRepair: true,
|
package/lib/guard-filename.js
CHANGED
|
@@ -399,7 +399,9 @@ function _detectIssues(input, opts) {
|
|
|
399
399
|
// by the maxBytes check at the end of this function (issues are
|
|
400
400
|
// reported all-at-once; an oversized name still gets traversal-shape
|
|
401
401
|
// detection so operators see the full failure surface).
|
|
402
|
-
|
|
402
|
+
// Unconditional, as the opts block has always said: a leaf name has no
|
|
403
|
+
// legitimate `..` component, so there is no policy value that turns this off.
|
|
404
|
+
{
|
|
403
405
|
if (_hasTraversalSegment(name) || name === "..") {
|
|
404
406
|
issues.push({
|
|
405
407
|
kind: "path-traversal", severity: "critical",
|
|
@@ -457,8 +459,10 @@ function _detectIssues(input, opts) {
|
|
|
457
459
|
}
|
|
458
460
|
}
|
|
459
461
|
|
|
460
|
-
// 6. NTFS alternate data streams — `name:stream`.
|
|
461
|
-
|
|
462
|
+
// 6. NTFS alternate data streams — `name:stream`. Unconditional: a write to
|
|
463
|
+
// `name:stream` lands on a hidden stream of the base file rather than the
|
|
464
|
+
// file the caller named, which no policy value makes safe.
|
|
465
|
+
{
|
|
462
466
|
if (_hasAdsSuffix(name) && name.charAt(0) !== "/") {
|
|
463
467
|
// Only flag when there's a `:` followed by stream-name characters
|
|
464
468
|
// and we're NOT at the start (relative path indicator).
|
|
@@ -616,11 +620,9 @@ function _sanitize(input, opts) {
|
|
|
616
620
|
if (opts.unicodeNormalization === "NFC") name = _normalizeNFC(name);
|
|
617
621
|
|
|
618
622
|
// Reject path traversal even in sanitize — there's no safe sanitization.
|
|
619
|
-
if (
|
|
620
|
-
|
|
621
|
-
|
|
622
|
-
throw _err("filename.traversal", "filename contains path-traversal sequence");
|
|
623
|
-
}
|
|
623
|
+
if (_hasTraversalSegment(name) || _hasAnyFolded(name, PERCENT_ENCODED_TRAVERSALS) ||
|
|
624
|
+
name === "." || name === "..") {
|
|
625
|
+
throw _err("filename.traversal", "filename contains path-traversal sequence");
|
|
624
626
|
}
|
|
625
627
|
if (_hasUncPrefix(name)) {
|
|
626
628
|
throw _err("filename.unc", "UNC path syntax");
|
|
@@ -659,7 +661,7 @@ function _sanitize(input, opts) {
|
|
|
659
661
|
}
|
|
660
662
|
|
|
661
663
|
// ADS detection.
|
|
662
|
-
if (
|
|
664
|
+
if (_hasAdsSuffix(name)) {
|
|
663
665
|
throw _err("filename.ntfs-ads", "filename contains NTFS alternate data stream syntax");
|
|
664
666
|
}
|
|
665
667
|
|
|
@@ -707,7 +709,9 @@ function _sanitize(input, opts) {
|
|
|
707
709
|
* traversalPolicy: "reject", // always reject
|
|
708
710
|
* reservedCharPolicy: "reject"|"strip"|"allow",
|
|
709
711
|
* reservedNamePolicy: "reject"|"audit"|"allow",
|
|
710
|
-
* adsPolicy: "reject",
|
|
712
|
+
* adsPolicy: "reject"|"allow", // reject here; "allow"
|
|
713
|
+
* // is honoured only
|
|
714
|
+
* // by verifyExtractionPath
|
|
711
715
|
* leadingTrailingPolicy: "reject"|"strip"|"allow",
|
|
712
716
|
* shellExecExtPolicy: "reject"|"audit"|"allow",
|
|
713
717
|
* pathSeparatorsPolicy: "reject"|"audit"|"allow",
|
|
@@ -1081,7 +1085,7 @@ var PATH_MAX_BYTES = 4096;
|
|
|
1081
1085
|
* @opts
|
|
1082
1086
|
* followSymlinks: boolean, // default false — symlink in the
|
|
1083
1087
|
* // resolved path refuses unless set
|
|
1084
|
-
* reservedNamePolicy: string, // "allow" opts out of the Windows
|
|
1088
|
+
* reservedNamePolicy: string, // "allow" opts out of the Windows-
|
|
1085
1089
|
* // reserved-device-name segment check
|
|
1086
1090
|
* adsPolicy: string, // "allow" opts out of the NTFS-ADS check
|
|
1087
1091
|
* leadingTrailingPolicy: string, // "allow" opts out of the trailing-dot /
|
|
@@ -1303,7 +1307,37 @@ var INTEGRATION_FIXTURES = Object.freeze({
|
|
|
1303
1307
|
// factory only supplies the error class, registry exports, buildProfile /
|
|
1304
1308
|
// compliancePosture / loadRulePack wiring, and the verifyExtractionPath /
|
|
1305
1309
|
// WIN_RESERVED_NAMES / SHELL_EXEC_EXTS extras.
|
|
1310
|
+
// Each policy's vocabulary, so a misspelling is a boot error rather than a
|
|
1311
|
+
// runtime surprise. Read leniently, a typo takes whichever branch is not the
|
|
1312
|
+
// strict one: `reservedNamePolicy: "rejct"` is not "allow", so the check runs,
|
|
1313
|
+
// and it is not "reject" either, so the finding drops to a warning.
|
|
1314
|
+
//
|
|
1315
|
+
// `traversalPolicy` and `nullBytePolicy` take one value each, as the opts block
|
|
1316
|
+
// has always said: a leaf name has no legitimate `..` component, and a NUL
|
|
1317
|
+
// truncates the name at whichever consumer reads it first. Both were reading
|
|
1318
|
+
// `"allow"` to turn the check off — a value neither of them offered, and one
|
|
1319
|
+
// the gate's floor overrode anyway, so it disabled the finding while the
|
|
1320
|
+
// refusal stood.
|
|
1321
|
+
//
|
|
1322
|
+
// `adsPolicy` keeps two, for a narrower reason: the detection is likewise a
|
|
1323
|
+
// floor, but `verifyExtractionPath` honours `"allow"` for an operator who is
|
|
1324
|
+
// deliberately extracting stream-suffixed names. `"audit"` was accepted there
|
|
1325
|
+
// and did nothing.
|
|
1326
|
+
var POLICY_ENUM = gateContract.policyVocabulary([
|
|
1327
|
+
"homoglyphPolicy", "reservedNamePolicy", "shellExecExtPolicy",
|
|
1328
|
+
], gateContract.POLICY_VALUES.rejectAuditAllow, {
|
|
1329
|
+
// No `audit-only`: the separator check suppresses its finding on the literal
|
|
1330
|
+
// "audit", so the synonym would fail a path-shaped name that "audit" accepts.
|
|
1331
|
+
pathSeparatorsPolicy: ["reject", "audit", "allow"],
|
|
1332
|
+
traversalPolicy: ["reject"],
|
|
1333
|
+
nullBytePolicy: ["reject"],
|
|
1334
|
+
adsPolicy: ["reject", "allow"],
|
|
1335
|
+
reservedCharPolicy: ["reject", "strip", "allow"],
|
|
1336
|
+
leadingTrailingPolicy: ["reject", "strip", "allow"],
|
|
1337
|
+
});
|
|
1338
|
+
|
|
1306
1339
|
module.exports = gateContract.defineGuard({
|
|
1340
|
+
enumOpts: POLICY_ENUM,
|
|
1307
1341
|
name: "filename",
|
|
1308
1342
|
kind: "filename",
|
|
1309
1343
|
charRepair: true,
|
package/lib/guard-graphql.js
CHANGED
|
@@ -626,7 +626,23 @@ var INTEGRATION_FIXTURES = Object.freeze({
|
|
|
626
626
|
// surface (validate / sanitize / bespoke gate) passed through verbatim.
|
|
627
627
|
// The custom KIND ("graphql-request") is accepted because the bespoke
|
|
628
628
|
// gate reads its own ctx fields (ctx.graphqlRequest / ctx.gql).
|
|
629
|
+
// Each policy's vocabulary, so a misspelling is a boot error rather than a
|
|
630
|
+
// runtime surprise. Read leniently, a typo takes whichever branch is not the
|
|
631
|
+
// strict one: `introspectionPolicy: "rejct"` is not "allow", so the check
|
|
632
|
+
// runs, and it is not "reject" either, so the finding drops to a warning.
|
|
633
|
+
//
|
|
634
|
+
// `persistedQueryPolicy` opens with "require" rather than "reject" because the
|
|
635
|
+
// finding is an absence: the operator is demanding a persisted-query id, not
|
|
636
|
+
// refusing a construct that appeared.
|
|
637
|
+
var POLICY_ENUM = gateContract.policyVocabulary([
|
|
638
|
+
"introspectionPolicy", "operationNamePolicy", "batchPolicy",
|
|
639
|
+
"aliasBombPolicy", "depthPolicy", "variableShapePolicy",
|
|
640
|
+
], gateContract.POLICY_VALUES.rejectAuditAllow, {
|
|
641
|
+
persistedQueryPolicy: ["require", "audit", "audit-only", "allow"],
|
|
642
|
+
});
|
|
643
|
+
|
|
629
644
|
var _guard = module.exports = gateContract.defineGuard({
|
|
645
|
+
enumOpts: POLICY_ENUM,
|
|
630
646
|
name: "graphql",
|
|
631
647
|
kind: "graphql-request",
|
|
632
648
|
errorClass: GuardGraphqlError,
|
package/lib/guard-html.js
CHANGED
|
@@ -980,7 +980,24 @@ var INTEGRATION_FIXTURES = Object.freeze({
|
|
|
980
980
|
// (escapeText / escapeAttr / wcag + the tag/attr/scheme/clobber tables)
|
|
981
981
|
// passed through verbatim. The bespoke `gate` carries HTML's
|
|
982
982
|
// sanitize-and-reemit chain unchanged.
|
|
983
|
+
// Each policy's vocabulary, so a misspelling is a boot error rather than a
|
|
984
|
+
// runtime surprise. Read leniently, a typo takes whichever branch is not the
|
|
985
|
+
// strict one: `cssPolicy: "rejct"` is not "allow", so the check runs, and it
|
|
986
|
+
// is not "reject" either, so a dangerous style attribute drops to a warning.
|
|
987
|
+
//
|
|
988
|
+
// The opts block types these as `string` rather than naming the values; these
|
|
989
|
+
// are the values the profiles use and the code compares against.
|
|
990
|
+
//
|
|
991
|
+
// The character policies are absent on purpose — they are derived for the
|
|
992
|
+
// whole guard family, and an entry here would shadow that derivation.
|
|
993
|
+
var POLICY_ENUM = gateContract.policyVocabulary([
|
|
994
|
+
"cssPolicy", "domClobberPolicy",
|
|
995
|
+
], gateContract.POLICY_VALUES.rejectStripAuditAllow, {
|
|
996
|
+
mxssHintPolicy: gateContract.POLICY_VALUES.rejectAuditAllow,
|
|
997
|
+
});
|
|
998
|
+
|
|
983
999
|
module.exports = gateContract.defineGuard({
|
|
1000
|
+
enumOpts: POLICY_ENUM,
|
|
984
1001
|
name: "html",
|
|
985
1002
|
kind: "content",
|
|
986
1003
|
// sanitize rewrites the document, so `strip` on a character policy is an
|