Skip to content

Conversation

@ashggan
Copy link
Collaborator

@ashggan ashggan commented Jan 19, 2026

Summary

  • Add complete Docker setup for local development with PostgreSQL, Redis, and Azurite (Azure Storage emulator)
  • Add Dockerfiles for both client and server containers
  • Add database initialization script with full schema
  • Fix POSTGRES_SSL_MODE in server config to read from environment variable, enabling local development without SSL

Services

Service Port Description
PostgreSQL 5432 Database
Redis 6379 Cache
Azurite 10000-10002 Azure Blob Storage emulator
Server 2999 Express API
Client 5173 Vite dev server (optional)

Usage

cd docker
docker-compose up -d

Test plan

  • Run docker-compose up -d and verify all containers start
  • Verify server connects to PostgreSQL and Redis (check logs)
  • Test API endpoints work correctly
  • Verify database schema is created via init script

🤖 Generated with Claude Code

- Add docker-compose.yml with PostgreSQL, Redis, Azurite, and server services
- Add Dockerfiles for client and server containers
- Add database initialization script with schema
- Fix POSTGRES_SSL_MODE to read from environment variable (enables local dev without SSL)
- Include README with setup instructions

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@@ -0,0 +1,20 @@
# Digital Safe Client Dockerfile (Development)

Check notice

Code scanning / Trivy

No HEALTHCHECK defined Low

Artifact: docker/Dockerfile.client
Type: dockerfile
Vulnerability DS026
Severity: LOW
Message: Add HEALTHCHECK instruction in your Dockerfile
Link: DS026
@@ -0,0 +1,33 @@
# Digital Safe Server Dockerfile

Check notice

Code scanning / Trivy

No HEALTHCHECK defined Low

Artifact: docker/Dockerfile.server
Type: dockerfile
Vulnerability DS026
Severity: LOW
Message: Add HEALTHCHECK instruction in your Dockerfile
Link: DS026
Add USER node directive to Dockerfiles to fix Trivy DS002
high-severity vulnerability about running containers as root.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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.

2 participants