Skip to content

Log warning message when file takes a long time#1479

Merged
zricethezav merged 1 commit into
gitleaks:masterfrom
rgmz:feat/slow-warning
May 1, 2025
Merged

Log warning message when file takes a long time#1479
zricethezav merged 1 commit into
gitleaks:masterfrom
rgmz:feat/slow-warning

Conversation

@rgmz
Copy link
Copy Markdown
Contributor

@rgmz rgmz commented Aug 23, 2024

Description:

An earlier version of GitLeaks logged a warning message when a fragment was slow to scan.

This was helpful for identifying problematic files and tuning the ignorelist. I feel like it would still be useful today.

Checklist:

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

Comment thread cmd/root.go Outdated
Comment thread detect/git.go Outdated
@rgmz rgmz force-pushed the feat/slow-warning branch from ab15267 to 21bef90 Compare August 26, 2024 18:53
@rgmz rgmz force-pushed the feat/slow-warning branch 6 times, most recently from d1a1520 to be117a2 Compare October 30, 2024 12:57
@rgmz rgmz force-pushed the feat/slow-warning branch from be117a2 to 830d21b Compare December 6, 2024 00:13
@rgmz rgmz force-pushed the feat/slow-warning branch 4 times, most recently from 16904cd to fd47e2d Compare January 19, 2025 21:12
@rgmz rgmz force-pushed the feat/slow-warning branch from fd47e2d to 2415cc9 Compare April 26, 2025 02:21
@zricethezav zricethezav merged commit 88f56d3 into gitleaks:master May 1, 2025
2 checks passed
@rgmz rgmz deleted the feat/slow-warning branch May 1, 2025 12:42
rgmz added a commit to rgmz/gitleaks that referenced this pull request May 2, 2025
rgmz added a commit to rgmz/gitleaks that referenced this pull request May 2, 2025
alayne222 pushed a commit to alayne222/gitleaks that referenced this pull request May 28, 2025
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