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

Adding experimental configuration that lets you set a base64 entropy … #192

Closed
wants to merge 1 commit into from

Conversation

michaellihs
Copy link
Contributor

@michaellihs michaellihs commented May 28, 2020

We discovered, that Talisman did not detect a accidentally added Azure System Access Token, due to its detected base64 threshold being slightly below
the detection threshold. this change lets you add a section

experimental:
  base64EntropyThreshold: 4.0

to your .talismanrc yaml file that allows you to overwrite this threshold individually. In our case lowering the threshold from 4.5 to 4.0 enabled us to detect
the secret as such and did the job.

Copy link
Collaborator

@svishwanath-tw svishwanath-tw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was involved in debugging why a particular token value wasn't flagged by talisman.
I think this change will help to debug other such occurrences.

@svishwanath-tw
Copy link
Collaborator

@michaellihs :

Merging is blocked
The base branch requires all commits to be signed. Learn more about signing commits.

…threshold

we discovered, that Talisman did not detect a accidentally added Azure System Access Token, due to its detected base64 threshold being slightly below
the detection threshold. this change lets you add a section

```yaml
experimental:
  base64EntropyThreshold: 4.0
```

to your `.talismanrc` yaml file that allows you to overwrite this threshold individually. In our case lowering the threshold from 4.5 to 4.0 enabled us to detect
the secret as such and did the job.

Signed-off-by: Michael Lihs <michael.lihs@thoughtworks.com>
@michaellihs
Copy link
Contributor Author

Re-created the fork due to issues with signing. So I will close the PR and create a new one. Sorry for the trouble...

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.

None yet

2 participants