Re: test debug

[Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>]

On Sun, Mar 6, 2022 at 6:12 PM Ammar Faizi wrote:
> 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.

I will send a patch to fix this, maybe tonight

-- Viro