Skip to content

Replace Caddy with Nginx#2225

Draft
ObadaS wants to merge 9 commits intodevelopfrom
nginx
Draft

Replace Caddy with Nginx#2225
ObadaS wants to merge 9 commits intodevelopfrom
nginx

Conversation

@ObadaS
Copy link
Copy Markdown
Collaborator

@ObadaS ObadaS commented Mar 2, 2026

A brief description of the purpose of the changes contained in this PR.

  • Replace Caddy with Nginx, allowing us to route every connection through it (like rabbit, minio, postgres) instead of only HTTP(S) connections
  • Add anti-scrapping measures
  • Add robots.txt file
  • Closed all the external ports of the containers except Nginx. Everything must go through Nginx now.
  • Made all containers (except Nginx, Django, Compute Worker and Builder) unable to access the outside world.
  • Add rate limit

Manual Intervention

There are some new environment variable that will be needed in the .env file

HTTPS=False # Or True
RATE_LIMIT=5

If your DOMAIN_NAME contains : you will need to remove it.
For example, localhost:80 becomes localhost

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • CircleCi tests are passing
  • Ready to merge

@ObadaS ObadaS force-pushed the nginx branch 2 times, most recently from 4f4265c to 01f2f1d Compare March 18, 2026 15:39
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