Re: PATCH: disable mailbox delivery
Jörg Delker <[email protected]>
| Newsgroups | gmane.comp.isp.ispman.devel |
|---|---|
| Message-ID | <[email protected]> |
Obviously, I forgot to send my comments to your patches... Well, better late than never ;) John K. Hohm, ProviNET Solutions wrote: > I hope sending occasional patches to the developers list is appropriate. Of course! > This patch is against HEAD. > > ChangeLog: > * Replace "Primary Mail Box Address" in user properties with "Mail Box > Delivery", which allows mailbox delivery to be enabled or disabled. > Implemented by generating (as in users.ldif.template) or clearing > mailRoutingAddress. > > -- John K. Hohm, ProviNET Solutions <jhohm-A2FxZZ/Jk9tWk0Htik3J/[email protected]> Though, this patch looks quite good, it doesn't work unfortunately. The problem is that you are disabling the mailRoutingAddress attribute, while keeping the mailLocalAddress attribute. This causes postfix to accept the mail (due to valid maillocaladdress) but can't deliver it (missing mailroutingaddress) resulting in a mail loop (mx lookup, redeliver to mx which is the host itself) with a final mailbounce. A proper solution is not that easy IMHO: To properly disable mail for an user, the maillocaladdress has to be changed/deleted, which could be a problem when mail retreival is enabled again (attribute value may not be recoverable). ------------------------------------------------------- This SF.Net email sponsored by Black Hat Briefings & Training. Attend Black Hat Briefings & Training, Las Vegas July 24-29 - digital self defense, top technical experts, no vendor pitches, unmatched networking opportunities. Visit www.blackhat.com