-
Notifications
You must be signed in to change notification settings - Fork 561
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
Presets #1058
Presets #1058
Conversation
Fix rare attributeerror in command helper
…n-debugging Better debugging during scan cancellation
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.
I have reviewed this branch and it looks good.
During this review, I launched BBOT and tested every code snippet found in the documentation and checked for errors. I reviewed the Preset TODO list to ensure that every change was indeed found in the code and completed. I reviewed every user document (README, etc), read every line, and followed every link in the documentation to ensure their functionality.
During the pytests review, I ran all the tests with debug logs viewable and watched for any false-positives. I paid close attention to the tests which had the most changes.
During the code review, I paid extra close attention to presets.py, scanner.py, environ.py, files.py, depsinstaller.py. I asked questions directly to the author on code snippets that I didn't understand and asked for feedback when I didn't see errors that I expected to see.
I have been using the presets branch while it was being developed. Thus, during this review, I focused on finding bugs that I had seen before. Every bug that I had encountered in the past is no where to be seen now.
I am submitting my approval of this branch and recommend it to be merged.
Implement Radix Tree
Faster Regexes
Shared Ansible Dependencies
Intercept Modules
DNS Engine - Offload DNS to Dedicated Process
This PR contains the new Presets feature, as mentioned here.
TODO:
Preset
classScanner
Scanner
(soScanner
needsPreset
, but not vice versa)scan
to emit events (which must be associated with the main scan).cli.py
DEBUG
, etc.)#{custom_variables}
PRESET_PATH
)Preset.bake()
Preset.modules
for cleaner+reversible serializationsecrets.yml
bbot/test/test_step_1/test_manager_deduplication.py:95
-p
/-c
(files)-p
/-c
(dot syntax)stdout
output modulesecrets.yml