Code review comment for lp://staging/~diogobaeder/txstatsd/fix-host-lookup

Revision history for this message
Sidnei da Silva (sidnei) wrote :

I'm fairly confident that calling transport.connect() does not do what you want. That sets the transport to 'connected udp' which is not what we want. See: http://twistedmatrix.com/documents/11.0.0/core/howto/udp.html#auto2

Instead, you should pass host, port to the transport gateway and use them in transport.write(data, (host, port)).

« Back to merge proposal