[issue1193577] add server.shutdown() method to SocketServer

Jeffrey Yasskin <[EMAIL PROTECTED]> added the comment: So this has a race. See issue 2302 to discuss a fix. -- resolution: -> fixed status: open -> closed superseder: -> Uses of SocketServer.BaseServer.shutdown have a race _ Tracker <[EMAIL PROTECTE

[issue1193577] add server.shutdown() method to SocketServer

Jeffrey Yasskin added the comment: Hearing no objections, I've submitted this as r61289. _ Tracker <[EMAIL PROTECTED]> _ ___ Python-bugs-list

[issue1193577] add server.shutdown() method to SocketServer

Jeffrey Yasskin added the comment: I'll submit the attached patch in a couple days unless I get comments. Added file: http://bugs.python.org/file9586/polling_shutdown.patch _ Tracker <[EMAIL PROTECTED]> __

[issue1193577] add server.shutdown() method to SocketServer

Jeffrey Yasskin added the comment: It seems that .await_request() was only added a month ago to fix issue 742598, so it's no great hardship to refactor it again now. Timeouts never worked for .serve_forever() because the try/except in .handle_request() turned their exception into a plain return,

[issue1193577] add server.shutdown() method to SocketServer

Jeffrey Yasskin added the comment: Also, Pedro's argument against a daemon_threads argument to TCPServer convinces me. I think we can add it in ThreadingMixIn.__init__ once this whole hierarchy is switched over to new-style classes. That's already done in 3.0, but I don't know what it would affec