homepage

The ssl module sets a "debug" flag on the OpenSSL SSL struct at the end of PySSL_SSLdo_handshake():
 self->ssl->debug = 1;
The OpenSSL header files have this to say about this flag:
 /* set this flag to 1 and a sleep(1) is put into all SSL_read()
 * and SSL_write() calls, good for nbio debuging :-) */
 int debug; 
Some uses of it in the OpenSSL source are actually guarded by a macro (#ifdef PKT_DEBUG), but some are not. Bill, do you remember why you put that call? I would find it strongly preferrable to remove that assignment (doing so doesn't seem to break anything in the test suite).

Actually, looking at the history of the SVN repo, this dates back to 1999 when the first SSL support was incorporated by Guido based on third-party patches, and no trace of a review discussion can be found. The only thing we can do is assume it's useless (and perhaps detrimental to performance) and remove it now.
Benjamin, do you think it's safe for 2.7 or should wait for 2.7.1?

Let it die.

Thanks! Committed in r82210 (trunk), r82211 (py3k), r82212 (2.6), r82213 (3.1).