[PATCH] Update mail parsing to be per protocol (imap/pop/smtp)

[Kunal Pariani]

The patch certainly adds more intelligence in the mail parser per protocol (adding more specific states & error codes) and seems valid even with the latest nginx mainline. 

Thanks
-Kunal

----- Original Message -----
From: "Maxim Dounin" <mdounin at mdounin.ru>
To: "nginx-devel" <nginx-devel at nginx.org>
Sent: Thursday, September 18, 2014 4:00:32 AM
Subject: Re: [PATCH] Update mail parsing to be per protocol (imap/pop/smtp)

Hello!

On Wed, Sep 17, 2014 at 03:49:50PM -0500, Kunal Pariani wrote:

> Hello,
> This patch adds separate mail parsing states per protocol 
> (imap/pop/smtp), specific error codes for parsing/auth/login 
> failures. Also includes some minor bug fixes in the mail parsing 
> code. Requesting to include this patch as Zimbra has been using 
> this since nginx 0.5.37, so it's quite heavily tested.

Mail parsing code a bit changed since 0.5.37, so this is more like 
an argument against.

In any case, it's not clear what the patch tries to achieve.  It's 
highly unlikely that it will be considered, see
http://nginx.org/en/docs/contributing_changes.html.

-- 
Maxim Dounin
http://nginx.org/

_______________________________________________
nginx-devel mailing list
nginx-devel at nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel