throw out a Lua exception instead of nginx error logging

Author: rainingmaster

src/ngx_http_lua_control.c

@@ -330,9 +330,8 @@ ngx_http_lua_ngx_exit(lua_State *L)
     if (ctx->context & NGX_HTTP_LUA_CONTEXT_BODY_FILTER) {
 
         if (rc != NGX_ERROR && rc < 400) {
-            ngx_log_error(NGX_LOG_ERR, r->connection->log, 0, "attempt to "
-                          "exit with the code smaller than 400 and not "
-                          "ngx.ERROR");
+            return luaL_error(L, "attempt to exit with the code smaller "
+                              "than 400 and not ngx.ERROR");
         }
 
         ctx->body_exited = 1;
@@ -497,9 +496,10 @@ ngx_http_lua_ffi_exit(ngx_http_request_t *r, int status, u_char *err,
     if (ctx->context & NGX_HTTP_LUA_CONTEXT_BODY_FILTER) {
 
         if (status != NGX_ERROR && status < 400) {
-            ngx_log_error(NGX_LOG_ERR, r->connection->log, 0, "attempt to "
-                          "exit with the code smaller than 400 and not "
-                          "ngx.ERROR");
+            *errlen = ngx_snprintf(err, *errlen, "attempt to exit with the "
+                                   "code smaller than 400 and not ngx.ERROR")
+                      - err;
+            return NGX_ERROR;
         }
 
         ctx->body_exited = 1;

t/082-body-filter.t

@@ -10,7 +10,7 @@ log_level('debug');
 
 repeat_each(2);
 
-plan tests => repeat_each() * (blocks() * 3 + 13);
+plan tests => repeat_each() * (blocks() * 3 + 11);
 
 #no_diff();
 no_long_string();
@@ -879,9 +879,7 @@ API disabled in the context of body_filter_by_lua*
     }
 --- request
     GET /t
---- response_body
-111
---- error_code: 200
+--- ignore_response
 --- error_log
 attempt to exit with the code smaller than 400 and not ngx.ERROR