-
Notifications
You must be signed in to change notification settings - Fork 13k
chore: upgrade mongodb to 7.0.0 to fix AWS IAM credentials #37490
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
- Updated mongodb dependency from 6.10.0 to 7.0.0 in root package.json resolutions - Updated mongodb dependency from 6.16.0 to 7.0.0 in apps/meteor/package.json - MongoDB 7.x properly supports AWS IAM authentication via default credential provider chain - Fixes issue with AWS_CONTAINER_CREDENTIALS_FULL_URI in containerized environments Fixes RocketChat#37456
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
🦋 Changeset detectedLatest commit: 7f7d9b9 The changes in this PR will be included in the next version bump. This PR includes changesets to release 41 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
|
WalkthroughMongoDB dependency bumped to 7.0.0 in the repository root resolutions and in Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant App as Rocket.Chat
participant Driver as mongodb@7.0.0
participant AWS as AWS Credential Provider (SDK)
participant MongoDB as MongoDB Atlas
Note over Driver,AWS: New flow (post-upgrade)
App->>Driver: connect(MONGO_URL, options)
Driver->>AWS: use default credential chain (env, container, IMDS, credproxy)
AWS-->>Driver: credentials
Driver->>MongoDB: authenticate with credentials
MongoDB-->>Driver: connection established
Driver-->>App: connected
sequenceDiagram
autonumber
participant App as Rocket.Chat
participant DriverOld as mongodb@<=6.x
participant LocalCred as aws_temporary_credentials.ts (custom)
participant IMDS as IMDS / EC2 endpoint
participant MongoDB as MongoDB Atlas
Note over DriverOld,LocalCred: Prior flow (pre-upgrade)
App->>DriverOld: connect(MONGO_URL, options)
DriverOld->>LocalCred: fallback custom credential lookup
LocalCred->>IMDS: direct IMDS requests (169.254.169.254)
IMDS-->>LocalCred: credentials or timeouts
LocalCred-->>DriverOld: credentials (limited sources)
DriverOld->>MongoDB: authenticate with credentials (or fail/timeout)
MongoDB-->>DriverOld: connection result
DriverOld-->>App: connected / error
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (5 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
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. Comment |
Upgrade mongodb to version 7.0.0 to resolve AWS IAM credentials issue.
Fixes #37456
Proposed changes (including videos or screenshots)
Issue(s)
Steps to test or reproduce
Further comments
Summary by CodeRabbit
Chores
Documentation