Send inn-workers mailing list submissions to
        [email protected]

To subscribe or unsubscribe via the World Wide Web, visit
        https://lists.isc.org/mailman/listinfo/inn-workers
or, via email, send a message with subject or body 'help' to
        [email protected]

You can reach the person managing the list at
        [email protected]

When replying, please edit your Subject line so it is more specific
than "Re: Contents of inn-workers digest..."


Today's Topics:

   1. Re: Relaxing a few checks (Julien ?LIE)
   2. Re: Relaxing a few checks (Kamil =?iso-8859-2?Q?Jo=F1ca?=)


----------------------------------------------------------------------

Message: 1
Date: Wed, 22 Mar 2017 14:44:34 +0100
From: Julien ?LIE <[email protected]>
To: [email protected]
Cc: Kamil Jo?ca <[email protected]>
Subject: Re: Relaxing a few checks
Message-ID: <[email protected]>
Content-Type: text/plain; charset=utf-8; format=flowed

Hi Kamil,

>> More generally, does anyone have in mind other checks that INN
>> currently performs but you find annoying? And on the contrary,
>> checks that INN currently does not perform but you would like to?
>
> Recently innd/nnrpd has been more restrict about empty headers. I
> have some mail grops that I fed to inn with mailpost. And I got a
> lot of errors. I know that proper way is to change mailpost, but this
> relax check would be a nice workaround.

Thanks for the suggestion of a new relaxed check.

It is of course also a mailpost bug that should be fixed.
Could you please try the following patch for mailpost?

--- mailpost    2017-03-20 15:26:45.000000000 +0100
+++ mailpost    2017-03-20 16:02:52.000000000 +0100
@@ -213,6 +213,9 @@
      next if /^Approved:\s/sio && defined($approved);
      next if /^Distribution:\s/sio && defined($distribution);

+    # Remove empty header field bodies.
+    next if /^[^:]+:\s*$/;
+
      if (/^($exclude):\s*/sio) {
      $real_news_hdrs .= "$_\n";
      next;




I've tested it and made sure it correctly removes empty header fields like:

Test:    \t   \r\n

or multi-line like:

Test:  \r\n
    \r\n

-- 
Julien ?LIE

? PowerPoint allows speakers to pretend that they are giving a real
   talk, and audiences to pretend that they are listening. ? (Edward R.
   Tufte, _The Cognitive Style of PowerPoint_)


------------------------------

Message: 2
Date: Thu, 23 Mar 2017 08:54:17 +0100
From: [email protected] (Kamil =?iso-8859-2?Q?Jo=F1ca?=)
To: Julien ?LIE <[email protected]>
Cc: [email protected]
Subject: Re: Relaxing a few checks
Message-ID: <[email protected]>
Content-Type: text/plain; charset=iso-8859-2

Julien ?LIE <[email protected]> writes:

[...]
> It is of course also a mailpost bug that should be fixed.
> Could you please try the following patch for mailpost?
>
> --- mailpost    2017-03-20 15:26:45.000000000 +0100
> +++ mailpost    2017-03-20 16:02:52.000000000 +0100
> @@ -213,6 +213,9 @@
>      next if /^Approved:\s/sio && defined($approved);
>      next if /^Distribution:\s/sio && defined($distribution);
>
> +    # Remove empty header field bodies.
> +    next if /^[^:]+:\s*$/;
> +
>      if (/^($exclude):\s*/sio) {
>      $real_news_hdrs .= "$_\n";

It looks it works.
Thanks.

KJ

-- 
http://stopstopnop.pl/stop_stopnop.pl_o_nas.html
"It may be that our role on this planet is not to worship God but to
 create him."
                -- Arthur C. Clarke


------------------------------

Subject: Digest Footer

_______________________________________________
inn-workers mailing list
[email protected]
https://lists.isc.org/mailman/listinfo/inn-workers

------------------------------

End of inn-workers Digest, Vol 93, Issue 3
******************************************

Reply via email to