2447: Documenation for ldap user mask w/ AD

solarmist
Jan. 22, 2012
http://reviews.reviewboard.org/


What version are you running?
1.6.3

What's the URL of the page this enhancement relates to, if any?
reviews.reviewboard.org/admin/settings/authentication/ under LDAP
and 
http://www.reviewboard.org/docs/manual/1.6/admin/configuration/authentication-settings/

Describe the enhancement and the motivation for it.
Change the example for LDAP user mask
For example: (uid=%s)
to
For example: (uid=%s) or (sAMAccountName=%s) [for active directory LDAP]

This is a common setup when using LDAP and developer inexperience with LDAP may have trouble determining this on their own.

Please provide any additional information below.
david
#1 david
  • +Documenation for ldap user mask w/ AD
david
#2 david
Fixed in release-1.6.x (1ba234d). Thanks!
  • +Fixed