On Fri, 22 Nov 2002 22:55:04 -0500 Jp Calderone wrote:

Diff for twisted/internet/base.py's resolve() function to make it perform lookups asynchronously using the new stuff in twisted/names/client.py.

Mmm. I don't like adding a dependency on an external package in twisted.internet. And this doesn't let people replace the name resolver with their own that does e.g. caching. And what if the reactor doesn't support UDP? I suggest instead a reactor.installResolver(r) method, where the default one stays as is using socket module (with the DNS type fix going in though). -- Itamar Shtull-Trauring http://itamarst.org/ Available for Python, Twisted, Zope and Java consulting ***> http://VoteNoWar.org -- vote/donate/volunteer <***

Hi, Jp Calderone:

I removed the type argument, since it complicates implementation for other reactors, isn't really useful in 99% of cases, and never worked when set to anything other than A (1) anyway. Maybe it should be deprecated first for a while, but I doubt anyone's using it anyway so it may not matter.

What -- you never look up records other than A? I've got a bunch of useful candidates here -- PTR, NS, SOA, CNAME, MX ... and, guess what, I'm going to need all of them (except MX ;-) early next year. -- Matthias Urlichs | noris network AG | http://smurf.noris.de/

On Fri, 22 Nov 2002, Itamar Shtull-Trauring wrote:

I suggest instead a reactor.installResolver(r) method, where the default one stays as is using socket module (with the DNS type fix going in though).

+1

On Sat, Nov 23, 2002 at 01:06:43PM -0000, Moshe Zadka wrote:

On Sat, 23 Nov 2002, Christopher Armstrong wrote:

...

Why wouldn't we use the twisted.names resolver by default, if available?

/etc/hosts, for one.

And if it supported /etc/hosts? :) -- Christopher Armstrong << radix@twistedmatrix.com >> http://twistedmatrix.com/users/radix.twistd/

On Sat, Nov 23, 2002 at 10:17:05AM +0100, Matthias Urlichs wrote:

Hi,

Hello :)

Jp Calderone:

...

I removed the type argument, since it complicates implementation for other reactors, isn't really useful in 99% of cases, and never worked when set to anything other than A (1) anyway. Maybe it should be deprecated first for a while, but I doubt anyone's using it anyway so it may not matter.

What -- you never look up records other than A?

I've got a bunch of useful candidates here -- PTR, NS, SOA, CNAME, MX ... and, guess what, I'm going to need all of them (except MX ;-) early next year.

All record types are supported ;) The question is just whether you should go through the reactor interface to make queries about them, or if you should use the "real" name lookup API instead. Essentially.. from twisted.internet import reactor from twisted.protocols import dns reactor.resolve('somehost.com', dns.MX).addCallback(whatever) vs from twisted.names import client client.lookupMailExchange('somehost.com').addCallback(whatever) I can't envision any circumstances where the core would want to do lookups other than of A records, so supporting them /without/ adding a dependancy of twisted.internet on twisted.names (something there are good reasons to want to avoid) seems like an unecessary burden on anyone else who wants to implement the reactor API. Jp -- 5:00pm up 22 days, 3:54, 3 users, load average: 0.00, 0.00, 0.00

On Sat, Nov 23, 2002 at 12:31:21PM -0600, Glyph Lefkowitz wrote:

...

And if it supported /etc/hosts? :)

/etc/hosts. Windows registry keys pointing at DNS servers. ZoneAlarm. Potentially there are wacky plug-ins for the system host resolution technique which use LDAP rather than DNS. (I don't know if any exist, but there's nothing to say they couldn't ^_^). Effectively replacing the system doman name lookup is tricky, especially on win32.

Note on mordern unixes: libnss (name service switch) allows to admin to use _whatever he wants_ to resolve host names. -- :(){ :|:&};:

Tommi Virtanen wrote:

On Sat, Nov 23, 2002 at 12:31:21PM -0600, Glyph Lefkowitz wrote:

...

...

And if it supported /etc/hosts? :)

/etc/hosts. Windows registry keys pointing at DNS servers. ZoneAlarm. Potentially there are wacky plug-ins for the system host resolution technique which use LDAP rather than DNS. (I don't know if any exist, but there's nothing to say they couldn't ^_^). Effectively replacing the system doman name lookup is tricky, especially on win32.

Note on mordern unixes: libnss (name service switch) allows to admin to use _whatever he wants_ to resolve host names.

And I've already pointed this out to exarkun since he's doing the work, but on Linux, glibc 2.2.4 and later expose an asynchronous name lookup interface already. Docs on it can be found here: http://people.redhat.com/drepper/asynchnl.pdf Windows has an asynch interface as well: http://msdn.microsoft.com/library/en-us/winsock/winsock/wsaasyncgethostbynam... - Bruce