Skip to content
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

remove deprecated useNewUrlParser and useUnifiedTopology #100

Merged
merged 1 commit into from
Nov 22, 2024
Merged

remove deprecated useNewUrlParser and useUnifiedTopology #100

merged 1 commit into from
Nov 22, 2024

Conversation

DLeibner
Copy link
Contributor

@DLeibner DLeibner commented Nov 20, 2024 •
edited
Loading

We're currently on Mongo version 6, where there is a deprecation warning for useNewUrlParser and useUnifiedTopology options, introduced with PR.

These options were removed in 4.0.0 but continued to be parsed and silently left unused. We have now added a deprecation warning through Node.js' warning system to prepare for finally removing these options in the next major release.

@DLeibner DLeibner merged commit a38c33e into main Nov 22, 2024
1 check passed
@DLeibner DLeibner deleted the chore/remove_deprecated_options branch November 22, 2024 08:57
inca
inca approved these changes Nov 22, 2024
View reviewed changes
Copy link
Contributor

@inca inca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be worth bumping mongodb library as well

@DLeibner
Copy link
Contributor Author

Bumped mongodb to latest version (6.10.0) directly on main.

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

@inca inca inca approved these changes

@danielolaviobr danielolaviobr danielolaviobr approved these changes

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
@DLeibner @inca @danielolaviobr