On 29/04/2019 13:20, Rémy Maucherat wrote: > On Mon, Apr 29, 2019 at 1:53 PM Mark Thomas <ma...@apache.org> wrote: > >> On 29/04/2019 12:30, r...@apache.org wrote: >>> This is an automated email from the ASF dual-hosted git repository. >>> >>> remm pushed a commit to branch master >>> in repository https://gitbox.apache.org/repos/asf/tomcat.git >>> >>> >>> The following commit(s) were added to refs/heads/master by this push: >>> new fa43440 Better error for empty header name >>> fa43440 is described below >>> >>> commit fa43440991fdb5d088a6417425cd18a80ba2cf71 >>> Author: remm <r...@apache.org> >>> AuthorDate: Mon Apr 29 13:30:06 2019 +0200 >>> >>> Better error for empty header name >>> >>> Also add back logging for a while. >>> --- >>> conf/logging.properties | 2 +- >>> java/org/apache/coyote/http2/LocalStrings.properties | 1 + >>> java/org/apache/coyote/http2/Stream.java | 7 ++++++- >>> 3 files changed, 8 insertions(+), 2 deletions(-) >>> >>> diff --git a/conf/logging.properties b/conf/logging.properties >>> index e8b7b16..fc52083 100644 >>> --- a/conf/logging.properties >>> +++ b/conf/logging.properties >>> @@ -73,7 +73,7 @@ >> org.apache.catalina.core.ContainerBase.[Catalina].[localhost].[/host-manager].ha >>> #org.apache.jasper.compiler.TldLocationsCache.level = FINE >>> >>> # To see debug messages for HTTP/2 handling, uncomment the following >> line: >>> -#org.apache.coyote.http2.level = FINE >>> +org.apache.coyote.http2.level = FINE >> >> This looks like a accidental commit. >> > > "Also add back logging for a while." :) So it wasn't an accident.
ACK. Still getting used to the new format commit messages. Sorry for the noise. Mark > After tens of builds which were fine, right after I wrote "The testing > looks fine so far so I don't plan to switch the default back at this > time.", #4253 produced a relevant anomaly > https://ci.apache.org/projects/tomcat/tomcat9/logs/4253/TEST-org.apache.coyote.http2.TestStreamQueryString.NIO.txt, > with this exception trace: > Exception in thread "http-nio-127.0.0.1-auto-38-exec-3" > java.lang.RuntimeException: java.lang.StringIndexOutOfBoundsException: > String index out of range: 0 > at > org.apache.coyote.http2.Http2AsyncParser.handleAsyncException(Http2AsyncParser.java:80) > at > org.apache.coyote.http2.Http2AsyncParser.readFrame(Http2AsyncParser.java:56) > at org.apache.coyote.http2.Http2Parser.readFrame(Http2Parser.java:69) > at > org.apache.coyote.http2.Http2UpgradeHandler.upgradeDispatch(Http2UpgradeHandler.java:322) > at > org.apache.coyote.http2.Http2AsyncUpgradeHandler.upgradeDispatch(Http2AsyncUpgradeHandler.java:37) > at > org.apache.coyote.http2.Http2AsyncParser$FrameCompletionHandler.completed(Http2AsyncParser.java:231) > at > org.apache.coyote.http2.Http2AsyncParser$FrameCompletionHandler.completed(Http2AsyncParser.java:85) > at > org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper$VectoredIOCompletionHandler.completed(NioEndpoint.java:1721) > at > org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper$OperationState.run(NioEndpoint.java:1537) > at > java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) > at > java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) > at > org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61) > at java.lang.Thread.run(Thread.java:748) > Caused by: java.lang.StringIndexOutOfBoundsException: String index out of > range: 0 > at java.lang.String.charAt(String.java:658) > at org.apache.coyote.http2.Stream.emitHeader(Stream.java:350) > at > org.apache.coyote.http2.HpackDecoder.emitHeader(HpackDecoder.java:430) > at org.apache.coyote.http2.HpackDecoder.decode(HpackDecoder.java:138) > at > org.apache.coyote.http2.Http2Parser.readHeaderPayload(Http2Parser.java:474) > at > org.apache.coyote.http2.Http2Parser.readHeadersFrame(Http2Parser.java:267) > at > org.apache.coyote.http2.Http2AsyncParser$FrameCompletionHandler.completed(Http2AsyncParser.java:167) > ... 7 more > > Rémy > >> >> Mark >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org >> For additional commands, e-mail: dev-h...@tomcat.apache.org >> >> > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
