Re: [pmg-devel] [PATCH pmg-api v2 0/4] improve local mail injection and add smtputf8 support

[Dominik Csapak <d.csapak@proxmox.com>]

2 high level comments
(one we already talked off-list, just to have it on record)

1. it seems that with this, at least the receiver gets encoded too many
times, since i get a different recipient (with non-ascii characters)
when hitting an ACCEPT rule vs. hitting a QUARANTINE rule and
delivering it (not sure which patch is the faulty one here,
but i guess api 4/4?)

2. AFAICS for 'reinject_local_mail' we decide to put SMTPUTF8 when
the sender/receiver has non-ascii characters

shouldn't we also make this depend on the config smtputf8 setting?

since when we accept a mail right through, we don't scan the
receiver/sender for that when no smtputf8 is set
(postfix docs say here that they don't accept utf8
without SMTPUTF8 dialog, but it worked when testing here...)
but do so when it goes into quarantine + deliver

rationale is that when i have to disable smtputf8 support,
it's because my downstream mail server does not accept it.
in that case, every attempt to send such a mail will fail
making the quarantine, etc. useless

what do you think?