This is an automated email from the ASF dual-hosted git repository.

github-merge-queue[bot] pushed a commit to branch 
gh-readonly-queue/main/pr-8722-63db8e3f43cdbbc5c031321727bbf937fbbdc6ea
in repository https://gitbox.apache.org/repos/asf/texera.git

commit 564070a56b0a184b1cb1dd4e61a943915a5e79d7
Author: Xuan Gu <[email protected]>
AuthorDate: Sun Sep 27 23:40:21 2026 +0000

    fix(deploy): lift nginx's 1 MiB body cap that rejects dataset upload parts 
(#8722)
    
    ### What changes were proposed in this PR?
    
    This PR adds two lines in the single-node nginx config's /api/dataset
    location: client_max_body_size 0; and proxy_request_buffering off;.
    
    nginx defaults client_max_body_size to 1 MiB, and the config never set
    it, so every dataset upload part larger than 1 MiB (the frontend default
    is 50 MiB) is rejected with 413 before reaching file-service. The upload
    size stays governed by the admin-configurable chunk setting; nginx no
    longer imposes a second, hidden cap.
    
    ### Any related issues, documentation, discussions?
    
    Closes #8720.
    
    ### How was this PR tested?
    
    Tested on single node deployment.
    
    ### Was this PR authored or co-authored using generative AI tooling?
    
    Generated-by: Claude Fable 5
    
    ---------
    
    Co-authored-by: Claude Fable 5 <[email protected]>
---
 bin/single-node/nginx.conf | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/bin/single-node/nginx.conf b/bin/single-node/nginx.conf
index 688e256d46..250e4cfcb0 100644
--- a/bin/single-node/nginx.conf
+++ b/bin/single-node/nginx.conf
@@ -34,6 +34,8 @@ http {
         }
 
         location /api/dataset {
+            client_max_body_size 0;
+            proxy_request_buffering off;
             proxy_pass http://file-service:9092;
             proxy_set_header Host $host;
             proxy_set_header X-Real-IP $remote_addr;
@@ -49,6 +51,8 @@ http {
         # is an exact match (location = /api/models), which outranks any prefix
         # location, so /api/models is unambiguous either way.
         location /api/model/ {
+            client_max_body_size 0;
+            proxy_request_buffering off;
             proxy_pass http://file-service:9092;
             proxy_set_header Host $host;
             proxy_set_header X-Real-IP $remote_addr;

Reply via email to