"urlopen" not thread safe

[John Nagle]

I was looking at the code for "urllib", and there's
some undocumented "FTP cacheing" code in there that's not thread safe.
The documentation for "urllib"

Is there any good reason to keep that code in "urllib"?
Unless something critical uses that undocumented feature,
that code probably ought to come out. Comments?

John Nagle

 

[Gabriel Genellina]

I was looking at the code for "urllib", and there's
some undocumented "FTP cacheing" code in there that's not thread safe.
The documentation for "urllib"

Is there any good reason to keep that code in "urllib"?
Unless something critical uses that undocumented feature,
that code probably ought to come out. Comments?

Maybe the default should be None, like tempcache; anyone interested could
assign to the ftpcache instance attribute.