pls add test cases

@halfcrazy $http_host ONLY get value from request header field, is that ok?
$host: in this order of precedence: host name from the request line, or host name from the “Host” request header field, or the server name matching a request
refer to: http://nginx.org/en/docs/http/ngx_http_core_module.html

proxy_set_header Host $host may lose port info, use http_host instead.

And is this PR anything with proxy_set_header?

proxy_set_header Host $host may lose port info, use http_host instead.

And is this PR anything with proxy_set_header?

Since we use proxy_set_header to set the Host field in header for upstream server. Nginx $host variable doesn't contain a port, as rfc2616 a request to ports besides 80, 443 should always include a port in the host field.

[1]https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.23

pls add test cases

Sorry, I have no idea how to add a test for this case.

pls add test cases

Sorry, I have no idea how to add a test for this case.

you can add test cases to test if port exists.

@halfcrazy https://github.com/apache/apisix/blob/master/.travis/apisix_cli_test.sh

you can take a look at this shell script for testing.

Test case added.

Test case added.

"proxy_set_header Host $host may lose port info, use http_host instead."

we need to add test for this case

ping @halfcrazy @membphis

Hi, May I ask the conclusion of this question? tks @halfcrazy

Hi, May I ask the conclusion of this question? tks @halfcrazy

There's a lot of change to apisix template since this pr, the $host indeed should change to $http_host but I don't familiar with Perl style test cant write a test for this.

let us reopen this PR, I think @Firstsawyou can help you, all right? @Firstsawyou

I think we need to process the conflict files first.

let us reopen this PR, I think @Firstsawyou can help you, all right? @Firstsawyou

Let me see.

hi @halfcrazy
The title of the PR should look like this:
fix: proxy_set_header Host $host may lose port info