Added ability to limit number of hosts saved #37
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# Conflicts: # autosploit.py
|
Wasn't paying attention and closed the PR. Lol, anyway, there seems to be a conflict, until that is resolved i can't merge this. |
|
Fixed conflicts |
|
still some conflicts in the trunk, fix them please |
|
Resolved new conflicts |
|
Looks good to me, @NullArray?
Second thought we can do it when we merge |
Ekultek
approved these changes
Feb 20, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Once option 2 is selected the user is prompted whether they want the total number of hosts limited.
The chosen limit is passed to the target function, if the user decided not to have a limit nothing will be passed
If a limit is inputted then that many results are saved to the file.