-
Notifications
You must be signed in to change notification settings - Fork 43
DOCSP-33943: mention dns seedlist default tls setting #481
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
DOCSP-33943: mention dns seedlist default tls setting #481
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.
1 suggestion + 1 issue, then LGTM
.. note:: | ||
|
||
If you connect by using the DNS seedlist protocol, indicated by the ``+srv`` | ||
connection string modifier, the driver enables TLS/SSL. To disable it, |
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.
connection string modifier, the driver enables TLS/SSL. To disable it, | |
connection-string modifier, the driver enables TLS/SSL. To disable it, |
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.
Thanks for the suggestion! I had intended to keep it consistent with the Server manual mention (linked in the PR text), but I think it could be more clear to describe a little differently.
|
||
If you connect by using the DNS seedlist protocol, indicated by the ``+srv`` | ||
connection string modifier, the driver enables TLS/SSL. To disable it, | ||
set the ``tls`` or ``ssl`` parameter value to ``false`` it in your |
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.
I:
set the ``tls`` or ``ssl`` parameter value to ``false`` it in your | |
set the ``tls`` or ``ssl`` parameter value to ``false`` in your |
* DOCSP-33943: mention dns seedlist default tls setting (cherry picked from commit f7ac2f8)
* DOCSP-33943: mention dns seedlist default tls setting (cherry picked from commit f7ac2f8)
* DOCSP-33943: mention dns seedlist default tls setting (cherry picked from commit f7ac2f8)
* DOCSP-33943: mention dns seedlist default tls setting (cherry picked from commit f7ac2f8)
* DOCSP-33943: mention dns seedlist default tls setting (cherry picked from commit f7ac2f8)
* DOCSP-33943: mention dns seedlist default tls setting (cherry picked from commit f7ac2f8)
* DOCSP-33943: mention dns seedlist default tls setting (cherry picked from commit f7ac2f8)
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-33943
Staging - Enable TLS/SSL
Self-Review Checklist