-
Notifications
You must be signed in to change notification settings - Fork 0
Production release – merge develop branch #1
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.
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
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Replaced res.status() with HTTPResponse.error for consistent API error format - Added specific error handling for expired and invalid JWTs
- Set target to ES2020 for improved language feature support - Added rootDir and outDir for cleaner build output structure - Removed commented defaults for a cleaner config file
- Renamed service from 'backend' to 'expressjs' for clarity - Removed redundant container_name entries - Reordered services for consistency
…sponses Replaced generic success and error methods with specific HTTP methods. - Improves clarity, reusability, and reduces repetition when returning responses
…sponses - Replaced generic success and error methods with specific HTTP methods. - Improves clarity, reusability, and reduces repetition when returning responses
0 Replaced raw res.status(400) with HTTPResponse.badRequest for consistency - Improved validation error messages for firstName and lastName - Combined all validation errors into a single readable string
…nd update dependencies
…est and Supertest
- Renamed package to brainbytes-api - Added Jest, Supertest, and ts-jest for testing - Added clean, build, and test scripts - Updated lint commands for better glob handling
- Renamed package to brainbytes-api - Added Jest, Supertest, and ts-jest for testing - Added clean, build, and test scripts - Updated lint commands for better glob handling
- Added pre-save hook in user model to handle hashing - Removed manual hashing from registerUser controller - Simplified controller logic and improved maintainability
- Removed cors (no longer needed without frontend/browser requests) - Moved @types/morgan to devDependencies
- Ignore build outputs, dependencies, env files, and coverage - Added Docker-related and local dev/testing files to ignore lists
- Set JWT_SECRET and MONGO_URI_DOCKER as environment secrets - Added 'environment: GitHub actions' to job
…pm cache, node versioning
…setup-node to fix pnpm not found error
- Upgrade actions/checkout to v4 - Upgrade pnpm/action-setup to v4 - Add Docker build job using buildx and GHCR - Use docker/login-action and build-push-action for publishing - Ignore changes to Markdown and docs in CI triggers
- Specify Dockerfile.prod as build target
- Use ghcr.io/<owner>/<image-name>:tag format
- Replace GITHUB_TOKEN with GHCR_PAT to bypass permission issues - Fix "denied: installation not allowed to Create organization package" error
…ondition - Use actions/setup-node to explicitly install node.js with pnpm cache support - Use pnpm/action-setup only for installing pnpm CLI - Add conditional check to run docker image build only on push to master (post-merge).
- Replaced pnpm/action-setup with npm global install
…tions - Use pnpm/action-setup before setup-node to ensure pnpm is available for caching
…gration - Defined 'brainbytes_net' as a project-scope network for internal service communication - Connected mongodb, expressjs, and watchtower to 'brainbytes_net' - Added 'traefik' external network for reverse proxy routing to expressjs - Removed unnecessary port binding from expressjs (relying on Traefik routing) - Improved network isolation and readiness for production deployment
- Added Traefik service with dynamic config from Docker - Enabled HTTP/HTTPS entrypoints and Let's Encrypt TLS via ACME - Configured Traefik dashboard and secure service exposure - Mounted Docker socket and cert storage for automation - Declared external 'traefik' network for container routing
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.