SkyeYoung commented on code in PR #12551:
URL: https://github.com/apache/apisix/pull/12551#discussion_r2309382550
##########
apisix/init.lua:
##########
@@ -290,21 +293,6 @@ end
local function set_upstream_headers(api_ctx, picked_server)
set_upstream_host(api_ctx, picked_server)
-
- local proto = api_ctx.var.http_x_forwarded_proto
- if proto then
- api_ctx.var.var_x_forwarded_proto = proto
- end
-
- local x_forwarded_host = api_ctx.var.http_x_forwarded_host
- if x_forwarded_host then
- api_ctx.var.var_x_forwarded_host = x_forwarded_host
- end
-
- local port = api_ctx.var.http_x_forwarded_port
- if port then
- api_ctx.var.var_x_forwarded_port = port
- end
Review Comment:
https://github.com/apache/apisix/blob/09f6e3682c5da1437f951273200d44cc18b13a28/apisix/plugins/ai.lua#L277-L282
Because the code above will replace `handle_upstream` with `ai_upstream`
below:
https://github.com/apache/apisix/blob/09f6e3682c5da1437f951273200d44cc18b13a28/apisix/plugins/ai.lua#L115-L117
This will cause this deleted part of the code to be skipped.
Now that we need to handle `X-Forwarded-*`, we need to update these
`var_x_forwarded_*`, so we need to extract this part of the code as
`update_var_x_forwarded_headers`.
https://github.com/apache/apisix/blob/3260931f301f741d5551af895cbfd6b12d1aff31/apisix/cli/ngx_tpl.lua#L835-L838
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
