search

gitleaks / gitleaks-action

Protect your secrets using Gitleaks-Action
https://gitleaks.io
Other
320 stars 123 forks source link

Use eventJson.commits for commit range rather than before and after #65

Closed zricethezav closed 2 years ago

zricethezav commented 2 years ago

@weineran would you mind reviewing this PR?

This change simplifies the log-opts logic and fixes a edge case where rebased commits on the "before" commit would cause an "invalid revision range" error as seen here: https://github.com/zricethezav/gitleaks/runs/7055060039?check_suite_focus=true

zricethezav commented 2 years ago

gitleaks/gitleaks(push) is failing because I rebased 3 commits and the action is using the code on the default branch, after this change has been merged we won't see this error anymore.