Opened 11 years ago

Last modified 11 years ago

#586 defect closed fixed (fixed)

UDP write() accepts hostnames rather than IPs

Reported by: itamarst Owned by:
Priority: highest Milestone:
Component: Keywords:
Cc: radix, exarkun, itamarst, anthony Branch:
Author: Launchpad Bug:

Description


Change History (5)

comment:1 Changed 11 years ago by itamarst

Or, to put it another way if we do want to allow it, doesn't use pluggable
resolver IIRC. But that would be bad, we kinda assume packets get sent
immediately (we have exceptions for message length, and this helps keep them in
order and so on) and using resolver would add lots of overhead.

At the very least we need to switch to using twisted resolver and document that
sending unresolved names is expensiive. Alternatively, add depreaction warning
for non IP addresses.

comment:2 Changed 11 years ago by exarkun

Resolving hostnames is easy.  I think we should deprecate non-IP addresses and
raise a ValueError in some future version.

comment:3 Changed 11 years ago by itamarst

and before 1.3.

comment:4 Changed 11 years ago by itamarst

write() no longer accepts hostnames as of 1.3.

comment:5 Changed 4 years ago by <automation>

  • Owner itamarst deleted
Note: See TracTickets for help on using tickets.