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

Add ntt.net record to example/jwhois.conf. Issue #3. #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add ntt.net record to example/jwhois.conf. Issue #3. #11

wants to merge 1 commit into from

Conversation

mykola-kyryk
Copy link
Contributor

No description provided.

@@ -56,6 +56,11 @@ whois-servers {
"^!?NET\\(BLK\\)?-VRIO\\(-[A-Z0-9]+\\)+$" = "rwhois.verio.net";

#
# Catch ntt.net netblock records
#
"^!?NET\\(BLK\\)?-VRIO\\(-[A-Z0-9]+\\)+$" = "rwhois.gin.ntt.net";
Copy link
Owner

Choose a reason for hiding this comment

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

This seems as if it would match the same as the previous verio regexp. Are there any examples of an ntt netblock record? The regexp needs to be updated accordingly.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well, it is really hard to read diff in the issue #3. Can you attach to an
issue as a file or email it to me?
On Jan 20, 2015 10:31 PM, "Jonas Öberg" [email protected] wrote:

In example/jwhois.conf
#11 (diff):

@@ -56,6 +56,11 @@ whois-servers {
"^!?NET(BLK)?-VRIO(-[A-Z0-9]+)+$" = "rwhois.verio.net";

#
  • Catch ntt.net netblock records

  • "^!?NET(BLK)?-VRIO(-[A-Z0-9]+)+$" = "rwhois.gin.ntt.net";

This seems as if it would match the same as the previous verio regexp. Are
there any examples of an ntt netblock record? The regexp needs to be
updated accordingly.


Reply to this email directly or view it on GitHub
https://github.com/jonasob/jwhois/pull/11/files#r23253032.

robert-scheck added a commit to robert-scheck/jwhois that referenced this pull request Mar 1, 2015
@robert-scheck
Copy link
Contributor

After spending quite a nice amount of time with this issue, I figured out this:

$ host rwhois.verio.net
Host rwhois.verio.net not found: 3(NXDOMAIN)
$ 

And Wikipedia says "In early 2000 Verio was sold to NTT" at https://en.wikipedia.org/wiki/Verio thus the WHOIS server needs to be changed from rwhois.verio.net to rwhois.gin.ntt.net simply. The rwhois.gin.ntt.net returns the right results for Verio objects. Patch suggested as part of pull request #17.

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

Successfully merging this pull request may close these issues.

3 participants