skoranga / node-dns-sync

Sync version of DNS lookup. Useful for server startup activities.
MIT License
7 stars 10 forks source link

Allow resolving things other than A records #2

Closed richthegeek closed 9 years ago

richthegeek commented 9 years ago

This allows resolution of TXT records and other non-A/AAAA records.

It also uses JSON.stringify/parse to pass data back from the subprocess.

Finally it uses some automated guessing on the argv values, that was causing an issue with testing

skoranga commented 9 years ago

Hi @richthegeek, Doesn't look like this PR is backward compatible. Let me think some more on this and add backward compatibility.

richthegeek commented 9 years ago

Made some changes to make it backwards compatible.

Default behavior when no "type" is passed is to perform a dns.lookup, and I added a "lookup" method to the module which does this also.

skoranga commented 9 years ago

thanks @richthegeek. Will be publishing shortly

skoranga commented 9 years ago

published dns-sync@0.1.3