Skip to content
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.

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

Update regexChecks.py #70

Merged
merged 1 commit into from
Dec 10, 2017
Merged

Conversation

carnal0wnage
Copy link

added a few regex additions. The ones with *['|"] weren't working for me but i didnt remove them i just created some additional ones that did work for me

i created a repo with fake secrets to test

https://github.com/carnal0wnage/fake_commited_secrets

@dxa4481 dxa4481 merged commit b6073aa into trufflesecurity:master Dec 10, 2017
@dxa4481
Copy link
Collaborator

dxa4481 commented Dec 10, 2017

I've merged the new ones, thanks for the additions. The ['|"] part ensures that the secret is rapped in quotes. This is to improve signal. A simple example with one of your newer ones, a github URL would likely flag on [g|G][i|I][t|T][h|H][u|U][b|B].*[0-9a-zA-Z]{35,40} such as:

https://github.com/google/googletest/commit/10ef1d9a1e7e6a0b3701d262cf1b889620fdbbc9

To prevent this URL from flagging, we ensure only secrets wrapped in quotes will flag. This means we'll miss out on some real secrets, but we increase our signal to noise ratio in doing so.

@carnal0wnage
Copy link
Author

thanks for the explanation on the ['|"] part. I dont see secrets wrapped in "" too frequently myself. All the ones in that fake repo are ones i found on github or pastebin and they werent matching. I appreciate the merge of the others!

@dxa4481
Copy link
Collaborator

dxa4481 commented Dec 10, 2017

If you can find another way to catch those secrets without increasing noise too much, I'll merge it. Maybe make sure the string is wrapped by an equals or a colon on the left and a newline on the right?

@carnal0wnage
Copy link
Author

cool. if i find a better way i'll do a PR / issue / or an email.

thanks!

@dxa4481
Copy link
Collaborator

dxa4481 commented Jul 26, 2018

@carnal0wnage 👋

dustin-decker pushed a commit that referenced this pull request Apr 3, 2022
)

Bumps [github.com/aws/aws-sdk-go-v2/service/sts](https://github.com/aws/aws-sdk-go-v2) from 1.15.0 to 1.16.0.
- [Release notes](https://github.com/aws/aws-sdk-go-v2/releases)
- [Changelog](https://github.com/aws/aws-sdk-go-v2/blob/main/CHANGELOG.md)
- [Commits](aws/aws-sdk-go-v2@v1.15.0...service/s3/v1.16.0)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go-v2/service/sts
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants