annotate src/http/v3/ngx_http_v3_request.c @ 8685:dbe33ef9cd9a quic

HTTP/3: call ngx_handle_read_event() from client header handler. This function should be called at the end of an event handler to prepare the event for the next handler call. Particularly, the "active" flag is set or cleared depending on data availability. With this call missing in one code path, read handler was not called again after handling the initial part of the client request, if the request was too big to fit into a single STREAM frame. Now ngx_handle_read_event() is called in this code path. Also, read timer is restarted.
author Roman Arutyunyan <arut@nginx.com>
date Fri, 29 Jan 2021 19:42:47 +0300
parents 916a2e1d6617
children 6bd8ed493b85
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
1
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
2 /*
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
3 * Copyright (C) Roman Arutyunyan
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
4 * Copyright (C) Nginx, Inc.
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
5 */
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
6
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
7
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
8 #include <ngx_config.h>
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
9 #include <ngx_core.h>
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
10 #include <ngx_http.h>
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
11
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
12
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
13 static void ngx_http_v3_process_request(ngx_event_t *rev);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
14 static ngx_int_t ngx_http_v3_process_header(ngx_http_request_t *r,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
15 ngx_str_t *name, ngx_str_t *value);
8682
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
16 static ngx_int_t ngx_http_v3_validate_header(ngx_http_request_t *r,
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
17 ngx_str_t *name, ngx_str_t *value);
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
18 static ngx_int_t ngx_http_v3_process_pseudo_header(ngx_http_request_t *r,
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
19 ngx_str_t *name, ngx_str_t *value);
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
20 static ngx_int_t ngx_http_v3_init_pseudo_headers(ngx_http_request_t *r);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
21 static ngx_int_t ngx_http_v3_process_request_header(ngx_http_request_t *r);
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
22
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
23
8665
96eb6915d244 HTTP/3: staticize ngx_http_v3_methods.
Sergey Kandaurov <pluknet@nginx.com>
parents: 8653
diff changeset
24 static const struct {
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
25 ngx_str_t name;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
26 ngx_uint_t method;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
27 } ngx_http_v3_methods[] = {
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
28
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
29 { ngx_string("GET"), NGX_HTTP_GET },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
30 { ngx_string("POST"), NGX_HTTP_POST },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
31 { ngx_string("HEAD"), NGX_HTTP_HEAD },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
32 { ngx_string("OPTIONS"), NGX_HTTP_OPTIONS },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
33 { ngx_string("PROPFIND"), NGX_HTTP_PROPFIND },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
34 { ngx_string("PUT"), NGX_HTTP_PUT },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
35 { ngx_string("MKCOL"), NGX_HTTP_MKCOL },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
36 { ngx_string("DELETE"), NGX_HTTP_DELETE },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
37 { ngx_string("COPY"), NGX_HTTP_COPY },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
38 { ngx_string("MOVE"), NGX_HTTP_MOVE },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
39 { ngx_string("PROPPATCH"), NGX_HTTP_PROPPATCH },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
40 { ngx_string("LOCK"), NGX_HTTP_LOCK },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
41 { ngx_string("UNLOCK"), NGX_HTTP_UNLOCK },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
42 { ngx_string("PATCH"), NGX_HTTP_PATCH },
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
43 { ngx_string("TRACE"), NGX_HTTP_TRACE }
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
44 };
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
45
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
46
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
47 void
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
48 ngx_http_v3_init(ngx_connection_t *c)
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
49 {
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
50 size_t size;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
51 ngx_buf_t *b;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
52 ngx_event_t *rev;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
53 ngx_http_request_t *r;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
54 ngx_http_connection_t *hc;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
55 ngx_http_core_srv_conf_t *cscf;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
56
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
57 if (ngx_http_v3_init_session(c) != NGX_OK) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
58 ngx_http_v3_finalize_connection(c, NGX_HTTP_V3_ERR_INTERNAL_ERROR,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
59 "internal error");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
60 ngx_http_close_connection(c);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
61 return;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
62 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
63
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
64 if (c->quic->id & NGX_QUIC_STREAM_UNIDIRECTIONAL) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
65 ngx_http_v3_init_uni_stream(c);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
66 return;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
67 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
68
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
69 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, c->log, 0, "http3 init request stream");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
70
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
71 hc = c->data;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
72
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
73 cscf = ngx_http_get_module_srv_conf(hc->conf_ctx, ngx_http_core_module);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
74
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
75 size = cscf->client_header_buffer_size;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
76
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
77 b = c->buffer;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
78
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
79 if (b == NULL) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
80 b = ngx_create_temp_buf(c->pool, size);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
81 if (b == NULL) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
82 ngx_http_close_connection(c);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
83 return;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
84 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
85
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
86 c->buffer = b;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
87
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
88 } else if (b->start == NULL) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
89
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
90 b->start = ngx_palloc(c->pool, size);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
91 if (b->start == NULL) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
92 ngx_http_close_connection(c);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
93 return;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
94 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
95
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
96 b->pos = b->start;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
97 b->last = b->start;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
98 b->end = b->last + size;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
99 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
100
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
101 c->log->action = "reading client request";
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
102
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
103 r = ngx_http_create_request(c);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
104 if (r == NULL) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
105 ngx_http_close_connection(c);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
106 return;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
107 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
108
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
109 r->http_version = NGX_HTTP_VERSION_30;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
110
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
111 c->data = r;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
112
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
113 rev = c->read;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
114 rev->handler = ngx_http_v3_process_request;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
115
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
116 ngx_http_v3_process_request(rev);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
117 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
118
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
119
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
120 static void
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
121 ngx_http_v3_process_request(ngx_event_t *rev)
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
122 {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
123 ssize_t n;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
124 ngx_buf_t *b;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
125 ngx_int_t rc;
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
126 ngx_connection_t *c;
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
127 ngx_http_request_t *r;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
128 ngx_http_core_srv_conf_t *cscf;
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
129 ngx_http_v3_parse_headers_t *st;
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
130
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
131 c = rev->data;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
132 r = c->data;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
133
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
134 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, rev->log, 0, "http3 process request");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
135
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
136 if (rev->timedout) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
137 ngx_log_error(NGX_LOG_INFO, c->log, NGX_ETIMEDOUT, "client timed out");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
138 c->timedout = 1;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
139 ngx_http_close_request(r, NGX_HTTP_REQUEST_TIME_OUT);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
140 return;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
141 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
142
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
143 st = r->h3_parse;
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
144
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
145 if (st == NULL) {
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
146 st = ngx_pcalloc(c->pool, sizeof(ngx_http_v3_parse_headers_t));
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
147 if (st == NULL) {
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
148 ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
149 return;
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
150 }
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
151
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
152 r->h3_parse = st;
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
153 }
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
154
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
155 b = r->header_in;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
156
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
157 for ( ;; ) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
158
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
159 if (b->pos == b->last) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
160
8685
dbe33ef9cd9a HTTP/3: call ngx_handle_read_event() from client header handler.
Roman Arutyunyan <arut@nginx.com>
parents: 8682
diff changeset
161 if (rev->ready) {
dbe33ef9cd9a HTTP/3: call ngx_handle_read_event() from client header handler.
Roman Arutyunyan <arut@nginx.com>
parents: 8682
diff changeset
162 n = c->recv(c, b->start, b->end - b->start);
dbe33ef9cd9a HTTP/3: call ngx_handle_read_event() from client header handler.
Roman Arutyunyan <arut@nginx.com>
parents: 8682
diff changeset
163
dbe33ef9cd9a HTTP/3: call ngx_handle_read_event() from client header handler.
Roman Arutyunyan <arut@nginx.com>
parents: 8682
diff changeset
164 } else {
dbe33ef9cd9a HTTP/3: call ngx_handle_read_event() from client header handler.
Roman Arutyunyan <arut@nginx.com>
parents: 8682
diff changeset
165 n = NGX_AGAIN;
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
166 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
167
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
168 if (n == NGX_AGAIN) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
169 if (!rev->timer_set) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
170 cscf = ngx_http_get_module_srv_conf(r,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
171 ngx_http_core_module);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
172 ngx_add_timer(rev, cscf->client_header_timeout);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
173 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
174
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
175 if (ngx_handle_read_event(rev, 0) != NGX_OK) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
176 ngx_http_close_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
177 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
178
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
179 break;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
180 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
181
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
182 if (n == 0) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
183 ngx_log_error(NGX_LOG_INFO, c->log, 0,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
184 "client prematurely closed connection");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
185 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
186
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
187 if (n == 0 || n == NGX_ERROR) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
188 c->error = 1;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
189 c->log->action = "reading client request";
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
190
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
191 ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
192 break;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
193 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
194
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
195 b->pos = b->start;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
196 b->last = b->start + n;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
197 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
198
8456
c9538aef3211 HTTP/3: refactored dynamic table implementation.
Roman Arutyunyan <arut@nginx.com>
parents: 8452
diff changeset
199 rc = ngx_http_v3_parse_headers(c, st, *b->pos);
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
200
8460
72f9ff4e0a88 HTTP/3: close QUIC connection with HTTP/QPACK errors when needed.
Roman Arutyunyan <arut@nginx.com>
parents: 8456
diff changeset
201 if (rc > 0) {
72f9ff4e0a88 HTTP/3: close QUIC connection with HTTP/QPACK errors when needed.
Roman Arutyunyan <arut@nginx.com>
parents: 8456
diff changeset
202 ngx_http_v3_finalize_connection(c, rc,
72f9ff4e0a88 HTTP/3: close QUIC connection with HTTP/QPACK errors when needed.
Roman Arutyunyan <arut@nginx.com>
parents: 8456
diff changeset
203 "could not parse request headers");
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
204 ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
205 break;
8460
72f9ff4e0a88 HTTP/3: close QUIC connection with HTTP/QPACK errors when needed.
Roman Arutyunyan <arut@nginx.com>
parents: 8456
diff changeset
206 }
72f9ff4e0a88 HTTP/3: close QUIC connection with HTTP/QPACK errors when needed.
Roman Arutyunyan <arut@nginx.com>
parents: 8456
diff changeset
207
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
208 if (rc == NGX_ERROR) {
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
209 ngx_http_v3_finalize_connection(c, NGX_HTTP_V3_ERR_INTERNAL_ERROR,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
210 "internal error");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
211 ngx_http_close_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
212 break;
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
213 }
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
214
8456
c9538aef3211 HTTP/3: refactored dynamic table implementation.
Roman Arutyunyan <arut@nginx.com>
parents: 8452
diff changeset
215 if (rc == NGX_BUSY) {
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
216 if (rev->error) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
217 ngx_http_close_request(r, NGX_HTTP_CLOSE);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
218 break;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
219 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
220
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
221 if (ngx_handle_read_event(rev, 0) != NGX_OK) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
222 ngx_http_close_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
223 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
224
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
225 break;
8456
c9538aef3211 HTTP/3: refactored dynamic table implementation.
Roman Arutyunyan <arut@nginx.com>
parents: 8452
diff changeset
226 }
c9538aef3211 HTTP/3: refactored dynamic table implementation.
Roman Arutyunyan <arut@nginx.com>
parents: 8452
diff changeset
227
c9538aef3211 HTTP/3: refactored dynamic table implementation.
Roman Arutyunyan <arut@nginx.com>
parents: 8452
diff changeset
228 b->pos++;
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
229 r->request_length++;
8456
c9538aef3211 HTTP/3: refactored dynamic table implementation.
Roman Arutyunyan <arut@nginx.com>
parents: 8452
diff changeset
230
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
231 if (rc == NGX_AGAIN) {
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
232 continue;
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
233 }
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
234
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
235 /* rc == NGX_OK || rc == NGX_DONE */
8233
1e45c02f6376 HTTP/3 $request_line variable.
Roman Arutyunyan <arut@nginx.com>
parents: 8230
diff changeset
236
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
237 if (ngx_http_v3_process_header(r, &st->header_rep.header.name,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
238 &st->header_rep.header.value)
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
239 != NGX_OK)
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
240 {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
241 break;
8405
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
242 }
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
243
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
244 if (rc == NGX_DONE) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
245 if (ngx_http_v3_process_request_header(r) != NGX_OK) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
246 break;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
247 }
8405
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
248
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
249 ngx_http_process_request(r);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
250 break;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
251 }
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
252 }
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
253
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
254 ngx_http_run_posted_requests(c);
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
255
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
256 return;
8405
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
257 }
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
258
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
259
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
260 static ngx_int_t
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
261 ngx_http_v3_process_header(ngx_http_request_t *r, ngx_str_t *name,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
262 ngx_str_t *value)
8405
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
263 {
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
264 ngx_table_elt_t *h;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
265 ngx_http_header_t *hh;
8682
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
266 ngx_http_core_srv_conf_t *cscf;
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
267 ngx_http_core_main_conf_t *cmcf;
8452
a6675a976560 HTTP/3: fixed dropping first non-pseudo header.
Roman Arutyunyan <arut@nginx.com>
parents: 8451
diff changeset
268
8682
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
269 if (ngx_http_v3_validate_header(r, name, value) != NGX_OK) {
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
270 ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST);
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
271 return NGX_ERROR;
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
272 }
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
273
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
274 if (r->invalid_header) {
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
275 cscf = ngx_http_get_module_srv_conf(r, ngx_http_core_module);
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
276
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
277 if (cscf->ignore_invalid_headers) {
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
278 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
279 "client sent invalid header: \"%V\"", name);
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
280
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
281 return NGX_OK;
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
282 }
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
283 }
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
284
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
285 if (name->len && name->data[0] == ':') {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
286 return ngx_http_v3_process_pseudo_header(r, name, value);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
287 }
8452
a6675a976560 HTTP/3: fixed dropping first non-pseudo header.
Roman Arutyunyan <arut@nginx.com>
parents: 8451
diff changeset
288
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
289 if (ngx_http_v3_init_pseudo_headers(r) != NGX_OK) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
290 return NGX_ERROR;
8405
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
291 }
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
292
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
293 h = ngx_list_push(&r->headers_in.headers);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
294 if (h == NULL) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
295 ngx_http_close_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
296 return NGX_ERROR;
8405
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
297 }
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
298
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
299 h->key = *name;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
300 h->value = *value;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
301 h->lowcase_key = h->key.data;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
302 h->hash = ngx_hash_key(h->key.data, h->key.len);
8406
66feab03d9b7 HTTP/3: restricted symbols in header names.
Roman Arutyunyan <arut@nginx.com>
parents: 8405
diff changeset
303
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
304 cmcf = ngx_http_get_module_main_conf(r, ngx_http_core_module);
8405
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
305
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
306 hh = ngx_hash_find(&cmcf->headers_in_hash, h->hash,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
307 h->lowcase_key, h->key.len);
8406
66feab03d9b7 HTTP/3: restricted symbols in header names.
Roman Arutyunyan <arut@nginx.com>
parents: 8405
diff changeset
308
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
309 if (hh && hh->handler(r, h, hh->offset) != NGX_OK) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
310 return NGX_ERROR;
8406
66feab03d9b7 HTTP/3: restricted symbols in header names.
Roman Arutyunyan <arut@nginx.com>
parents: 8405
diff changeset
311 }
66feab03d9b7 HTTP/3: restricted symbols in header names.
Roman Arutyunyan <arut@nginx.com>
parents: 8405
diff changeset
312
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
313 ngx_log_debug2(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
314 "http3 header: \"%V: %V\"", name, value);
8405
d2759e4cc437 HTTP/3: split header parser in two functions.
Roman Arutyunyan <arut@nginx.com>
parents: 8394
diff changeset
315 return NGX_OK;
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
316 }
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
317
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
318
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
319 static ngx_int_t
8682
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
320 ngx_http_v3_validate_header(ngx_http_request_t *r, ngx_str_t *name,
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
321 ngx_str_t *value)
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
322 {
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
323 u_char ch;
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
324 ngx_uint_t i;
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
325 ngx_http_core_srv_conf_t *cscf;
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
326
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
327 r->invalid_header = 0;
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
328
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
329 cscf = ngx_http_get_module_srv_conf(r, ngx_http_core_module);
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
330
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
331 for (i = (name->data[0] == ':'); i != name->len; i++) {
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
332 ch = name->data[i];
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
333
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
334 if ((ch >= 'a' && ch <= 'z')
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
335 || (ch == '-')
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
336 || (ch >= '0' && ch <= '9')
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
337 || (ch == '_' && cscf->underscores_in_headers))
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
338 {
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
339 continue;
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
340 }
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
341
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
342 if (ch == '\0' || ch == LF || ch == CR || ch == ':'
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
343 || (ch >= 'A' && ch <= 'Z'))
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
344 {
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
345 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
346 "client sent invalid header name: \"%V\"", name);
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
347
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
348 return NGX_ERROR;
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
349 }
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
350
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
351 r->invalid_header = 1;
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
352 }
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
353
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
354 for (i = 0; i != value->len; i++) {
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
355 ch = value->data[i];
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
356
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
357 if (ch == '\0' || ch == LF || ch == CR) {
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
358 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
359 "client sent header \"%V\" with "
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
360 "invalid value: \"%V\"", name, value);
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
361
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
362 return NGX_ERROR;
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
363 }
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
364 }
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
365
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
366 return NGX_OK;
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
367 }
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
368
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
369
916a2e1d6617 HTTP/3: client header validation.
Roman Arutyunyan <arut@nginx.com>
parents: 8680
diff changeset
370 static ngx_int_t
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
371 ngx_http_v3_process_pseudo_header(ngx_http_request_t *r, ngx_str_t *name,
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
372 ngx_str_t *value)
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
373 {
8680
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
374 u_char ch, c;
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
375 ngx_uint_t i;
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
376
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
377 if (r->request_line.len) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
378 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
379 "client sent out of order pseudo-headers");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
380 goto failed;
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
381 }
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
382
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
383 if (name->len == 7 && ngx_strncmp(name->data, ":method", 7) == 0) {
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
384
8680
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
385 if (r->method_name.len) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
386 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
387 "client sent duplicate \":method\" header");
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
388 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
389 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
390
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
391 if (value->len == 0) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
392 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
393 "client sent empty \":method\" header");
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
394 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
395 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
396
8650
9dce2978e4fd HTTP/3: eliminated r->method_start.
Roman Arutyunyan <arut@nginx.com>
parents: 8630
diff changeset
397 r->method_name = *value;
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
398
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
399 for (i = 0; i < sizeof(ngx_http_v3_methods)
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
400 / sizeof(ngx_http_v3_methods[0]); i++)
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
401 {
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
402 if (value->len == ngx_http_v3_methods[i].name.len
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
403 && ngx_strncmp(value->data,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
404 ngx_http_v3_methods[i].name.data, value->len)
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
405 == 0)
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
406 {
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
407 r->method = ngx_http_v3_methods[i].method;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
408 break;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
409 }
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
410 }
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
411
8680
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
412 for (i = 0; i < value->len; i++) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
413 ch = value->data[i];
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
414
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
415 if ((ch < 'A' || ch > 'Z') && ch != '_' && ch != '-') {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
416 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
417 "client sent invalid method: \"%V\"", value);
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
418 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
419 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
420 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
421
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
422 ngx_log_debug2(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
423 "http3 method \"%V\" %ui", value, r->method);
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
424 return NGX_OK;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
425 }
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
426
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
427 if (name->len == 5 && ngx_strncmp(name->data, ":path", 5) == 0) {
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
428
8680
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
429 if (r->uri_start) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
430 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
431 "client sent duplicate \":path\" header");
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
432 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
433 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
434
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
435 if (value->len == 0) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
436 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
437 "client sent empty \":path\" header");
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
438 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
439 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
440
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
441 r->uri_start = value->data;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
442 r->uri_end = value->data + value->len;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
443
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
444 if (ngx_http_parse_uri(r) != NGX_OK) {
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
445 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
446 "client sent invalid \":path\" header: \"%V\"",
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
447 value);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
448 goto failed;
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
449 }
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
450
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
451 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
452 "http3 path \"%V\"", value);
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
453 return NGX_OK;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
454 }
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
455
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
456 if (name->len == 7 && ngx_strncmp(name->data, ":scheme", 7) == 0) {
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
457
8680
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
458 if (r->schema.len) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
459 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
460 "client sent duplicate \":scheme\" header");
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
461 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
462 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
463
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
464 if (value->len == 0) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
465 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
466 "client sent empty \":scheme\" header");
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
467 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
468 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
469
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
470 for (i = 0; i < value->len; i++) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
471 ch = value->data[i];
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
472
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
473 c = (u_char) (ch | 0x20);
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
474 if (c >= 'a' && c <= 'z') {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
475 continue;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
476 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
477
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
478 if (((ch >= '0' && ch <= '9')
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
479 || ch == '+' || ch == '-' || ch == '.')
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
480 && i > 0)
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
481 {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
482 continue;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
483 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
484
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
485 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
486 "client sent invalid \":scheme\" header: \"%V\"",
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
487 value);
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
488 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
489 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
490
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
491 r->schema = *value;
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
492
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
493 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
494 "http3 schema \"%V\"", value);
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
495 return NGX_OK;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
496 }
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
497
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
498 if (name->len == 10 && ngx_strncmp(name->data, ":authority", 10) == 0) {
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
499
8680
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
500 if (r->host_start) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
501 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
502 "client sent duplicate \":authority\" header");
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
503 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
504 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
505
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
506 r->host_start = value->data;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
507 r->host_end = value->data + value->len;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
508
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
509 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
510 "http3 authority \"%V\"", value);
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
511 return NGX_OK;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
512 }
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
513
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
514 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
515 "client sent unknown pseudo-header \"%V\"", name);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
516
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
517 failed:
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
518
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
519 ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
520 return NGX_ERROR;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
521 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
522
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
523
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
524 static ngx_int_t
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
525 ngx_http_v3_init_pseudo_headers(ngx_http_request_t *r)
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
526 {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
527 size_t len;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
528 u_char *p;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
529 ngx_int_t rc;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
530 ngx_str_t host;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
531
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
532 if (r->request_line.len) {
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
533 return NGX_OK;
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
534 }
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
535
8680
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
536 if (r->method_name.len == 0) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
537 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
538 "client sent no \":method\" header");
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
539 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
540 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
541
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
542 if (r->schema.len == 0) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
543 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
544 "client sent no \":scheme\" header");
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
545 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
546 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
547
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
548 if (r->uri_start == NULL) {
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
549 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
550 "client sent no \":path\" header");
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
551 goto failed;
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
552 }
58acdba9b3b2 HTTP/3: client pseudo-headers restrictions.
Roman Arutyunyan <arut@nginx.com>
parents: 8679
diff changeset
553
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
554 len = r->method_name.len + 1
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
555 + (r->uri_end - r->uri_start) + 1
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
556 + sizeof("HTTP/3.0") - 1;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
557
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
558 p = ngx_pnalloc(r->pool, len);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
559 if (p == NULL) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
560 ngx_http_close_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
561 return NGX_ERROR;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
562 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
563
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
564 r->request_line.data = p;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
565
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
566 p = ngx_cpymem(p, r->method_name.data, r->method_name.len);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
567 *p++ = ' ';
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
568 p = ngx_cpymem(p, r->uri_start, r->uri_end - r->uri_start);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
569 *p++ = ' ';
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
570 p = ngx_cpymem(p, "HTTP/3.0", sizeof("HTTP/3.0") - 1);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
571
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
572 r->request_line.len = p - r->request_line.data;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
573
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
574 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
575 "http3 request line: \"%V\"", &r->request_line);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
576
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
577 ngx_str_set(&r->http_protocol, "HTTP/3.0");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
578
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
579 if (ngx_http_process_request_uri(r) != NGX_OK) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
580 return NGX_ERROR;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
581 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
582
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
583 if (r->host_end) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
584
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
585 host.len = r->host_end - r->host_start;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
586 host.data = r->host_start;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
587
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
588 rc = ngx_http_validate_host(&host, r->pool, 0);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
589
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
590 if (rc == NGX_DECLINED) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
591 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
592 "client sent invalid host in request line");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
593 goto failed;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
594 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
595
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
596 if (rc == NGX_ERROR) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
597 ngx_http_close_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
598 return NGX_ERROR;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
599 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
600
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
601 if (ngx_http_set_virtual_server(r, &host) == NGX_ERROR) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
602 return NGX_ERROR;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
603 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
604
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
605 r->headers_in.server = host;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
606 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
607
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
608 if (ngx_list_init(&r->headers_in.headers, r->pool, 20,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
609 sizeof(ngx_table_elt_t))
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
610 != NGX_OK)
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
611 {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
612 ngx_http_close_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
613 return NGX_ERROR;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
614 }
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
615
8226
268f4389130d Refactored HTTP/3 parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8219
diff changeset
616 return NGX_OK;
8679
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
617
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
618 failed:
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
619
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
620 ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
621 return NGX_ERROR;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
622 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
623
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
624
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
625 static ngx_int_t
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
626 ngx_http_v3_process_request_header(ngx_http_request_t *r)
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
627 {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
628 if (ngx_http_v3_init_pseudo_headers(r) != NGX_OK) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
629 return NGX_ERROR;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
630 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
631
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
632 if (r->headers_in.server.len == 0) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
633 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
634 "client sent neither \":authority\" nor \"Host\" header");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
635 goto failed;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
636 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
637
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
638 if (r->headers_in.host) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
639 if (r->headers_in.host->value.len != r->headers_in.server.len
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
640 || ngx_memcmp(r->headers_in.host->value.data,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
641 r->headers_in.server.data,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
642 r->headers_in.server.len)
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
643 != 0)
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
644 {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
645 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
646 "client sent \":authority\" and \"Host\" headers "
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
647 "with different values");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
648 goto failed;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
649 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
650 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
651
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
652 if (r->headers_in.content_length) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
653 r->headers_in.content_length_n =
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
654 ngx_atoof(r->headers_in.content_length->value.data,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
655 r->headers_in.content_length->value.len);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
656
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
657 if (r->headers_in.content_length_n == NGX_ERROR) {
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
658 ngx_log_error(NGX_LOG_INFO, r->connection->log, 0,
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
659 "client sent invalid \"Content-Length\" header");
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
660 goto failed;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
661 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
662 }
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
663
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
664 return NGX_OK;
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
665
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
666 failed:
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
667
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
668 ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST);
e1eb7f4ca9f1 HTTP/3: refactored request parser.
Roman Arutyunyan <arut@nginx.com>
parents: 8665
diff changeset
669 return NGX_ERROR;
8215
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
670 }
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
671
38c0898b6df7 HTTP/3.
Roman Arutyunyan <arut@nginx.com>
parents:
diff changeset
672
8295
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
673 ngx_int_t
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
674 ngx_http_v3_parse_request_body(ngx_http_request_t *r, ngx_buf_t *b,
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
675 ngx_http_chunked_t *ctx)
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
676 {
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
677 ngx_int_t rc;
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
678 ngx_connection_t *c;
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
679 ngx_http_v3_parse_data_t *st;
8548
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
680 enum {
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
681 sw_start = 0,
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
682 sw_skip
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
683 };
8295
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
684
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
685 c = r->connection;
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
686 st = ctx->h3_parse;
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
687
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
688 if (st == NULL) {
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
689 ngx_log_debug0(NGX_LOG_DEBUG_HTTP, c->log, 0,
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
690 "http3 parse request body");
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
691
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
692 st = ngx_pcalloc(c->pool, sizeof(ngx_http_v3_parse_data_t));
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
693 if (st == NULL) {
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
694 goto failed;
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
695 }
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
696
8511
830680e78b24 HTTP/3: fixed context storage in request body parser.
Sergey Kandaurov <pluknet@nginx.com>
parents: 8497
diff changeset
697 ctx->h3_parse = st;
8295
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
698 }
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
699
8548
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
700 while (b->pos < b->last && ctx->size == 0) {
8295
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
701
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
702 rc = ngx_http_v3_parse_data(c, st, *b->pos++);
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
703
8460
72f9ff4e0a88 HTTP/3: close QUIC connection with HTTP/QPACK errors when needed.
Roman Arutyunyan <arut@nginx.com>
parents: 8456
diff changeset
704 if (rc > 0) {
72f9ff4e0a88 HTTP/3: close QUIC connection with HTTP/QPACK errors when needed.
Roman Arutyunyan <arut@nginx.com>
parents: 8456
diff changeset
705 ngx_http_v3_finalize_connection(c, rc,
72f9ff4e0a88 HTTP/3: close QUIC connection with HTTP/QPACK errors when needed.
Roman Arutyunyan <arut@nginx.com>
parents: 8456
diff changeset
706 "could not parse request body");
72f9ff4e0a88 HTTP/3: close QUIC connection with HTTP/QPACK errors when needed.
Roman Arutyunyan <arut@nginx.com>
parents: 8456
diff changeset
707 goto failed;
72f9ff4e0a88 HTTP/3: close QUIC connection with HTTP/QPACK errors when needed.
Roman Arutyunyan <arut@nginx.com>
parents: 8456
diff changeset
708 }
72f9ff4e0a88 HTTP/3: close QUIC connection with HTTP/QPACK errors when needed.
Roman Arutyunyan <arut@nginx.com>
parents: 8456
diff changeset
709
8295
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
710 if (rc == NGX_ERROR) {
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
711 goto failed;
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
712 }
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
713
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
714 if (rc == NGX_AGAIN) {
8548
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
715 ctx->state = sw_skip;
8295
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
716 continue;
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
717 }
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
718
8549
d70a38acaea0 HTTP/3: skip unknown frames on request stream.
Roman Arutyunyan <arut@nginx.com>
parents: 8548
diff changeset
719 if (rc == NGX_DONE) {
d70a38acaea0 HTTP/3: skip unknown frames on request stream.
Roman Arutyunyan <arut@nginx.com>
parents: 8548
diff changeset
720 return NGX_DONE;
d70a38acaea0 HTTP/3: skip unknown frames on request stream.
Roman Arutyunyan <arut@nginx.com>
parents: 8548
diff changeset
721 }
d70a38acaea0 HTTP/3: skip unknown frames on request stream.
Roman Arutyunyan <arut@nginx.com>
parents: 8548
diff changeset
722
d70a38acaea0 HTTP/3: skip unknown frames on request stream.
Roman Arutyunyan <arut@nginx.com>
parents: 8548
diff changeset
723 /* rc == NGX_OK */
8295
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
724
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
725 ctx->size = st->length;
8548
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
726 ctx->state = sw_start;
8295
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
727 }
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
728
8548
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
729 if (ctx->state == sw_skip) {
8295
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
730 ctx->length = 1;
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
731 return NGX_AGAIN;
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
732 }
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
733
8548
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
734 if (b->pos == b->last) {
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
735 ctx->length = ctx->size;
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
736 return NGX_AGAIN;
8295
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
737 }
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
738
8548
9ffef6054abf HTTP/3: fixed handling request body eof.
Roman Arutyunyan <arut@nginx.com>
parents: 8511
diff changeset
739 return NGX_OK;
8295
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
740
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
741 failed:
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
742
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
743 return NGX_ERROR;
5649079a41f4 Parsing HTTP/3 request body.
Roman Arutyunyan <arut@nginx.com>
parents: 8292
diff changeset
744 }