search

openresty / lua-resty-upstream-healthcheck

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

log whole response body on error to give a chance to understand a reason #6

Open sitano opened 9 years ago

sitano commented 9 years ago

usually response for /status? kind of requests would contain a description of why it failed

agentzh commented 9 years ago

@sitano Thanks for the patch and I understand this is more useful. However, your change breaks the existing test suite. I'm getting these failures:

https://gist.github.com/agentzh/adf13526aa1625d7e50b

Will you update the test suite accordingly as well? Thanks!