diff --git a/src/rules.ts b/src/rules.ts index 779cf09..d9a03f2 100644 --- a/src/rules.ts +++ b/src/rules.ts @@ -113,7 +113,7 @@ export function checkCSP(headers: RawHeaders): HeaderFinding { // fetch/navigation directive — not just as the first token of default-src/ // script-src. img-src/style-src/font-src/media-src are intentionally omitted // as a wildcard there is low-risk and commonly legitimate. - const wildcardDirectives = ['default-src', 'script-src', 'connect-src', 'form-action', 'frame-src', 'worker-src']; + const wildcardDirectives = ['default-src', 'script-src', 'connect-src', 'form-action', 'frame-src', 'worker-src', 'object-src']; const wildcarded = wildcardDirectives.filter(d => { const sources = extractCspDirective(raw, d); return sources !== undefined && sources.some(isPermissiveSource); diff --git a/test/analyzer.test.ts b/test/analyzer.test.ts index ed22923..9fc6279 100644 --- a/test/analyzer.test.ts +++ b/test/analyzer.test.ts @@ -214,6 +214,23 @@ describe('checkCSP', () => { expect(r.findings.some(f => /Wildcard or bare-scheme/i.test(f))).toBe(false); }); + it('detects wildcard in object-src', () => { + const r = checkCSP({ 'content-security-policy': "default-src 'self'; form-action 'self'; base-uri 'self'; object-src *" }); + expect(r.findings.some(f => /Wildcard.*object-src/i.test(f))).toBe(true); + expect(r.score).toBeLessThan(20); + }); + + it('detects bare scheme in object-src', () => { + const r = checkCSP({ 'content-security-policy': "default-src 'self'; form-action 'self'; base-uri 'self'; object-src https:" }); + expect(r.findings.some(f => /Wildcard or bare-scheme/i.test(f))).toBe(true); + }); + + it("does not flag restrictive object-src 'none'", () => { + const r = checkCSP({ 'content-security-policy': "default-src 'self'; form-action 'self'; base-uri 'self'; object-src 'none'" }); + expect(r.findings.some(f => /object-src/i.test(f))).toBe(false); + expect(r.score).toBe(20); + }); + it('clean CSP returns score 20', () => { const r = checkCSP({ 'content-security-policy': "default-src 'self'; form-action 'self'; base-uri 'self'" }); expect(r.score).toBe(20);