-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Adding open
option to nmap
#6200
Adding open
option to nmap
#6200
Conversation
Sorry, I built this off of a current PR: #6165 So this is dependent on that. Let me know if I should pull out the |
Docs Build 📝Thank you for contribution!✨ This PR has been merged and your docs changes will be incorporated when they are next published. |
1cd4088
to
bf8f295
Compare
@barloff-st was it intentional that you closed this one (by pushing an 'empty' branch)? |
@felixfontein No that was not my intention. I synced my |
This comment was marked as outdated.
This comment was marked as outdated.
f85c4c0
to
9f5fbe7
Compare
This comment was marked as outdated.
This comment was marked as outdated.
9f5fbe7
to
56f1f14
Compare
@felixfontein issues with rebase have been resolved. Should be good now. |
Co-authored-by: Felix Fontein <felix@fontein.de>
Co-authored-by: Felix Fontein <felix@fontein.de>
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'll merge the end of this weekend if nobody objects.
Backport to stable-6: 💚 backport PR created✅ Backport PR branch: Backported as #6246 🤖 @patchback |
@barloff-st thanks for your contribution! |
* Adding nmap `open` option * Removing accidental copy/paste issue * Update changelogs/fragments/6200-adding-open-option-to-nmap.yml Co-authored-by: Felix Fontein <felix@fontein.de> * Update plugins/inventory/nmap.py Co-authored-by: Felix Fontein <felix@fontein.de> --------- Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit 59e5807)
…6246) Adding `open` option to nmap (#6200) * Adding nmap `open` option * Removing accidental copy/paste issue * Update changelogs/fragments/6200-adding-open-option-to-nmap.yml Co-authored-by: Felix Fontein <felix@fontein.de> * Update plugins/inventory/nmap.py Co-authored-by: Felix Fontein <felix@fontein.de> --------- Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit 59e5807) Co-authored-by: barloff-st <42866449+barloff-st@users.noreply.github.com>
SUMMARY
This will add in the
--open
option for thenmap
inventory plugin. Currently the port scan (-sP
) just scans ports and returns results from all ports, regardless of if they are closed. Theopen
option will all for only open ports to be returned. This will also clarify this by removing the "open ports" mention in the currentports
argument.ISSUE TYPE
COMPONENT NAME
community.general.nmap
--open
optionADDITIONAL INFORMATION