You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by rp...@apache.org on 2020/06/17 18:33:21 UTC

svn commit: r1878936 - /httpd/httpd/trunk/modules/http2/h2_request.c

Author: rpluem
Date: Wed Jun 17 18:33:20 2020
New Revision: 1878936

URL: http://svn.apache.org/viewvc?rev=1878936&view=rev
Log:
* Add comments to document why we do this.
  No functional change. [skip ci].

Modified:
    httpd/httpd/trunk/modules/http2/h2_request.c

Modified: httpd/httpd/trunk/modules/http2/h2_request.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http2/h2_request.c?rev=1878936&r1=1878935&r2=1878936&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/http2/h2_request.c (original)
+++ httpd/httpd/trunk/modules/http2/h2_request.c Wed Jun 17 18:33:20 2020
@@ -278,6 +278,10 @@ request_rec *h2_request_create_rec(const
     
     /* Time to populate r with the data we have. */
     r->request_time = req->request_time;
+    /*
+     * Use HTTP/1.2 as ap_parse_request_line only deals with
+     * HTTP/1.x requests.
+     */
     r->the_request = apr_psprintf(r->pool, "%s %s HTTP/1.2", 
                                   req->method, req->path ? req->path : "");
     r->headers_in = apr_table_clone(r->pool, req->headers);
@@ -295,6 +299,7 @@ request_rec *h2_request_create_rec(const
         r->status = HTTP_OK;
         goto die;
     }
+    /* Note that this is actually a HTTP/2.0 request */
     r->protocol = "HTTP/2.0";
     r->proto_num = HTTP_VERSION(2, 0);