-1 (veto).
This was discussed several years ago, and the decision was to use "HTTP/2.0"
https://bz.apache.org/bugzilla/show_bug.cgi?id=58605

вс, 31 мар. 2019 г. в 11:04, <micha...@apache.org>:

>
> This is an automated email from the ASF dual-hosted git repository.
>
> michaelo pushed a commit to branch wrong-http2-version
> in repository https://gitbox.apache.org/repos/asf/tomcat.git
>
> commit 0e1de0d34302cdea6b3c2a47b03dcca4c7e2f9b7
> Author: Michael Osipov <micha...@apache.org>
> AuthorDate: Sun Mar 31 10:03:29 2019 +0200
>
>     Fix wrong protocol version usage
>
>     When serving a HTTP/2 request the protocol version was set as "HTTP/2.0"
>     which does not exist.
> ---
>  java/org/apache/coyote/http2/Stream.java | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/java/org/apache/coyote/http2/Stream.java 
> b/java/org/apache/coyote/http2/Stream.java
> index 3e64329..437279a 100644
> --- a/java/org/apache/coyote/http2/Stream.java
> +++ b/java/org/apache/coyote/http2/Stream.java
> @@ -126,7 +126,7 @@ class Stream extends AbstractStream implements 
> HeaderEmitter {
>          this.coyoteRequest.setSendfile(handler.hasAsyncIO() && 
> handler.getProtocol().getUseSendfile());
>          this.coyoteResponse.setOutputBuffer(http2OutputBuffer);
>          this.coyoteRequest.setResponse(coyoteResponse);
> -        this.coyoteRequest.protocol().setString("HTTP/2.0");
> +        this.coyoteRequest.protocol().setString("HTTP/2");
>          if (this.coyoteRequest.getStartTime() < 0) {
>              this.coyoteRequest.setStartTime(System.currentTimeMillis());
>          }
>
>
> ---------------------------------------------------------------------
> 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

Reply via email to