Author: markt Date: Fri Oct 14 10:49:19 2016 New Revision: 1764867 URL: http://svn.apache.org/viewvc?rev=1764867&view=rev Log: Align with HTTP/1.1. An unhandled exception should trigger a 500 response
Modified: tomcat/trunk/java/org/apache/coyote/http2/StreamProcessor.java tomcat/trunk/webapps/docs/changelog.xml Modified: tomcat/trunk/java/org/apache/coyote/http2/StreamProcessor.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http2/StreamProcessor.java?rev=1764867&r1=1764866&r2=1764867&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/coyote/http2/StreamProcessor.java (original) +++ tomcat/trunk/java/org/apache/coyote/http2/StreamProcessor.java Fri Oct 14 10:49:19 2016 @@ -221,6 +221,7 @@ class StreamProcessor extends AbstractPr if (log.isDebugEnabled()) { log.debug(sm.getString("streamProcessor.service.error"), e); } + response.setStatus(500); setErrorState(ErrorState.CLOSE_NOW, e); } Modified: tomcat/trunk/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1764867&r1=1764866&r2=1764867&view=diff ============================================================================== --- tomcat/trunk/webapps/docs/changelog.xml (original) +++ tomcat/trunk/webapps/docs/changelog.xml Fri Oct 14 10:49:19 2016 @@ -94,6 +94,11 @@ merge these into a single Cookie header before processing continues. (markt) </fix> + <fix> + Align the HTTP/2 implementation with the HTTP/1.1 implementation and + return a 500 response when an unhandled exception occurs during request + processing. (markt) + </fix> </changelog> </subsection> <subsection name="Jasper"> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org