Skip to content

Dockerize full stack app #28

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
mano8 merged 32 commits into from
Apr 10, 2025
Merged

Dockerize full stack app #28

mano8 merged 32 commits into from
Apr 10, 2025

Conversation

@mano8
Copy link
Member

@mano8 mano8 commented Apr 10, 2025

Changes:

  • Add docker file's and docker compose examples to dockerize fullstack app
  • Improve and secure fastapi Settings module

mano8 added 28 commits April 8, 2025 13:30
@mano8
Normalize line endings to LF
9ab6d3b
@mano8
feat: add .gitattributes to enforce LF line endings for shell scripts
13b5d16
@mano8
feat: rename pre-start script for service initialization with retry l…
6851286 
…ogic
@mano8
feat: improve error handling in pre_start script with exit codes
65d690b
@mano8
feat: add paths.py utility for dotenv file discovery
9167e44
@mano8
fix: reorder docstring and import statement in database.py
722a009
@mano8
feat: add ValidationConstants class with regex patterns for configura…
0a692df 
…tion validation
@mano8
feat: enhance configuration settings with CORS validation and secure …
1d4ad67 
…defaults
@mano8
fix: ensure newline at end of file in config.py
496a171
@mano8
feat: add wait_for_db.sh script for MySQL database readiness check
3e3475a
@mano8
feat: add docker_start.sh script for initializing database and starti…
5923bd2 
…ng FastAPI server in docker container
@mano8
feat: add .dockerignore to exclude Dockerfile and alembic versions fr…
0ca32ec 
…om Docker context
@mano8
feat: add requirements-docker-dev.txt for Docker development dependen…
16220e4 
…cies
@mano8
feat: add Dockerfile for setting up FastAPI environment with MySQL su…
320ea34 
…pport
@mano8
fix: improve logging message in wait_for_db.sh to include database ho…
d196e5f 
…st and port
@mano8
refactor: update CORS handling in settings and improve BACKEND_CORS_O…
1043a47 
…RIGINS validation
@mano8
refactor: streamline CORS middleware configuration by using settings …
3fcdadd 
…for allowed origins
@mano8
refactor: remove pip upgrade command and fix script naming in Dockerfile
27fe10b
@mano8
refactor: update Vite server configuration for Docker compatibility
3fbc9d3
@mano8
feat: add .dockerignore to exclude node_modules and Dockerfile from b…
edb7c76 
…uild context
@mano8
feat: add Dockerfile for Vite development environment setup
30bbad9
@mano8
feat: add Docker Compose configuration and environment setup for deve…
08853a4 
…lopment
@mano8
feat: add example environment configuration for local development
395eef2
@mano8
feat: update .gitignore to include records and alembic version files
df7ccf9
@mano8
refactor: improve CORS origin parsing and validation error messages
aeb812c
@mano8
fix: retrieve superuser password securely using get_secret_value()
45669c3
@mano8
fix: retrieve JWT token payload securely using get_secret_value()
15ad93b
@mano8
fix: retrieve secret key securely using get_secret_value() in JWT tok…
0fc191d 
…en creation and decoding
@codecov
Copy link

codecov bot commented Apr 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

mano8
mano8 commented Apr 10, 2025
View reviewed changes
Copy link
Member Author

@mano8 mano8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@mano8 mano8 merged commit 3ab75d3 into main Apr 10, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mano8