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
feat(NODE-3633): add Socks5 support #3041
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got some initial feedback for ya, its looking good, mostly nits. Also, feel free to rebase to pull in the test fix on main so we can get the ✅ I don't mind the files viewed getting messed up for that.
- Spec PR: mongodb/specifications#1103 - Node.js driver PR that makes use of this script: mongodb/node-mongodb-native#3041
- Spec PR: mongodb/specifications#1103 - Node.js driver PR that makes use of this script: mongodb/node-mongodb-native#3041
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thanks for all the effort on this!!
Description
What is changing?
Is there new documentation needed for these changes?
Yes. This adds connection string parameters and MongoClient options.
What is the motivation for this change?
See NODE-3633 and adjacent tickets
Double check the following
npm run check:lint
script<type>(NODE-xxxx)<!>: <description>