Skip to content

Fix for Issue #13652 caused by MongoDB Node.js Driver#13710

Merged
italojs merged 1 commit intometeor:release-3.3from
welkinwong:fix-13652
Apr 25, 2025
Merged

Fix for Issue #13652 caused by MongoDB Node.js Driver#13710
italojs merged 1 commit intometeor:release-3.3from
welkinwong:fix-13652

Conversation

@welkinwong
Copy link
Copy Markdown
Contributor

This PR primarily addresses the bug reported in #13652. The issue was caused by the MongoDB Node.js Driver and has been fixed in version v6.16.0.

In Commit 3dac12a, I noticed a temporary version downgrade from 6.10.0 to 6.9.0 due to legacy database connection issues. However, after reviewing the 6.10.0 changelog, the only documented breaking change was the dropped compatibility with MongoDB 3.6. No other critical issues were mentioned in the release notes.

If the issue was solely related to pre-MongoDB 4.x compatibility, I believe we should proceed with the upgrade. According to MongoDB's official support policy (MongoDB Software Lifecycle Schedules), versions below 4.x are already end-of-life (EOL) and no longer supported. Maintaining compatibility with deprecated database versions would introduce unnecessary technical debt and potential security risks.

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 25, 2025

Deploy Preview for v3-meteor-api-docs canceled.

Name Link
🔨 Latest commit 583310e
🔍 Latest deploy log https://app.netlify.com/sites/v3-meteor-api-docs/deploys/680bb09e5a03890008e2f06a

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 25, 2025

Deploy Preview for v3-migration-docs canceled.

Name Link
🔨 Latest commit 583310e
🔍 Latest deploy log https://app.netlify.com/sites/v3-migration-docs/deploys/680bb09e6fd0080007bb01e8

@italojs italojs changed the base branch from devel to release-3.3 April 25, 2025 18:54
@italojs italojs merged commit 43af635 into meteor:release-3.3 Apr 25, 2025
10 checks passed
@nachocodoner
Copy link
Copy Markdown
Member

nachocodoner commented Apr 26, 2025

We had to downgrade the Mongo driver to a specific version because it caused breaking changes in a few apps, including Galaxy, where data was not published properly in an edge case. We should be cautious about including this in 3.3, research and test it beforehand.

It would be a pity if existing apps were blocked by a Mongo issue and couldn't benefit from the performance improvements. It might be better to delay it to a future version, with more focus in Mongo.

@welkinwong welkinwong deleted the fix-13652 branch April 27, 2025 14:20
nachocodoner added a commit that referenced this pull request May 7, 2025
nachocodoner added a commit that referenced this pull request May 7, 2025
@nachocodoner
Copy link
Copy Markdown
Member

nachocodoner commented May 7, 2025

I’ve reverted the MongoDB upgrade and will postpone it to the next beta, either 3.3.1-beta or 3.4-beta. Upgrading to later versions from mongodb 6.0.9 caused issues in some community apps without clear reproduction steps. In the next beta cycle, we’ll give Meteor apps an isolated window to test the Mongo upgrade and help us reproduce and fix any issues. That way, Meteor 3.3 can focus on build enhancements and any major fixes on this area if araise.

@welkinwong: Feel free to open a new PR with this. After the official Meteor 3.3 we can create a beta for this upgrade. We'll do it otherwise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants