----- Original Message ----- From: "Kurt Bigler" <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Saturday, February 22, 2003 10:53 PM Subject: Re: [sqwebmail] NEW: domainmap Patch
> on 2/22/03 11:11 AM, Jesse Guardiani <[EMAIL PROTECTED]> wrote: > > > Howdy list, > > > > What does this patch do? > > ------------------------ > > This patch adds functionality to sqwebmail. It allows the sys admin to add a > > file to the configuration directory > > (/usr/local/share/sqwebmail on my FreeBSD 4.7 system) named 'domainmap'. > > > > Once this file has been created, it can be populated with IP:DOMAIN pairs, or > > DOMAINALIAS:DOMAIN pairs. > > [snip] > > > I tested this patch on my server which uses name-based virtual hosting and > it works like a charm. In other words I tested the use of domainmap > containing DOMAINALIAS:DOMAIN pairs. > > I tested it without a logindomainlist in which case the hidden field kicked > in and the user id could be entered alone for a successful login. I also > tested it with a logindomain list in which case the domain list popup > correctly defaulted correctly. > > Thanks a bunch Jesse. Works like a charm! You're welcome! I'm as relieved as you. My users will be very very happy. > > I still wish the logindomainlist and domainmap could be integrated into one > file. Compared to everything else, this is a minor issue, but any chance > this possibility is still open to discussion? Sure, it's still open to discussion. I'm not the type of person that entertains delusions of perfection. However, I really don't see the point. It would unnecessarily complicate things, IMHO. When programming, I try to group things logically, and the 'logindomainlist' file does something ENTIRELY different, and serves an entirely different purpose, than the 'domainmap' file. Why does having two separate files bother you so much? Perhaps if I knew why it was important to you then I could better understand your reasoning, or, at the very least, suggest a solution that might work better. In any case, thanks for testing the patch! Jesse > > Thanks, > Kurt Bigler > > > >
