fix: use container-internal env vars in mongo healthcheck by HardMax71 · Pull Request #259 · HardMax71/Integr8sCode

HardMax71 · 2026-03-03T19:19:32Z

Summary by cubic

Fix MongoDB healthcheck in docker-compose by using explicit host/port and container env vars for credentials. This avoids the “storedKey mismatch” on mongosh 2.6.0 + MongoDB 8 and prevents password mangling.

Bug Fixes
- Use mongosh with --host/--port and db.adminCommand("ping") against admin instead of a URI with a database path.
- Read creds from $$MONGO_INITDB_ROOT_USERNAME/$$MONGO_INITDB_ROOT_PASSWORD to defer expansion to the container and support special characters.

^{Written for commit d06cdce. Summary will update on new commits.}

Summary by CodeRabbit

Chores
- Enhanced database health monitoring for improved service reliability.

mongosh 2.6.0 + MongoDB 8.0 produces "storedKey mismatch" when the connection URI includes a database path (localhost/integr8scode) while --authenticationDatabase points to a different db (admin). Switching to explicit --host/--port flags with --username/--password avoids this. Also replaces ${MONGO_PASSWORD:-...} host-side interpolation (which mangles passwords containing $, `, " via the container's sh) with $$MONGO_INITDB_ROOT_USERNAME / $$MONGO_INITDB_ROOT_PASSWORD — the $$ escape tells docker-compose to emit a literal $ so the container shell expands the variables from the mongo container's own environment, where they are already set by the environment: block. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

sonarqubecloud · 2026-03-03T19:20:08Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

codecov-commenter · 2026-03-03T19:20:17Z

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Flag	Coverage Δ
backend-e2e	`83.22% <ø> (+0.01%)`	⬆️
backend-unit	`67.91% <ø> (ø)`
frontend-unit	`86.86% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
Backend	`90.07% <ø> (ø)`
Frontend	`86.86% <ø> (ø)`

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

coderabbitai · 2026-03-03T19:21:00Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d5f635c and d06cdce.

📒 Files selected for processing (1)

docker-compose.yaml

📝 Walkthrough

Walkthrough

The MongoDB healthcheck in docker-compose.yaml was updated to use an explicit mongosh invocation with host 127.0.0.1, port 27017, and root credentials derived from environment variables, evaluating db.adminCommand("ping") instead of piping a shell-based ping command.

Changes

Cohort / File(s)	Summary
Configuration Update `docker-compose.yaml`	Updated MongoDB healthcheck from shell-piped ping command to explicit mongosh invocation with programmatic authentication using environment-derived credentials.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A healthier check, from the shell to the code,
MongoDB now proves its status on the road,
With explicit credentials and pingable cheer,
The docker-compose whispers: "All systems are clear!" 🎉

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the primary change: updating the MongoDB healthcheck to use container-internal environment variables instead of inline credentials.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch fix/deploy-creds

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

cubic-dev-ai

No issues found across 1 file

Copilot

Pull request overview

This PR fixes the MongoDB healthcheck command in docker-compose.yaml to use container-internal environment variables ($$MONGO_INITDB_ROOT_USERNAME / $$MONGO_INITDB_ROOT_PASSWORD) instead of host-level shell variables (${MONGO_USER} / ${MONGO_PASSWORD}), ensuring the healthcheck works reliably regardless of the host environment.

Changes:

Replaces shell-interpolated host env vars with Docker-escaped ($$) container-internal env vars
Switches from echo ... | mongosh to mongosh --eval style invocation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings March 3, 2026 19:19

cubic-dev-ai Bot reviewed Mar 3, 2026

View reviewed changes

Copilot AI reviewed Mar 3, 2026

View reviewed changes

HardMax71 merged commit 833bcf9 into main Mar 3, 2026
18 of 20 checks passed

HardMax71 deleted the fix/deploy-creds branch March 3, 2026 19:26

coderabbitai Bot mentioned this pull request Mar 3, 2026

fix: mongo check #260

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: use container-internal env vars in mongo healthcheck#259

fix: use container-internal env vars in mongo healthcheck#259
HardMax71 merged 1 commit intomainfrom
fix/deploy-creds

HardMax71 commented Mar 3, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

sonarqubecloud Bot commented Mar 3, 2026

Uh oh!

codecov-commenter commented Mar 3, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Mar 3, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Poem

Review ran into problems

Uh oh!

cubic-dev-ai Bot left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

HardMax71 commented Mar 3, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by cubic

Summary by CodeRabbit

Uh oh!

sonarqubecloud Bot commented Mar 3, 2026

Quality Gate passed

Uh oh!

codecov-commenter commented Mar 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

coderabbitai Bot commented Mar 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Poem

Review ran into problems

Uh oh!

cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

HardMax71 commented Mar 3, 2026 •

edited by coderabbitai Bot

Loading

codecov-commenter commented Mar 3, 2026 •

edited

Loading

coderabbitai Bot commented Mar 3, 2026 •

edited

Loading