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

docs for aux scanner finger #8298

Merged
merged 2 commits into from
Apr 28, 2017
Merged

docs for aux scanner finger #8298

merged 2 commits into from
Apr 28, 2017

Conversation

h00die
Copy link
Contributor

@h00die h00die commented Apr 26, 2017

Part of #8296

This adds module docs for the finger_users scanner.

Verification

List the steps needed to make sure this thing works

  • Start msfconsole
  • Do: use auxiliary/scanner/finger/finger_users
  • Do: info -d
  • Verify spelling and grammar

@wvu
Copy link
Contributor

wvu commented Apr 26, 2017

@h00die h00die added the blocked Blocked by one or more additional tasks label Apr 26, 2017
@h00die
Copy link
Contributor Author

h00die commented Apr 26, 2017

delaying so i can add the nmap script equivalent.

@h00die h00die removed the blocked Blocked by one or more additional tasks label Apr 26, 2017
@dmohanty-r7 dmohanty-r7 self-assigned this Apr 28, 2017
@dmohanty-r7 dmohanty-r7 merged commit 1843f03 into rapid7:master Apr 28, 2017
dmohanty-r7 added a commit that referenced this pull request Apr 28, 2017
Utilizing the [finger](https://nmap.org/nsedoc/scripts/finger.html) script

```
# nmap -p 79 -sC 127.0.0.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should provide a more specific example. It looks like this is the default example in the NSE docs, since @usage isn't set in the script.

This command runs the default scripts in Nmap, not the finger script directly. To do that, you want to use --script finger or even --script +finger if fingerd breaks the portrule.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

100% agree. Good check on me getting lazy, I'll fix it up. Thanks for keeping me honest :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nah, it's not laziness. I'm being nitpicky, since NSE docs have that fallback. :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@busterb busterb added the rn-no-release-notes no release notes label May 8, 2017
@h00die h00die deleted the finger branch May 11, 2017 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants