Commits on Apr 3, 2017

1. fix(balancer) do not send body from balancer ctx …

   The `kong.tools.responses` module sends a body with `ngx.say` when
   calling `responses.send_HTTP_SERVER_ERROR()`. This produces an error on
   top of a previous error, as in:
   
   ```
   2017/04/03 23:42:00 [error] 11076#0: *29 [lua] kong.lua:239: balancer(): failed to set the current peer (address:'2400:cb00:2048:1:0:0:681c:767' port:80): invalid port while connecting to upstream, client: 127.0.0.1, server: kong, request: "GET /status/200?apikey=secret123 HTTP/1.1", host: "acl_test1.com"
   
   2017/04/03 23:42:00 [error] 11076#0: *29 failed to run balancer_by_lua*: ...he/luarocks-2.4.2/share/lua/5.1/kong/tools/responses.lua:127: API disabled in the context of balancer_by_lua*
   
   stack traceback:
   	[C]: in function 'say'
   	...he/luarocks-2.4.2/share/lua/5.1/kong/tools/responses.lua:127: in function 'balancer'
   	balancer_by_lua:2: in function <balancer_by_lua:1> while connecting to upstream, client: 127.0.0.1, server: kong, request: "GET /status/200?apikey=secret123 HTTP/1.1", host: "acl_test1.com"'
   ```
   
   This is because, as pointed out in the error, `ngx.say` is not supported
   in the `balancer_by_lua` context.
   
   This logs the errors directly from the balancer context and simply sends
   the `500` status to the client.

   

   thibaultcha committed Apr 3, 2017
   Configuration menu

   • View commit details
   • Copy full SHA for 233e984
   • Browse repository at this point

   Copy the full SHA

   233e984 View commit details

   Browse the repository at this point in the history