adding keepalives to upstream connection

[Rider-Linden]

No description provided.

[Copilot]

Pull Request Overview

This PR adds a file-descriptor limit to the Nginx worker process and documents it, defaulting the new limit to twice the worker connections setting.

• Add worker_rlimit_nofile directive to nginx.conf.template.
• Compute and export WORKER_FILE_LIMIT in the Docker entrypoint script.
• Document the new WORKER_FILE_LIMIT variable in README.md.

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

File	Description
src/etc/nginx/nginx.conf.template	Introduce worker_rlimit_nofile {{ .Env.WORKER_FILE_LIMIT }}
src/docker-entrypoint.d/00-render-templates.sh	Export and default WORKER_FILE_LIMIT=$(( WORKER_CONNECTIONS * 2 ))
README.md	Add WORKER_FILE_LIMIT entry and default in environment variable
Dockerfile	Adjusted whitespace around ENV block

Comments suppressed due to low confidence (4)

src/docker-entrypoint.d/00-render-templates.sh:19

• Add a test or integration check to verify that WORKER_FILE_LIMIT is correctly calculated and applied when rendering the Nginx config.

export WORKER_FILE_LIMIT=${WORKER_FILE_LIMIT:=$(( WORKER_CONNECTIONS * 2 ))}

src/docker-entrypoint.d/00-render-templates.sh:19

• [nitpick] It may be safer to validate that WORKER_CONNECTIONS is set and is a positive integer before performing arithmetic expansion, to avoid unexpected defaults.

export WORKER_FILE_LIMIT=${WORKER_FILE_LIMIT:=$(( WORKER_CONNECTIONS * 2 ))}

src/etc/nginx/nginx.conf.template:10

• [nitpick] Align the indentation of worker_rlimit_nofile with other top-level directives for consistent readability.

worker_rlimit_nofile {{ .Env.WORKER_FILE_LIMIT }};