988-different-passwords-for-databases #1017

Closed
b25marem wants to merge 8 commits from 988-different-passwords-for-databases into team_1_week_6
Collaborator

What has been done

  • Added separate database usernames and passwords for each stack service

  • Updated compose files for:

    • user
    • parking
    • temperature
    • energy
    • roomBooking
  • Updated healthchecks to use stack-specific database users

  • Added stack-specific environment variables

Testing

  • Tested all stack compose configurations using:

    • docker compose config
  • Tested runtime startup for user stack

  • Verified database initialization worked with new credentials

Result

Each stack now uses separate database credentials instead of shared global database credentials.

## What has been done * Added separate database usernames and passwords for each stack service * Updated compose files for: * user * parking * temperature * energy * roomBooking * Updated healthchecks to use stack-specific database users * Added stack-specific environment variables ## Testing * Tested all stack compose configurations using: * `docker compose config` * Tested runtime startup for user stack * Verified database initialization worked with new credentials ## Result Each stack now uses separate database credentials instead of shared global database credentials.
To check if sub-composing works, how it works and how we can work with it.
Started doing breakdown, it currently doesn't work
Frontend, middle, and temperature now exist is separate compose files, and still being able to communicate with each other. The connectionString is hard-coded to only communicate with the temperature database, which should be changed next. Also all other service/mockups should be added.
All service files added and the start/rebuild script is updated with correct mapping
Energy service implemented with bare minimum changes (will be used as a template for further changes) and general code cleanup
Middleware will now connect to the separate databases with an unique connectionString for each service.
All services are now broken out, some work while others are missing components (old branch). This is at least the general idea. Need to update branch to continue working...
Collaborator

This should not be merged until #985 has been merged.

This should not be merged until #985 has been merged.
Collaborator

@b25marem do you want to make a new branch from the week_7 branch that now has the PR #985 merged? I that case I will close this one.

@b25marem do you want to make a new branch from the week_7 branch that now has the PR #985 merged? I that case I will close this one.
Author
Collaborator

Yes, that sounds good. I’ll create a new branch from week_7 now that #985 has been merged and move these changes there. You can close this PR.

Yes, that sounds good. I’ll create a new branch from `week_7` now that #985 has been merged and move these changes there. You can close this PR.
a24julot closed this pull request 2026-05-21 09:58:49 +00:00

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!1017
No description provided.