Re: sender_dependent_default_transport_maps ignored by smtpd

2016-06-20 Thread Russell Yanofsky
On Mon, Jun 20, 2016 at 6:02 PM, Wietse Venema wrote: > I have opted to use the recipient as the "sender context" for sender > address resolving, if the recipient is available. This means that > the sender will "exist" when the recipient is allowed to send mail > to it. That's clever, and seems t

Re: sender_dependent_default_transport_maps ignored by smtpd

2016-06-20 Thread Wietse Venema
Russell Yanofsky: > Thanks for taking a look at the patch. I attached an updated version > that uses vstring primitives and adds the missing free. > > On Thu, Jun 16, 2016 at 7:42 PM, Wietse Venema wrote: > > One complication is that the smtpd_resolve_addr cache is not only > > used for validatin

Re: sender_dependent_default_transport_maps ignored by smtpd

2016-06-20 Thread Russell Yanofsky
Thanks for taking a look at the patch. I attached an updated version that uses vstring primitives and adds the missing free. On Thu, Jun 16, 2016 at 7:42 PM, Wietse Venema wrote: > One complication is that the smtpd_resolve_addr cache is not only > used for validating recipients, but also for val

Re: sender_dependent_default_transport_maps ignored by smtpd

2016-06-16 Thread Wietse Venema
Russell Yanofsky: > The attached patch fixes the problem for me. It changes the relevant > smtpd_resolve_addr() call in smtpd_check.c to use the message sender. One complication is that the smtpd_resolve_addr cache is not only used for validating recipients, but also for validating senders. See ch

Re: sender_dependent_default_transport_maps ignored by smtpd

2016-06-16 Thread Russell Yanofsky
The attached patch fixes the problem for me. It changes the relevant smtpd_resolve_addr() call in smtpd_check.c to use the message sender. diff --git a/src/smtpd/smtpd_check.c b/src/smtpd/smtpd_check.c index 74e42d7..2d8c6b7 100644 --- a/src/smtpd/smtpd_check.c +++ b/src/smtpd/smtpd_check.c @@ -510

Re: sender_dependent_default_transport_maps ignored by smtpd

2016-06-16 Thread Russell Yanofsky
Could anyone confirm whether this seems like a real bug before I spend time trying to work around or fix it? To summarize, my configuration is: default_transport = error:External delivery disabled sender_dependent_default_transport_maps = inline:{ @yanofsky.org=smtp:[smtp-relay.gmail.com]:587

Re: sender_dependent_default_transport_maps ignored by smtpd

2016-06-14 Thread Russell Yanofsky
To be more specific: the check_recipient_rcpt_maps() function is called multiple places, but the call that fails in this case is in the one at the bottom of smtpd_check_rcpt(). On Tue, Jun 14, 2016 at 6:28 PM, Russell Yanofsky wrote: > Hi, > > I think there is a bug in handling of sender_dependen

sender_dependent_default_transport_maps ignored by smtpd

2016-06-14 Thread Russell Yanofsky
Hi, I think there is a bug in handling of sender_dependent_default_transport_maps within smtpd when default_transport is set to error:... I'm configuring postfix as follows using default_transport and sender_dependent_default_transport_maps to reject all external outgoing mail, unless the envelop