Problem
Dockerfile and CLI binary disagree on the default listen port:
| Source |
Port |
File |
Dockerfile EXPOSE / ENV TRAPFALL_LISTEN |
3000 |
Dockerfile L54, L59 |
CLI default_value |
9090 |
crates/trapfalld/src/main.rs L37 |
| README docker example |
3000 |
README.md L30 |
| Marketing copy (all slides, docs) |
Mixed 3000/9090 |
codecoradev/marketing |
This causes confusion for users: the published Docker image listens on 3000, but if someone builds from source or runs the binary directly, it listens on 9090.
Suggested Fix
Standardize on one port across all surfaces:
- Dockerfile — update
ENV TRAPFALL_LISTEN + EXPOSE to match CLI default
- README — update all
docker run -p examples
- README — update
localhost:3000 references
- Marketing repo — update carousel slides + per-product-copy (we'll handle this side)
Trade-off: 3000 vs 9090
| Factor |
3000 |
9090 |
| Convention |
Common dev server port (Grafana, etc.) |
Sentry-compatible (Sentry default is 9000, adjacent) |
| Collision risk |
Higher (many apps use 3000) |
Lower |
| Current Docker |
✅ Active |
❌ CLI only |
| Current README |
✅ Active |
❌ Not used |
Recommendation: Pick one and stick with it. If Sentry compatibility matters (users coming from Sentry expect ports near 9000), go 9090. If simplicity matters, go 3000 since Dockerfile + README already use it.
Discovered By
Cora AI code review (cora review) on codecoradev/marketing commit 1ef8af6 — flagged 3 MAJOR issues for port mismatch between carousel slides and copy pack.
Problem
Dockerfile and CLI binary disagree on the default listen port:
EXPOSE/ENV TRAPFALL_LISTENDockerfileL54, L59default_valuecrates/trapfalld/src/main.rsL37README.mdL30codecoradev/marketingThis causes confusion for users: the published Docker image listens on
3000, but if someone builds from source or runs the binary directly, it listens on9090.Suggested Fix
Standardize on one port across all surfaces:
ENV TRAPFALL_LISTEN+EXPOSEto match CLI defaultdocker run -pexampleslocalhost:3000referencesTrade-off: 3000 vs 9090
Recommendation: Pick one and stick with it. If Sentry compatibility matters (users coming from Sentry expect ports near 9000), go 9090. If simplicity matters, go 3000 since Dockerfile + README already use it.
Discovered By
Cora AI code review (
cora review) oncodecoradev/marketingcommit1ef8af6— flagged 3 MAJOR issues for port mismatch between carousel slides and copy pack.