Skip to content

Serve Finance web with Caddy - #8

Merged
iPior merged 3 commits into
mainfrom
agent/swap-web-runtime-to-caddy
Aug 7, 2026
Merged

Serve Finance web with Caddy#8
iPior merged 3 commits into
mainfrom
agent/swap-web-runtime-to-caddy

Conversation

@iPior

@iPior iPior commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • replace the Finance web container runtime from Nginx to caddy:2-alpine
  • add a minimal static-only packages/web/Caddyfile with SPA fallback
  • remove the obsolete Nginx config and its internal /api reverse proxy

Why

platform-deploy already routes /api/* directly to finance-api:3000 and sends all other Finance traffic to finance-web:80. The app-level Nginx proxy was therefore redundant in production.

This aligns Finance with the current service-auth and template-webapp pattern:

  • platform Caddy owns domains, TLS, ingress, and API routing
  • the web container Caddy only serves the compiled SPA

Scope

This is intentionally a small runtime-only change. No API, database, auth, Compose, or platform routing behavior is changed.

Validation

  • branch is based on current main
  • final diff is limited to three web-runtime files
  • existing Finance Caddy routing remains compatible with this change

@iPior
iPior marked this pull request as ready for review August 7, 2026 22:08
@iPior
iPior merged commit e930b1e into main Aug 7, 2026
2 checks passed
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