10000 Generate `gitleaks.toml` with multiple allowlists by rgmz · Pull Request #1691 · gitleaks/gitleaks · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Generate gitleaks.toml with multiple allowlists #1691

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jan 13, 2025
Merged

Generate gitleaks.toml with multiple allowlists #1691

merged 1 commit into from
Jan 13, 2025

Conversation

rgmz
Copy link
Contributor
@rgmz rgmz commented Jan 10, 2025 •
edited
Loading

Description:

GitLab has upgraded to Gitleaks v8.22, which was the motivation for keeping the older syntax.

Checklist:

  • Does your PR pass tests?
  • Have you written new tests for your changes?
  • Have you lint your code locally prior to submission?

@rgmz rgmz force-pushed the feat/generate-multiple-allowlists branch 4 times, most recently from cce90cf to d850948 Compare January 10, 2025 13:11
@rgmz rgmz marked this pull request as ready for review January 10, 2025 13:12
@rgmz rgmz mentioned this pull request Jan 10, 2025
Merged
3 tasks
@rgmz rgmz force-pushed the feat/generate-multiple-allowlists branch from d850948 to dee8f2f Compare January 13, 2025 14:40
@zricethezav
Copy link
Collaborator

Alright, let's try this once again. I think enough time has passed to give people time to adjust

@zricethezav zricethezav merged commit db8e5e6 into gitleaks:master Jan 13, 2025
1 check passed
@rgmz rgmz deleted the feat/generate-multiple-allowlists branch January 13, 2025 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rgmz @zricethezav
0