Permalink
Browse files

Clarify README file

Spell out the lack of chunked encoding support
  • Loading branch information...
jlindamood
jlindamood committed Mar 19, 2010
1 parent 092020e commit aa5d5c22236590e20acdc8bb6993e0d60bb16b85
Showing with 11 additions and 1 deletion.
  1. +6 −0 README
  2. +5 −1 sample_ngx_config.conf
View
6 README
@@ -23,6 +23,12 @@ to the upstream_hash plugin here:
http://github.com/cep21/nginx_upstream_hash/tree/support_http_healthchecks
+==Limitations==
+The module only supports HTTP 1.0, not 1.1. What that really means is it
+doesn't understand chunked encoding. You should ask for a 1.0 reponse with
+your healthcheck, unless you're sure the upstream won't send back chunked
+encoding. See the sample config for an example.
+
==INSTALL==
# Similar to the upstream_hash module
View
@@ -19,8 +19,12 @@ http {
healthcheck_delay 1000;
healthcheck_timeout 1000;
healthcheck_failcount 1;
+ # Important: There is no \n at the end of this. Or \r. Make sure you
+ # don't have a \n or \r or anything else at the end of your healthcheck
+ # response
healthcheck_expected 'I_AM_ALIVE';
- healthcheck_send "GET /health HTTP/1.1" 'Host: www.mysite.com' 'Connection: close';
+ # Important: HTTP/1.0
+ healthcheck_send "GET /health HTTP/1.0" 'Host: www.mysite.com';
}
server {

0 comments on commit aa5d5c2

Please sign in to comment.