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

NAPTR queries incorrectly timed out #74

Open
CodeMonkeyKevin opened this issue Jul 11, 2014 · 5 comments
Open

NAPTR queries incorrectly timed out #74

CodeMonkeyKevin opened this issue Jul 11, 2014 · 5 comments

Comments

@CodeMonkeyKevin
Copy link

When doing NAPTR query. All queries get timed out but tcpdump shows the server responded correctly and successfully to the query.

TCPDUMP, node-dns reported timeout but server sent back successful response before timeout window:
08:04:19.612311 IP 192.168.1.120.53110 > 192.168.1.90.53: 2407+ NAPTR? 3.3.2.2.1.1.5.5.5.5.1.e164enum.net. (52)
08:04:19.670588 IP 192.168.1.90.53 > 192.168.1.120.53110: 2407*- 1/1/0 NAPTR (200)

Doing the same query with dig shows same response and packet but without timeout.

@taoeffect
Copy link
Collaborator

NAPTR is broken in the current library. I'll merge the okTurtles fork in soon (today am traveling all day). You can use it in the meantime, it should work there.

https://github.com/okTurtles/node-dns

@CodeMonkeyKevin
Copy link
Author

Any chance on merging this in?

@taoeffect
Copy link
Collaborator

@CodeMonkeyKevin Yes, sorry, I was at a conference and now am back. My plan is to merge that in soon, maybe today or if not then this week. Can you confirm that the issue is fixed in that fork?

@taoeffect
Copy link
Collaborator

@CodeMonkeyKevin Sorry about not getting to merge the PR yet. I haven't forgotten and it's high up on my priority list, it's just competing with other slightly more high priority items. Were you able to check if the fork fixes your issue?

@CodeMonkeyKevin
Copy link
Author

Sorry for the delay. It did not fix the issue, NAPTR queries still have the timeout issue.

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

No branches or pull requests

2 participants