[Twisted-Python] Twisted Names strings

exarkun at twistedmatrix.com exarkun at twistedmatrix.com
Tue Jan 22 12:36:49 MST 2013


On 03:37 pm, angelo.dellaera at gmail.com wrote:
>
>Hi,
>I didn't have the possibility to test the fix during the last days 
>(hopefully will do
>in the next hours) but taking a look at the fix proposed by ralphm
>
>http://twistedmatrix.com/trac/changeset/36867
>
>it seems like the patch is not complete because a similar check (and 
>potential
>conversion) should be inserted in the twisted.names.dns.Name __init__ 
>method
>too. Am I wrong?

Hi Angelo,

Thanks for taking a look at the changeset, at least!  I think that you 
are correct.  I would even go so far as to say that the change should be 
reverted, as it does not actually fix the *regression*, it works around 
it in a higher-level API.  As you say, this results in an incomplete fix 
for the issue since many applications may (and likely are) using `Name` 
directly rather than via SRVConnector.

I hope Ralph or Tom will chime in with their thoughts on this.

Jean-Paul




More information about the Twisted-Python mailing list