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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.