Skip to content

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
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 28, 2025
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