Skip to content

ci(gitleaks): allowlist docfx's generated public/*.min.js bundle and source maps - #353

Merged
Chris-Wolfgang merged 1 commit into
mainfrom
ci/gitleaks-allow-docfx-bundle
Sep 18, 2026
Merged

Chris-Wolfgang merged 1 commit into
mainfrom
ci/gitleaks-allow-docfx-bundle

Conversation

@Chris-Wolfgang

Copy link
Copy Markdown
Owner

gitleaks git scans every ref the full-depth checkout fetched — including gh-pages — and docfx's generated site bundle (public/chunk-*.min.js.map, also under versions/<tag>/public/) trips generic-api-key on embedded constants. Chris-Wolfgang/ETL-Abstractions#533 (README-only) hit 15 such findings right after its v0.25.0 docs deploy, blocking every PR in that repo; this repo will hit the same on its next docs deploy with that docfx version.

Adds the path allowlist from Chris-Wolfgang/repo-template#579. Not a protected file. Note the PR's own gitleaks job reads .gitleaks.toml from main (trusted config), so this PR cannot demonstrate its own fix — it only matters once merged.

🤖 Generated with Claude Code

…source maps

`gitleaks git` scans every ref the full-depth checkout fetched, including
gh-pages; docfx's generated public/chunk-*.min.js.map trips generic-api-key
on embedded constants and blocks every PR once the docs are deployed (seen
first on ETL-Abstractions after its v0.25.0 docs deploy). Same allowlist as
repo-template#579.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 18, 2026 20:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Chris-Wolfgang
Chris-Wolfgang merged commit 6115266 into main Sep 18, 2026
16 checks passed
@Chris-Wolfgang
Chris-Wolfgang deleted the ci/gitleaks-allow-docfx-bundle branch September 18, 2026 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants