Item8978: Config.spec misses {Ldap}{MailAttribute}
Priority: Normal
Current State: Closed
Released In: n/a
Target Release: n/a
LdapContrib's
Config.spec
misses
{Ldap}{MailAttribute}
, making it impossible to be changed from
configure
. The default value
mail
is good, but not always correct.
--
GilmarSantosJr - 05 May 2010
Oh sure. Have a try to add it ton Config.spec.
--
MichaelDaum - 08 Jun 2010
I made this change:
$ git diff
diff --git a/LdapContrib/lib/Foswiki/Contrib/LdapContrib/Config.spec b/LdapContrib/lib/Foswiki/Contrib/LdapContrib/Config.spec
index 19bce32..ed6e26a 100644
--- a/LdapContrib/lib/Foswiki/Contrib/LdapContrib/Config.spec
+++ b/LdapContrib/lib/Foswiki/Contrib/LdapContrib/Config.spec
@@ -107,6 +107,11 @@ $Foswiki::cfg{Ldap}{UserScope} = 'sub';
$Foswiki::cfg{Ldap}{LoginAttribute} = 'uid';
# **STRING**
+# The user mail attribute. This is the attribute name used to fetch
+# users e-mail.
+$Foswiki::cfg{Ldap}{MailAttribute} = 'mail';
+
+# **STRING**
# The user's wiki name attribute. This is the attribute to generate
# the WikiName from.
$Foswiki::cfg{Ldap}{WikiNameAttribute} = 'cn';
And it worked. May I commit?
--
GilmarSantosJr - 08 Jun 2010
Sure.
--
MichaelDaum - 08 Jun 2010
Done. I marked as
Waiting for Release
to remind you this fix is not released yet
--
GilmarSantosJr - 09 Jun 2010