On Thu, Feb 09, 2006 at 01:18:20PM +0100, Fernando Gozalo wrote: > I don't know if this is of interest, but... > > In /etc/authlib/authldaprc you can read > > ===================== > ##NAME: LDAP_MAILDIR:0 > # > # The MAILDIR attribute is OPTIONAL, and specifies the location of the > # mail directory. If not specified, ./Maildir will be used > > # LDAP_MAILDIR mailbox > ===================== > > I think this means that if not LDAP_MAILDIR defined then > "homeDirectory + /Maildir" will be used. > > The attached patch makes this sentence true.
I can't see why this patch is neeed. If the authentication module returns null for Maildir, then the client program will use the value in MAILDIRPATH (default ./Maildir), which is I think what that comment is trying to say. There is no need to concatenate homeDir + Maildir. The authdaemon client code always does a chdir(homeDir), before the application does chdir(Maildir), so a relative path is fine. See courier-authlib/authdaemon.c, function auth_callback_default(). Admittedly, the MAILDIRPATH logic currently sites in the client programs themselves (pop3d, imapd, sqwebmail) rather than in authlib, where it would arguably would make more sense. It would at least mean that if you want a non-default MAILDIRPATH you'd only have to change it in one place, rather than in three or more. Regards, Brian. ------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Do you grep through log files for problems? Stop! Download the new AJAX search engine that makes searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642 _______________________________________________ Courier-imap mailing list [email protected] Unsubscribe: https://lists.sourceforge.net/lists/listinfo/courier-imap
