Skip to content

fix: avoid nginx redirect cycles#361

Merged
vitormattos merged 1 commit intomainfrom
fix/nginx-redirection-cycle
Apr 7, 2026
Merged

fix: avoid nginx redirect cycles#361
vitormattos merged 1 commit intomainfrom
fix/nginx-redirection-cycle

Conversation

@vitormattos
Copy link
Copy Markdown
Member

Summary

  • stop recursive internal redirects in the Nginx static-site config
  • return 404 for missing assets and sensitive paths like /.env instead of 500
  • serve generated HTML files explicitly so pretty URLs such as /account and /pricing keep working

Validation

  • docker compose exec web nginx -t
  • docker compose exec web nginx -s reload
  • curl probes against /account, /pricing, /.env, /robots.txt and /favicon.ico
  • verified web logs no longer show "rewrite or internal redirection cycle" after the reload

Notes

  • local build_local in this worktree does not currently include a root index.html, so / returns 404 locally after the loop fix; that is separate from the Nginx redirect-cycle bug being fixed here

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://libresign.github.io/site-preview/pr-preview/pr-361/

Built to branch main at 2026-04-07 02:22 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@vitormattos vitormattos merged commit 3a74829 into main Apr 7, 2026
3 checks passed
@vitormattos vitormattos deleted the fix/nginx-redirection-cycle branch April 7, 2026 02:25
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