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

Consistently refer to black hole as two words #18974

Merged
merged 2 commits into from
Mar 20, 2024

Conversation

zeroSteiner
Copy link
Contributor

Fix a typo in the help menu of the dns command.

@adfoster-r7
Copy link
Contributor

Just before we lock things in; What was the rationale behind choosing two words for black-hole versus one word blackhole - similar to what IANA uses:

https://www.iana.org/help/dns-blackhole

As well as Wikipedia:

This article is about blackhole DNS servers. For other uses, see black hole (disambiguation).
https://en.wikipedia.org/wiki/Blackhole_server

Is there a semantic difference for Metasploit's implementation versus that definition? 👀

@zeroSteiner
Copy link
Contributor Author

The rationale was that the actual phenomenon is spelled with two words. Also from Wikipedia: https://en.wikipedia.org/wiki/Black_hole

Do you want it changed back to one word?

@adfoster-r7
Copy link
Contributor

Working for me, thanks!

@adfoster-r7 adfoster-r7 merged commit 0399768 into rapid7:master Mar 20, 2024
34 checks passed
@adfoster-r7
Copy link
Contributor

adfoster-r7 commented Mar 20, 2024

Release Notes

Fixes a typo in the help menu of the dns command.

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

Successfully merging this pull request may close these issues.

3 participants