Skip to content

Added a .gitleaks.toml to mark some test-keys as 'ok'.#979

Merged
ggainey merged 1 commit intopulp:mainfrom
ggainey:gitleaks
Aug 26, 2025
Merged

Added a .gitleaks.toml to mark some test-keys as 'ok'.#979
ggainey merged 1 commit intopulp:mainfrom
ggainey:gitleaks

Conversation

@ggainey
Copy link
Copy Markdown
Contributor

@ggainey ggainey commented Aug 25, 2025

No description provided.

Comment thread .gitleaks.toml
@@ -0,0 +1,7 @@
[allowlist]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should this go into the templated section too, as we write these keys into all the plugins branches?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That's probably a good idea - but the first step is "stop the bleeding" here. Also, GitLeaks only checks the main branch, iirc, so we won't get warnings for the supported ones.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Well, this repository doesn't have "supported branches".

@ggainey ggainey enabled auto-merge (rebase) August 26, 2025 12:49
@ggainey ggainey merged commit 8cce0f9 into pulp:main Aug 26, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants