-
Notifications
You must be signed in to change notification settings - Fork 10
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
Automate hash scans #54
Conversation
This part was commented out durring development testing. Accidentally left that way at commit.
For the same reasons as discussed in #53, this PR will not require a version change to this repo, so I deleted the "Pre-merge" and "Post-merge" checklist sections in the PR description. |
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.
Thanks for building out this script- it will be an excellent improvement from our current manual process.
Please take a look at my first round of suggestions.
@bra1ncramp Just pinging on this PR so that it doesn't get forgotten. It's good stuff- we should try to get it across the finish line soon. |
Co-authored-by: Shane Frasier <[email protected]>
Co-authored-by: Shane Frasier <[email protected]>
Co-authored-by: Shane Frasier <[email protected]>
Co-authored-by: Shane Frasier <[email protected]>
Co-authored-by: Shane Frasier <[email protected]>
Co-authored-by: Shane Frasier <[email protected]>
Co-authored-by: dav3r <[email protected]>
Co-authored-by: dav3r <[email protected]>
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.
Here is a suggestion.
Co-authored-by: dav3r <[email protected]>
This may go away - but committing the suggestion for now until we improve the language overall. Co-authored-by: dav3r <[email protected]>
Simply running `hostname` should work fine with `AWS-StartNonInteractiveCommand` Co-authored-by: dav3r <[email protected]>
Co-authored-by: dav3r <[email protected]>
Co-authored-by: dav3r <[email protected]>
Instead of placing AWS environmental variables in the script, declare them on the command line.
@bra1ncramp - Are you running |
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.
Here are a couple more small thangs to clean up.
…ironmental" Co-authored-by: dav3r <[email protected]>
Co-authored-by: dav3r <[email protected]>
We've removed the pkill command, so do not need this anymore
@bra1ncramp - Are you running |
I was running it wrong. I should have run |
You should run |
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.
Looks solid - strong work! 💪 💼
🗣 Description
Create extra file to automate the IOC Hash scans.
ioc-scanner/extras
netcat
is installed in the instance and startsnetcat
listening on port6666
. It uploads the latest copy ofioc_scanner.py
with the latest hashes to the instance. Then it executesioc_scanner.py
and directs the output to a local log file.💭 Motivation and context
The current process requires manually uploading the
ioc_scanner.py
file to each instance one at a time. This is a laborious method and not scaleable.🧪 Testing
This was tested using the default test blobs in
ioc_scanner.py
.✅ Pre-approval checklist