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.

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-14874 TLS/SSL Page #192

Merged
merged 7 commits into from
Feb 21, 2024

Conversation

jordan-smith721
Copy link
Contributor

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCS-14874
Staging - https://preview-mongodbjordansmith721.gatsbyjs.io/spark-connector/DOCSP-14874-ssl/tls/

Self-Review Checklist

  • Is this free of any warnings or errors in the RST?
  • Did you run a spell-check?
  • Did you run a grammar-check?
  • Are all the links working?
  • Are the facets and meta keywords accurate?

Copy link
Contributor

@rachel-mack rachel-mack left a comment

Choose a reason for hiding this comment

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

LGTM, one suggestion

--------------

You can enable TLS/SSL for the connection to your MongoDB instance through the
``tls`` parameter in your connection URI.
Copy link
Contributor

Choose a reason for hiding this comment

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

You could link to a description or instructions for connection URI here.

I'm not sure if there's anything more specific than this?
https://www.mongodb.com/docs/upcoming/reference/connection-string/#find-your-connection-string

@jordan-smith721 jordan-smith721 requested review from rozza and removed request for rozza January 29, 2024 18:01
Copy link
Member

@rozza rozza left a comment

Choose a reason for hiding this comment

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

LGTM!

@jordan-smith721 jordan-smith721 merged commit bb837fb into mongodb:master Feb 21, 2024
jordan-smith721 added a commit that referenced this pull request Feb 21, 2024
(cherry picked from commit bb837fb)
jordan-smith721 added a commit that referenced this pull request Feb 21, 2024
(cherry picked from commit bb837fb)
jordan-smith721 added a commit that referenced this pull request Feb 21, 2024
(cherry picked from commit bb837fb)
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