-
Notifications
You must be signed in to change notification settings - Fork 466
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
Add filtering of english words from entropy (and keyword) plugins #241
Merged
Merged
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
d93e9bd
:mortar_board: Standardize `do not` -> `don't`
KevinHock 5b66a26
:snake: Refactor awkward [0] to a short-circuit
KevinHock bb543c5
:bug: We were doing e.g. '.svg' == 'svg'
KevinHock f10cce8
:snake: Refactor `for...if...return` with `any`
KevinHock bc4f922
:snake: Remove is_false_positive from RegexBasedDetector
KevinHock 0b2c0e1
:mortar_board: Standardize comments
KevinHock f8cb31f
:tada: Add --word-list option
KevinHock 7bdf06f
:100: Coverage for jwt.py
KevinHock e4494ea
:snake: Separate not :100: covered files
KevinHock d3c9583
:snake: Make automaton case-insensitive
KevinHock 9e3669d
:snake: Slightly better condition for is_verified = True
KevinHock 7ce4b85
:telescope: Don't put words less than 4 chars in automaton
KevinHock de7fbd2
:100: Coverage for high_entropy_strings.py
KevinHock 139c64a
:snake: Refactor `audit --display-results` code
KevinHock 6c6e028
:mortar_board: Use plural variable name for list
KevinHock 977c4fb
:tada: Add verification for Mailchimp API keys
KevinHock 9cabbe0
:tada: Add verification for Stripe secret API keys
KevinHock 20d1921
:100: Coverage for baseline.py initialize.py usage.py
KevinHock 433b75e
:snake: Add stats to `audit --display-results` code
KevinHock a096600
:performing_arts: Add more to `IGNORED_FILE_EXTENSIONS`
KevinHock ef5d000
:bug: Fix auditing a baseline with MailChimp in it
KevinHock a493356
:bug: Fix TypeError thrown when Yaml was all comments
KevinHock 171eeca
:bug: Fix scanning files that don't exist
KevinHock 69a3aac
:snake: Use hash.update to be more succinct
KevinHock 19d22da
:snake: Inline leading . in constants.py
KevinHock b440623
:snake: Remove useless WordListSupportedDetector class
KevinHock File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
💯 Coverage for high_entropy_strings.py
- 🐍 Refactor hadouken code - 🐍 Remove high_entropy_strings.py from the uncovered files list in tox
- Loading branch information
commit de7fbd265942b1d404a3dc7292db386312a9b63d
There are no files selected for viewing
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,8 +14,8 @@ commands = | |
coverage run -m pytest tests -v | ||
coverage report --show-missing --include=tests/* --fail-under 100 | ||
# This is so that we do not regress unintentionally | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ❤️ |
||
coverage report --show-missing --include=detect_secrets/* --omit=detect_secrets/core/audit.py,detect_secrets/core/baseline.py,detect_secrets/core/secrets_collection.py,detect_secrets/core/usage.py,detect_secrets/main.py,detect_secrets/plugins/common/ini_file_parser.py,detect_secrets/plugins/common/initialize.py,detect_secrets/plugins/high_entropy_strings.py --fail-under 100 | ||
coverage report --show-missing --include=detect_secrets/core/audit.py,detect_secrets/core/baseline.py,detect_secrets/core/secrets_collection.py,detect_secrets/core/usage.py,detect_secrets/main.py,detect_secrets/plugins/common/ini_file_parser.py,detect_secrets/plugins/common/initialize.py,detect_secrets/plugins/high_entropy_strings.py --fail-under 97 | ||
coverage report --show-missing --include=detect_secrets/* --omit=detect_secrets/core/audit.py,detect_secrets/core/baseline.py,detect_secrets/core/secrets_collection.py,detect_secrets/core/usage.py,detect_secrets/main.py,detect_secrets/plugins/common/ini_file_parser.py,detect_secrets/plugins/common/initialize.py --fail-under 100 | ||
coverage report --show-missing --include=detect_secrets/core/audit.py,detect_secrets/core/baseline.py,detect_secrets/core/secrets_collection.py,detect_secrets/core/usage.py,detect_secrets/main.py,detect_secrets/plugins/common/ini_file_parser.py,detect_secrets/plugins/common/initialize.py --fail-under 96 | ||
pre-commit run --all-files --show-diff-on-failure | ||
|
||
[testenv:venv] | ||
|
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice refactor!