Skip to content
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 utils for a range of ip addresses #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

joranbeasley
Copy link

IP ranges support (from todos and roadmaps ...)

no non-builtin imports required, and includes tests

 IP ranges support (from todos and roadmaps ...)

no non-builtin imports required, and includes tests
split a large block of ip address into num_blocks into smaller blocks
@evyatarmeged
Copy link
Owner

Hey @joranbeasley, looks good.
I'm not yet sure how this should be integrated in the main run. That is the reason I've avoided adding this as of yet.
If you have any ideas how to actually combine this with the main run I would love to hear (and see) them.
Rembmer - all tasks are on an event loop while url/subdomain bruteforcing alsorun a thread pool and nmap runs on its own thread in the background.

Looking forward to your response. Nonetheless, this is much appreciated.

@joranbeasley
Copy link
Author

I don't offhand I started to look into that ... I use these methods in my own little ip scanner thing (thats just implemented using python sockets and paramiko) and saw you had this on your futures list, and i am pretty happy with this implementation in my own project, and figured maybe you could use the utility methods) ... although maybe you just want to use nmaps target ranges argument (which looks different than 2 quad dotted ip addresses) (I committed the utils before I went and looked into how you might use them) ... but i can look into it more :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants