search

openresty / lua-resty-upstream-healthcheck

Health Checker for Nginx Upstream Servers in Pure Lua
521 stars 134 forks source link

Support for specifying custom port #28

Closed bompus closed 8 years ago

bompus commented 8 years ago

Would it be possible to support the following?

type = "http", port = 80

This would be a reasonable workaround for those needing support for SSL backends, as long as they also listen on port 80, it's a good enough health check for me, and should be much easier to implement than adding type = "https"

agentzh commented 8 years ago

@bompus It's not possible since the servers defined in upstream {} already have port numbers (either explicitly or implicitly). Overriding port numbers with force is too hacky to be accepted and may lead to hard-to-debug issues.