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

http-default-accounts: add iDRAC9 #2096

Closed
wants to merge 2 commits into from
Closed

http-default-accounts: add iDRAC9 #2096

wants to merge 2 commits into from

Conversation

@cnotin
Copy link

@cnotin cnotin commented Aug 5, 2020

Works fine :)

PORT    STATE SERVICE REASON
443/tcp open  https   syn-ack ttl 128
| http-default-accounts: 
|   [Dell iDRAC9] at /
|_    root:calvin

Similar to #2095

@cnotin
Copy link
Author

@cnotin cnotin commented Aug 5, 2020

Ping @nnposter as you know this very well :)
It might interest you for your fork too!

@nnposter nnposter self-assigned this Aug 17, 2020
Copy link

@nnposter nnposter left a comment

This is a nice fingerprint contribution. Please see my inline comments for minor issues.

nselib/data/http-default-accounts-fingerprints.lua Outdated Show resolved Hide resolved
nselib/data/http-default-accounts-fingerprints.lua Outdated Show resolved Hide resolved
nselib/data/http-default-accounts-fingerprints.lua Outdated Show resolved Hide resolved
nselib/data/http-default-accounts-fingerprints.lua Outdated Show resolved Hide resolved
nselib/data/http-default-accounts-fingerprints.lua Outdated Show resolved Hide resolved
@cnotin cnotin force-pushed the cnotin:patch-9 branch from f2a3bb3 to cd91f45 Aug 18, 2020
@cnotin
Copy link
Author

@cnotin cnotin commented Aug 18, 2020

Thanks for your detailed feedback @nnposter!
Two comments remain open for now
I'll test your version too ;)

@cnotin cnotin force-pushed the cnotin:patch-9 branch from cd91f45 to bd302b5 Aug 18, 2020
cnotin added 2 commits Aug 5, 2020
@cnotin
Copy link
Author

@cnotin cnotin commented Aug 19, 2020

What do you think of this? Hope the parenthesis indentation is fine!

@nnposter
Copy link

@nnposter nnposter commented Aug 20, 2020

The fingerprint has been committed as r37980. Thank you for contributing to nmap!

@nmap-bot nmap-bot closed this in ae84080 Aug 20, 2020
@cnotin cnotin deleted the cnotin:patch-9 branch Aug 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.