bpo-27682: Handle client connection terminations in wsgiref (GH-9713)

bpo-27682: Handle client connection terminations in wsgiref (GH-9713) · python/cpython@3d37ea2

3 files changed

lines changed

Original file line number	Diff line number	Diff line change
`@@ -788,6 +788,24 @@ def flush(self):`
`788`	`788`	`b"Hello, world!",`
`789`	`789`	`written)`
`790`	`790`
	`791`	`+def testClientConnectionTerminations(self):`
	`792`	`+environ = {"SERVER_PROTOCOL": "HTTP/1.0"}`
	`793`	`+for exception in (`
	`794`	`+ConnectionAbortedError,`
	`795`	`+BrokenPipeError,`
	`796`	`+ConnectionResetError,`
	`797`	`+ ):`
	`798`	`+with self.subTest(exception=exception):`
	`799`	`+class AbortingWriter:`
	`800`	`+def write(self, b):`
	`801`	`+raise exception`
	`802`	`+`
	`803`	`+stderr = StringIO()`
	`804`	`+h = SimpleHandler(BytesIO(), AbortingWriter(), stderr, environ)`
	`805`	`+h.run(hello_app)`
	`806`	`+`
	`807`	`+self.assertFalse(stderr.getvalue())`
	`808`	`+`
`791`	`809`
`792`	`810`	`if __name__ == "__main__":`
`793`	`811`	`unittest.main()`

Original file line number	Diff line number	Diff line change
`@@ -136,6 +136,10 @@ def run(self, application):`
`136`	`136`	`self.setup_environ()`
`137`	`137`	`self.result = application(self.environ, self.start_response)`
`138`	`138`	`self.finish_response()`
	`139`	`+except (ConnectionAbortedError, BrokenPipeError, ConnectionResetError):`
	`140`	`+# We expect the client to close the connection abruptly from time`
	`141`	`+# to time.`
	`142`	`+return`
`139`	`143`	`except:`
`140`	`144`	`try:`
`141`	`145`	`self.handle_error()`

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	+:class:`wsgiref.handlers.BaseHandler` now handles abrupt client connection
	`2`	`+terminations gracefully. Patch by Petter Strandmark.`