Skip to content

fix: remove duplicate proxy_read_timeout that crashes nginx on start - #34

Closed
ronibhakta1 wants to merge 1 commit into
readium:mainfrom
ronibhakta1:fix/nginx-duplicate-proxy-timeout
Closed

ronibhakta1 wants to merge 1 commit into
readium:mainfrom
ronibhakta1:fix/nginx-duplicate-proxy-timeout

Conversation

@ronibhakta1

Copy link
Copy Markdown
Member

Problem

nginx fails to start: "proxy_read_timeout" directive is duplicate in 20-speech-server.conf:80. /synthesize includes proxy-pass.conf (which sets proxy_read_timeout 120s) and then sets 300s again in the same block. nginx rejects repeated directives in one block. The old container kept running an older config, so this only surfaced when the deploy recreated nginx, and the site went down.

Change

  • Drop proxy_read_timeout from the shared proxy-pass.conf snippet.
  • Set it per location: 120s in location /, 300s in /synthesize. Effective values are unchanged.

Verified

nginx -t in nginx:alpine with the rendered template: fails on main with the error above, passes with this change.

Follow-up worth considering: run this nginx -t in CI so config errors are caught before deploy.

@ronibhakta1

Copy link
Copy Markdown
Member Author

Folded into #33 so the fix and the CI check that catches it land together (and #33 stays green).

@ronibhakta1 ronibhakta1 closed this Oct 8, 2026
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.

1 participant