Skip to content

Conversation

@guoqqqi
Copy link
Contributor

@guoqqqi guoqqqi commented Jan 14, 2026

support some nginx_config.http.x config

@guoqqqi guoqqqi requested a review from nic-6443 January 14, 2026 07:33
Comment on lines 200 to 204
keepalive_timeout: {{ default "60s" .Values.nginx.http.keepaliveTimeout }} # timeout during which a keep-alive client connection will stay open on the server side.
client_header_timeout: {{ default "60s" .Values.nginx.http.clientHeaderTimeout }} # timeout for reading client request header, then 408 (Request Time-out) error is returned to the client
client_body_timeout: {{ default "60s" .Values.nginx.http.clientBodyTimeout }} # timeout for reading client request body, then 408 (Request Time-out) error is returned to the client
send_timeout: {{ default "10s" .Values.nginx.http.sendTimeout }} # timeout for transmitting a response to the client.then the connection is closed
client_max_body_size: {{ default 0 .Values.nginx.http.clientMaxBodySize }} # The maximum allowed size of the client request body.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to use default xx, the values in values.yaml have this effect, you see other fields don't write this.

@guoqqqi guoqqqi merged commit 2de1771 into main Jan 14, 2026
2 checks passed
@guoqqqi guoqqqi deleted the feat/support-more-config branch January 14, 2026 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants