fix: add demo/ folder to Dockerfile so /demo route works in container #34

Merged
jason merged 1 commits from feature/demo-dockerfile into master 2026-03-08 00:12:11 -06:00
Owner

Problem

The demo/ folder and /demo static route in server.js were both present on master, but /demo/index.html was still serving the React app inside Docker.

Root cause: The Dockerfile uses explicit COPY statements for each directory in the production stage. demo/ was never listed, so it was silently excluded from the container image — Express had nothing to serve and fell through to the SPA catch-all.

Change

Added one line to the production stage of the Dockerfile:

COPY demo/        ./demo/

Placed after the existing pdf/ copy, before RUN mkdir -p /data.

After merging

Rebuild and restart the container — /demo/ and /demo/index.html will serve the standalone stakeholder demo page as intended.

## Problem The `demo/` folder and `/demo` static route in `server.js` were both present on master, but `/demo/index.html` was still serving the React app inside Docker. **Root cause:** The Dockerfile uses explicit `COPY` statements for each directory in the production stage. `demo/` was never listed, so it was silently excluded from the container image — Express had nothing to serve and fell through to the SPA catch-all. ## Change Added one line to the production stage of the Dockerfile: ```dockerfile COPY demo/ ./demo/ ``` Placed after the existing `pdf/` copy, before `RUN mkdir -p /data`. ## After merging Rebuild and restart the container — `/demo/` and `/demo/index.html` will serve the standalone stakeholder demo page as intended.
jason added 1 commit 2026-03-08 00:11:45 -06:00
jason merged commit 87649b59d0 into master 2026-03-08 00:12:11 -06:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: jason/cpas#34