feat: allow to set client_max_body_size to 0 (#11)

Author: spacewander

src/ngx_http_apisix_module.c

@@ -304,6 +304,7 @@ ngx_http_apisix_client_set_max_body_size(ngx_http_request_t *r,
                    ctx->client_max_body_size);
 
     ctx->client_max_body_size = bytes;
+    ctx->client_max_body_size_set = 1;
 
     return NGX_OK;
 }
@@ -317,7 +318,7 @@ ngx_http_apisix_client_max_body_size(ngx_http_request_t *r)
 
     ctx = ngx_http_get_module_ctx(r, ngx_http_apisix_module);
 
-    if (ctx != NULL && ctx->client_max_body_size) {
+    if (ctx != NULL && ctx->client_max_body_size_set) {
         ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
                        "get client max body size %O",
                        ctx->client_max_body_size);

src/ngx_http_apisix_module.h

@@ -15,6 +15,8 @@ typedef struct {
     EVP_PKEY            *upstream_pkey;
 
     off_t                client_max_body_size;
+
+    unsigned             client_max_body_size_set:1;
 } ngx_http_apisix_ctx_t;
 
 

t/client.t

@@ -215,3 +215,22 @@ POST /t
 --- error_code: 413
 --- error_log
 client intended to send too large body
+
+
+
+=== TEST 11: set client_max_body_size to 0 means no limitation
+--- config
+    location /up {
+        return 200;
+    }
+    location /t {
+        client_max_body_size 1;
+        access_by_lua_block {
+            local client = require("resty.apisix.client")
+            assert(client.set_client_max_body_size(0))
+        }
+        proxy_pass http://127.0.0.1:1984/up;
+    }
+--- request
+POST /t
+1234