Skip to content

DOCS-9489: Update LDAP tutorial admonition #2812

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

Closed
wants to merge 1 commit into from

Conversation

ravindk89
Copy link
Contributor

@ravindk89 ravindk89 commented Dec 30, 2016

/tutorial/configure-ldap-sasl-activedirectory and /tutorial/configure-ldap-sasl-openldap require updated admonition to point to new 3.4 LDAP support for Windows LDAP authentication and Authorization


This change is Reviewable

/tutorial/configure-ldap-sasl-activedirectory
requires updated admonition to point to new 3.4
LDAP support for Windows LDAP authentication and Authorization
@i80and i80and self-requested a review April 25, 2017 14:45
Copy link
Contributor

@i80and i80and left a comment

Choose a reason for hiding this comment

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

Some optional stuff. Seems sensible to me!

:abbr:`Windows (Microsoft Windows)` does not support the ``saslauthd``
daemon. MongoDB servers running version 3.2 or prior cannot connect to an
LDAP server for authentication. You cannot apply this tutorial to a MongoDB
server running on :abbr:`Windows (Microsoft Windows)`.
Copy link
Contributor

Choose a reason for hiding this comment

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

The sentence order here is a little odd. "Windows does not support saslauthd. You need MongoDB 3.4+. You cannot use this tutorial with Windows".

Can you consolidate the last sentence with the first?

LDAP server for authentication. You cannot apply this tutorial to a MongoDB
server running on :abbr:`Windows (Microsoft Windows)`.

MongoDB 3.4 or later supports binding to an LDAP server via system
Copy link
Contributor

Choose a reason for hiding this comment

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

I think that "MongoDB 3.4 and later" is preferable

@ravindk89 ravindk89 closed this Jun 23, 2017
@ravindk89 ravindk89 deleted the DOCS-9489 branch June 23, 2017 18:00
mongo-cr-bot pushed a commit that referenced this pull request Mar 28, 2023
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.

2 participants