Performance updates to nginx configuration.

[Rider-Linden]

No description provided.

[Copilot]

Pull Request Overview

This PR enhances NGINX performance by introducing configurable file descriptor limits and separating WSGI timeouts from keepalive settings. It also sets sensible defaults in the entrypoint script and updates related documentation.

• Add worker_rlimit_nofile controlled by WORKER_FILE_LIMIT.
• Replace KEEPALIVE_TIMEOUT with WSGI_TIMEOUT for uWSGI timeouts.
• Provide defaults for new env vars in 00-render-templates.sh and update docs.

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/etc/nginx/nginx.conf.template	Added worker_rlimit_nofile {{ .Env.WORKER_FILE_LIMIT }}
src/etc/nginx/includes/uwsgi.conf.template	Swapped uwsgi_read/send_timeout to use WSGI_TIMEOUT
src/docker-entrypoint.d/00-render-templates.sh	Defaulted WORKER_FILE_LIMIT and WSGI_TIMEOUT env variables
README.md	Documented WORKER_FILE_LIMIT; missing entry for new WSGI_TIMEOUT
Dockerfile	Removed unnecessary blank line for consistent formatting

Comments suppressed due to low confidence (5)

src/etc/nginx/includes/uwsgi.conf.template:5

• The new WSGI_TIMEOUT variable is not documented in README.md; please add an entry explaining its purpose and default behavior.

uwsgi_read_timeout {{ .Env.WSGI_TIMEOUT }};

README.md:31

• [nitpick] Clarify that the default 2*WORKER_CONNECTIONS value is computed at runtime by the entrypoint script rather than set literally in the environment.

| `WORKER_FILE_LIMIT` | Set the number of available file descriptors | No | 2*WORKER_CONNECTIONS | 4096 |

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

• Consider adding a test or CI check to verify that WORKER_FILE_LIMIT is rendered correctly into the final nginx.conf, ensuring the intended file descriptor limit is applied.

worker_rlimit_nofile {{ .Env.WORKER_FILE_LIMIT }};

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

• If KEEPALIVE_TIMEOUT is undefined, WSGI_TIMEOUT may be empty; consider providing a fallback or validating that KEEPALIVE_TIMEOUT is set before using it.

export WSGI_TIMEOUT=${WSGI_TIMEOUT:-${KEEPALIVE_TIMEOUT}}

[rohvani]

@Rider-Linden Are you intending to merge this?

[Rider-Linden]

@Rider-Linden Are you intending to merge this?

Oh. This got lost in the shuffle. I'll merge it later today.