ngx_http_proxy_create_request can be more friendly to the case about

xfeep · xfeep · commit 8ab657356c17 · 2023-05-01T12:37:42.000+08:00
removal of the headers in rewrite handlers #1
diff --git a/src/http/modules/ngx_http_proxy_module.c b/src/http/modules/ngx_http_proxy_module.c
@@ -1383,7 +1383,7 @@ ngx_http_proxy_create_request(ngx_http_request_t *r)
                 i = 0;
             }
 
-            if (ngx_hash_find(&headers->hash, header[i].hash,
+            if (header[i].hash == 0 || ngx_hash_find(&headers->hash, header[i].hash,
                               header[i].lowcase_key, header[i].key.len))
             {
                 continue;
@@ -1519,7 +1519,7 @@ ngx_http_proxy_create_request(ngx_http_request_t *r)
                 i = 0;
             }
 
-            if (ngx_hash_find(&headers->hash, header[i].hash,
+            if (header[i].hash == 0 || ngx_hash_find(&headers->hash, header[i].hash,
                               header[i].lowcase_key, header[i].key.len))
             {
                 continue;

Original file line number	Diff line number	Diff line change
`@@ -1383,7 +1383,7 @@ ngx_http_proxy_create_request(ngx_http_request_t *r)`
`1383`	`1383`	`i = 0;`
`1384`	`1384`	`}`
`1385`	`1385`
`1386`		`- if (ngx_hash_find(&headers->hash, header[i].hash,`
	`1386`	`+ if (header[i].hash == 0 \|\| ngx_hash_find(&headers->hash, header[i].hash,`
`1387`	`1387`	`header[i].lowcase_key, header[i].key.len))`
`1388`	`1388`	`{`
`1389`	`1389`	`continue;`
`@@ -1519,7 +1519,7 @@ ngx_http_proxy_create_request(ngx_http_request_t *r)`
`1519`	`1519`	`i = 0;`
`1520`	`1520`	`}`
`1521`	`1521`
`1522`		`- if (ngx_hash_find(&headers->hash, header[i].hash,`
	`1522`	`+ if (header[i].hash == 0 \|\| ngx_hash_find(&headers->hash, header[i].hash,`
`1523`	`1523`	`header[i].lowcase_key, header[i].key.len))`
`1524`	`1524`	`{`
`1525`	`1525`	`continue;`