-
Notifications
You must be signed in to change notification settings - Fork 72
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 --entropy-sensitivity option for controlling entropy checks #272
Merged
Commits on Nov 12, 2021
-
Fixes #265 Provide more comprehensible alternative for tuning entropy checking. This is applied consistently across all target character sets, and stated in a way that is slightly easier to understand ("higher means more likely to flag a given string"). The older `--b64-entropy-score` and `--hex-entropy-score` options are marked as deprecated but retained for backwards compatibility (and they override `--sensitivity` if used together with it).
Configuration menu - View commit details
-
Copy full SHA for 700af2d - Browse repository at this point
Copy the full SHA 700af2dView commit details -
Use `--entropy-sensitivity` instead of `--sensitivity`
Configuration menu - View commit details
-
Copy full SHA for 7f2480f - Browse repository at this point
Copy the full SHA 7f2480fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 23c0552 - Browse repository at this point
Copy the full SHA 23c0552View commit details -
Configuration menu - View commit details
-
Copy full SHA for 1a5daa4 - Browse repository at this point
Copy the full SHA 1a5daa4View commit details -
Configuration menu - View commit details
-
Copy full SHA for 799af80 - Browse repository at this point
Copy the full SHA 799af80View commit details -
Configuration menu - View commit details
-
Copy full SHA for b19a1b4 - Browse repository at this point
Copy the full SHA b19a1b4View commit details -
Configuration menu - View commit details
-
Copy full SHA for 54f5c9c - Browse repository at this point
Copy the full SHA 54f5c9cView commit details -
Expose magic numbers as properties
...and use them instead of private members
Configuration menu - View commit details
-
Copy full SHA for 7cad444 - Browse repository at this point
Copy the full SHA 7cad444View commit details -
Configuration menu - View commit details
-
Copy full SHA for a1006bc - Browse repository at this point
Copy the full SHA a1006bcView commit details -
Remove entropy scoring members
Do everything from scratch instead of storing explicitly. This is a PITA because you can't combine `@property` and `@lru_cache()` and skipping the caching would be a killer.
Configuration menu - View commit details
-
Copy full SHA for 017f43b - Browse repository at this point
Copy the full SHA 017f43bView commit details -
* Consolidate common code for entropy limit back into a single method, and rework properties related to it so they are cleaner. * Invert sensitivity scale; adjust math and doc to match. It's still weird but aligns more closely to the underlying entropy metric.
Configuration menu - View commit details
-
Copy full SHA for b8f3a5a - Browse repository at this point
Copy the full SHA b8f3a5aView commit details
Commits on Nov 15, 2021
-
Configuration menu - View commit details
-
Copy full SHA for b8264ed - Browse repository at this point
Copy the full SHA b8264edView commit details
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.