Merged
Conversation
* Closes #544 Co-authored-by: mercuriete <mercuriete@gmail.com>
* adding pypi and basic azure rules * Adding new github format
* Docker-based pre-commit configuration example * Docker-based pre-commit configuration example
* add global file check * rm whitespace
* Add pre-commit support * Update README.md
* Pass the entropy data back to the Leak struct Do this to make it easier to tune entropy checks and make decisions in systems consuming the output. ~ B'ezrat Hashem ~ * Return negative number when entropy not checked That way you can tell the difference between not checking or an actual entropy level of 0 ~ B'ezrat Hashem ~ * Make sure to handle range checks properly Make sure to show when something had an entropy returned but was outside range, or didn't have a hit at all, etc... ~ B'zrat Hashem ~ * Add a few doc strings Follow the project's conventiona add a comment above the methods ~ B'ezrat Hashem ~ * Update tests and get them to pass ~ B'ezrat Hashem ~ * Remove checked in `.got` files ~ B'ezrat Hashem ~ * Add `*.got` to the `.gitignore` Make sure the test output files aren't checked-in ~ B'ezrat Hashem ~
* Add support for custom username when using SSH auth Previously, the user "git" was hard-coded This commit also adds support for ssh:// URLs * Add comment to explain username parsing
Signed-off-by: Norman Ziegner <norman.ziegner@ufz.de>
Regexps for default Twitter rules ("Twitter Secret Key" and "Twitter Client
ID") have a small flaw that make the default configuration vulnerable to
some false-positives.
I believe these rules should detect the cases like (SOME_CLIENT_ID should
be longer):
```
"twitter_client_id": "SOME_CLIENT_ID"
```
However, currently the twitter rules also detect the false positives for the
cases like:
```
someObj := twitter.NewObjectWithALongName()
config.Twitter.DomainAccessToken
```
I'm trying to address this issue the similar way it's done for facebook client
ids and AWS secret keys, where the secret is expected to be quoted.
* Updating alpine, use gitleaks user instead of root * remove comments
* drop travis * remove travis * rename, use example * typo * different syntax * rename to test * split test and build into two jobs * add gosec job * drop gosec for now
* fix premature exit on nogit scan, actually limit concurreny for nogit * removing files scanned log
* no-git support fingerprint support * updating gitleaksignore w/ no-git false positives * fix test
* bump gitdiff, add git.Err state, better log messages * remove cmd.Start * forgot to start...
* Add grafana tokens rules * Adding upper bound limits to Grafana tokens
pretty output
* refactor: more precise rule for private keys The current regex didn't match PGP private keys anymore, since they start with `BEGIN PGP PRIVATE KEY BLOCK` and the `BLOCK` never matched for the existing regex. I've made that part optional so that all strings matching for the current regex will still match for the new regex. * refactor: more precise rule for private keys Co-authored-by: Fabian F Groß <fabian.f.gross@deutschebahn.com>
* Add baseline * Update doc, add error, move baseline to detect namespace, ignore findings instead of reactively filter them out * Update detect/detect.go Co-authored-by: Zachary Rice <zricezrice@gmail.com> * Update IsNew function (no check on tags - omit finger print check) * Update README.md Co-authored-by: Zachary Rice <zricezrice@gmail.com> * Update examples in readme to make it ensure it's clear that a baseline is indeed a gitleaks report * Fix test - updated tags doesn't make a finding new * Add missing err assignment * Allow scanner to continue without baseline if file is malformed * Fix typo in comment * Fix control flow err. (Real life testing) * Fix wording * Auto-ignore baseline path
* Adding quiet mode to silence banner * Changing flag description. Adding flag to README * Updating argument name * updating variable name to aline with argument * fixing readme spacing * Fixing variable name
* silence warning about unchecked errors * go-fmt change to add newline * Zerolog requires you to always call .Msg() When logging with zerolog, you need to always end with .Msg(), even if you just pass an empty string. If you read the README on https://github.com/rs/zerolog, they write: > It is very important to note that when using the zerolog > chaining API, as shown above (log.Info().Msg("hello world"), the > chain must have either the Msg or Msgf method call. If you > forget to add either of these, the log will not occur and there > is no compile time error to alert you of this. * Create empty slice without literal * Fix variable / package name collision with literal instead of having a variable named "config", which collides with the package name "config", just pass a literal config.Config{} struct to the function * Replace call to deprecated ioutil.ReadAll() Use io.ReadAll() instead * Check error when closing jsonFile Make it a warning and log error
* upgrade go version to 1.19 * upgrade go version to 1.19 in dockerfile and test.yml
Dacpac refactorlogs contains Key's that are false positives. This commit excludes those files.
Thanks for gitleaks, its amazing to have it in the pipeline so I can have some extra peace of mind!
* Add detection rules for DigitalOcean tokens * go fmt correction
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description:
Explain the purpose of the PR.
Checklist: