elvis-cai commented on issue #4906: URL: https://github.com/apache/apisix/issues/4906#issuecomment-908259817
Hi @tzssangglass thanks for checking, unfortunately, it doesn't work for me, after some post checks, I found something interesting here. - from the lua script, active health check is performed here https://github.com/apache/apisix/blob/master/apisix/balancer.lua#L75, active health check host name is obtained from `upstream.checks.active.host`, so if I add this field, it will work if my `upstream.nodes` only has `apple.internal.com` one entry, however in my case, it won't work, since I have two nodes one is `apple.internal.com`, another is `banana.internal.com`, as you may know, I am using **_"pass_host": "node"_** to pass each host header, so I would like active health check to do the same thing which is to read Host header from `upstream.nodes` rather than `upstream.checks.active.host` since it only support one host header, hopefully it explain the issue more detailed, thanks. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
