client-max-body-size is not always enforced when applied to a virtual server upstream

[kate-osborn]

Describe the bug
The client-max-body-size set on a virtual server upstream is not enforced when the following is true:
(1) the directive is set in an internal location block, and
(2) the value is greater than the default value of 1m for client_max_body_size.

To Reproduce
Steps to reproduce the behavior:

1. Deploy the ingress controller
2. Follow the Advanced Routing example
   but add client_max_body_size to the tea-post upstream. Set it to "2m".

apiVersion: k8s.nginx.org/v1
kind: VirtualServer
metadata:
  name: cafe
spec:
  host: cafe.example.com
  upstreams:
  - name: tea-post
    service: tea-post-svc
    port: 80
    client-max-body-size: "2m"
  - name: tea
    service: tea-svc
    port: 80
  - name: coffee-v1
    service: coffee-v1-svc
    port: 80
  - name: coffee-v2
    service: coffee-v2-svc
    port: 80
  routes:
  - path: /tea
    matches:
    - conditions:
      - variable: $request_method
        value: POST
      action:
        pass: tea-post
    action:
      pass: tea
  - path: /coffee
    matches:
    - conditions:
      - cookie: version
        value: v2
      action:
        pass: coffee-v2
    action:
      pass: coffee-v1

3. Send a POST request to /tea with a payload > 1m
   

Expected behavior
The client-max-body-size for the tea-post upstream should be enforced instead of the default client-max-body-size.

Your environment

• Version of the Ingress Controller - NGINX Ingress Controller Version=3.5.2 Commit=90d40829ca0b92e4a8ebcc36415c220e72da7405
• Version of Kubernetes: 1.30
• Kubernetes platform (e.g. Mini-kube or GCP): minikube
• Using NGINX or NGINX Plus: nginx

Additional context
This bug was initially found in NGINX Gateway Fabric and may apply to more than just the client_max_body_size directive.
The underlying issue is that nginx checks the body size of the request twice -- once in the external location block /tea and again in the internal location block that actually proxies the request to tea-post. If the request body exceeds the max body size in the external location block, a 413 is returned without nginx ever redirecting the request to the internal location block with the greater client_max_body_size. Even if client_max_body_size is not specified in the external location block, nginx will still check that the request body is under the default value of "1m". This is why this bug only applies when the internal client_max_body_size is greater than "1m".

Related: nginx/nginx-gateway-fabric#2079 and nginx/nginx-gateway-fabric#2105

[github-actions]

Hi @kate-osborn thanks for reporting!

Be sure to check out the docs and the Contributing Guidelines while you wait for a human to take a look at this 🙂

Cheers!

[haywoodsh]

This seems to be relevant to #5317

[github-actions]

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 10 days.

[github-actions]

This issue was closed because it has been stalled for 10 days with no activity.