Skip to content

Fix Docker deployment issues - Redis connectivity & environment variables #186

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
Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Fix Docker deployment issues - Redis connectivity & environment variables #186

wants to merge 14 commits into from

Conversation

Mufeth7
Copy link

@Mufeth7 Mufeth7 commented Apr 15, 2025 •
edited
Loading

This PR resolves critical deployment issues in the Docker configuration that prevent note creation.

Problem

  1. Redis Connection Failure: Missing REDIS_URL environment variable caused backend to fail connecting to Redis
  2. Environment Variable Syntax: SIZE_LIMIT: 4 MiB (with space) would be parsed incorrectly by Rust's byte_unit
  3. Port Conflicts: Frontend proxy (port 3000) tried reaching backend via port 8000, but Docker mapped port 80→8000 (Matches frontend proxy expectations from vite.config.js)

Impact

  • ✅ Ensures Redis connectivity for note storage
  • ✅ Prevents environment variable parsing errors
  • ✅ Resolves frontend/backend port conflicts
  • ✅ Explicit network definition improves deployment reliability

@Mufeth7 Mufeth7 changed the title Modified in docker-compose.yaml file, added network and changed in p… Fix Docker deployment issues - Redis connectivity & environment variables Apr 15, 2025
cupcakearmy and others added 13 commits May 8, 2025 17:20
@cupcakearmy @Mufeth7
remove proxy
7172b04
@cupcakearmy @Mufeth7
update to svelte 5
2c0caf3
@cupcakearmy @Mufeth7
version bump
9ad42c7
@cupcakearmy @Mufeth7
add csp draft
61a938a
@dependabot @Mufeth7
Bump vite in the npm_and_yarn group across 1 directory
fc1b85f 
Bumps the npm_and_yarn group with 1 update in the / directory: [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite).


Updates `vite` from 6.0.7 to 6.0.9
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v6.0.9/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-type: direct:development
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <[email protected]>
@cupcakearmy @Mufeth7
update docs about ram only redis
15c5a09
@cupcakearmy @Mufeth7
version bump
8939e98
@cupcakearmy @Mufeth7
update some versions
6a1baf2
@cupcakearmy @Mufeth7
fix: password eye toggle not working
67df272
@larsgerber @Mufeth7
docs(compose): prevent anonymous volume creation
e4219e8
@dependabot @Mufeth7
Bump ring in /packages/backend in the cargo group across 1 directory
74d5a3f 
Bumps the cargo group with 1 update in the /packages/backend directory: [ring](https://github.com/briansmith/ring).


Updates `ring` from 0.16.20 to 0.17.12
- [Changelog](https://github.com/briansmith/ring/blob/main/RELEASES.md)
- [Commits](https://github.com/briansmith/ring/commits)

---
updated-dependencies:
- dependency-name: ring
  dependency-type: direct:production
  dependency-group: cargo
...

Signed-off-by: dependabot[bot] <[email protected]>
@Mufeth7
Modified in docker-compose.yaml file, added network and changed in p…
c330d91 
…ort section
@Mufeth7
Add workflow to deploy Docker image to AWS ECR public repository
a4a4062
@Mufeth7 Mufeth7 force-pushed the main branch 2 times, most recently from df1facb to 686a5e8 Compare May 12, 2025 11:55
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.
3 participants
@Mufeth7 @cupcakearmy @larsgerber