Re: test debug

[Ammar Faizi <ammarfaizi2@gnuweeb.org>]

On 3/6/22 6:08 PM, Alviro Iskandar Setiawan wrote:
> On Sun, Mar 6, 2022 at 6:04 PM Ammar Faizi wrote:
>> On 3/6/22 6:03 PM, Alviro Iskandar Setiawan wrote:
>>> test again
>>
>> Got your email!
>>
> 
> I think this regex is wrong sir, if the message-id is located at the
> end of the header, we will not have double LF, because it's already
> split by the explode.
> 
> if (!preg_match("/(?:^|\\n)message-id:\s+?(.+?)(?:\\n\S+\:|\\n\\n)/si",
> $hdr, $m)) {
>          $err = "Cannot get the \"message-id\" line";
>          goto out;
> }

Oh right, I forgot that we've already split it before parsing.

> maybe we should use this:
> 
>     "/(?:^|\\n)message-id:\s+?(.+?)(?:\\n\S+\:|$)/si"

That one looks good to me, we should also fix other headers too.

-- 
Ammar Faizi