Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(cli): Update MongoDB connection string for node 17+ #2875

Merged
merged 1 commit into from
Nov 18, 2022
Merged

Conversation

marshallswain
Copy link
Member

On Node version 17+, localhost gets converted to the IPV6 version :::1. The MongoDB database doesn’t receive the connection.

This PR updates the default MongoDB connection string to use the IPV4 loopback address 127.0.0.1, which restores the ability to connect to the MongoDB server.

One line in the docs was also updated.

On Node version 17+, `localhost` gets converted to the IPV6 version `:::1`.  The MongoDB database doesn’t receive the connection.

This PR updates the default MongoDB connection string to use the IPV4 loopback address `127.0.0.1`, which restores the ability to connect to the MongoDB server.

One place in the docs was also updated.
@netlify
Copy link

netlify bot commented Nov 15, 2022

Deploy Preview for feathers-dove ready!

Name Link
🔨 Latest commit 7e94c7a
🔍 Latest deploy log https://app.netlify.com/sites/feathers-dove/deploys/6373cb89e1d3240008906db5
😎 Deploy Preview https://deploy-preview-2875--feathers-dove.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@marshallswain marshallswain requested a review from daffl November 15, 2022 21:37
@daffl daffl changed the title fix(cli): update MongoDB connection string for node 17+ fix(cli): Update MongoDB connection string for node 17+ Nov 18, 2022
@daffl daffl merged commit 7fa2012 into dove Nov 18, 2022
@daffl daffl deleted the dove-cli-mongodb branch November 18, 2022 21:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants