-
Notifications
You must be signed in to change notification settings - Fork 180
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
Harpoon V2 #190
Comments
totally unsollicited feedback here, but here we go. ;) not a fan of click. tries to do too many things and i quickly hit limitations on others. it's quite a big pill to swallow, i prefer plain argparse nowadays. one thing that's blocking me in adopting harpoon wholesale (including writing plugins, crowdsec would be nice, for example) is the lack of a batch mode... right now i found that i can do
I would like to feed a plain text file listing IPs (maybe sorted, but not deduplicated aka not uniq(1)) and have harpoon just "do the right thing"... Right now I use this as a very crude approximation of part of the first one:
all the rest is manual lookups, one IP at a time. so batch lookups would be very important for me... |
I totally welcome feedback on Harpoon :) Indeed, I tried to move to click but creating commands based on plugins seemed way more complex than argparse so argparse will stay. The intel plugin cycles through all plugin that have intel enabled in the configuration file (doc is also needed for v2 :D). I am not sure exactly how to implement what you say. Would you like one command for all these questions? It is possible to just implement intel by reading in a file with output in JSON, would that help? Or having different commands for each one? Also if you don't know it yet, Greynoise looks like a great addition to your toolkit |
On 2023-11-09 15:30:13, Tek wrote:
I totally welcome feedback on Harpoon :)
Indeed, I tried to move to click but creating commands based on plugins seemed way more complex than argparse so argparse will stay.
The intel plugin cycles through all plugin that have intel enabled in the configuration file (doc is also needed for v2 :D).
I am not sure exactly how to implement what you say. Would you like one command for all these questions? It is possible to just implement intel by reading in a file with output in JSON, would that help? Or having different commands for each one?
Also if you don't know it yet, [Greynoise](https://www.greynoise.io/) looks like a great addition to your toolkit
Thanks for the feedback!
I think one command that takes a list of IPs would be nice. Something
simple: just one IP per line is fine, even better if it counts the
number of duplicates so you get a hit rate profile.
Here the "intel" command doesn't fire up virustotal, even though it has
intel: true in the config, not sure what's up with that...
|
Need a major rewrite to:
It needs some thinking but this first version is very messy
The text was updated successfully, but these errors were encountered: