Skip to content

gitleaks:bugfix - updating formatter to gitleaks v8 #1010

Merged
nathanmartinszup merged 1 commit into
mainfrom
bugfix/gitleaks
Mar 7, 2022
Merged

gitleaks:bugfix - updating formatter to gitleaks v8 #1010
nathanmartinszup merged 1 commit into
mainfrom
bugfix/gitleaks

Conversation

@nathanmartinszup

@nathanmartinszup nathanmartinszup commented Mar 3, 2022

Copy link
Copy Markdown
Contributor

When the v8 version of gitaleks was released, the way to do an analysis
and the output of that analysis changed. This pr updates the formatter
to the new format and updates gitleaks to v8.3.0. It also contains some
improvements in the formatter code.

- What I did

- How to verify it

- Description for the changelog

@nathanmartinszup nathanmartinszup changed the title Bugfix/gitleaks gitleaks:bugfix - updating formatter to gitleaks v8 Mar 3, 2022
@nathanmartinszup
nathanmartinszup force-pushed the bugfix/gitleaks branch 3 times, most recently from 21a52c6 to 3e28797 Compare March 4, 2022 13:52
@nathanmartinszup

Copy link
Copy Markdown
Contributor Author

Depends on #1013

@nathanmartinszup
nathanmartinszup force-pushed the bugfix/gitleaks branch 2 times, most recently from 86a192e to a7462c0 Compare March 4, 2022 18:39
@nathanmartinszup
nathanmartinszup marked this pull request as ready for review March 4, 2022 18:58
Comment thread internal/services/formatters/leaks/gitleaks/formatter.go Outdated
When the v8 version of gitaleks was released, the way to do an analysis
and the output of that analysis changed. This pr updates the formatter
to the new format and updates gitleaks to v8.3.0. It also contains some
improvements in the formatter code.

Signed-off-by: Nathan Martins <[email protected]>
@nathanmartinszup
nathanmartinszup merged commit 29c7330 into main Mar 7, 2022
@nathanmartinszup
nathanmartinszup deleted the bugfix/gitleaks branch March 8, 2022 12:40
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.

3 participants