[PATCH] making ngx_http_parse_time support 1 digit day

Maxim Dounin mdounin at mdounin.ru
Fri Nov 9 17:39:25 UTC 2012


Hello!

On Fri, Nov 09, 2012 at 02:33:33PM -0200, Wandenberg Peixoto wrote:

> Hello Maxim,
> 
> thanks for you explanation.
> Only one doubt, the rfc2616 on the same paragraph says
> "HTTP/1.1 clients and servers that parse the date value MUST accept all
> three formats (for compatibility with HTTP/1.0), though they MUST only
> generate the RFC 1123 <http://tools.ietf.org/html/rfc1123> format for
> representing HTTP-date values in header fields. See section
> 19.3<http://tools.ietf.org/html/rfc2616#section-19.3>for further
> information."
> 
> This isn't the case for relax the parser? We are parsing dates.

What nginx currently do is exactly as required by the quoted text: 
it accepts all three formats specified.

> 
> Regards
> 
> 
> On Fri, Nov 9, 2012 at 1:53 PM, Maxim Dounin <mdounin at mdounin.ru> wrote:
> 
> > Hello!
> >
> > On Fri, Nov 09, 2012 at 12:37:50PM -0200, Wandenberg Peixoto wrote:
> >
> > > I was checking a strange behavior when parsing dates coming from
> > different
> > > browsers with this function and observed that it not supports dates with
> > > only one digit, it expect that the date comes with 01, 02, 03 ...
> > > But as rfc822 and rfc2822 says the day can be 1, 2 , 3, ... 10, 15, ...
> > > So I am proposing this patch to be compliant with the rfc.
> >
> > What nginx supports is specified here:
> >
> > http://tools.ietf.org/html/rfc2616#section-3.3.1
> >
> > This includes support for 1-digit years as per asctime() format.
> > Format which is subset of date format defined by RFC 1123 (RFC822)
> > require two digits as per syntax.
> >
> > That is, nginx is perfectly RFC complaint - you just looking at
> > wrong RFC.
> >
> > Just for reference, here are the syntax provided in RFC 2616:
> >
> >        HTTP-date    = rfc1123-date | rfc850-date | asctime-date
> >        rfc1123-date = wkday "," SP date1 SP time SP "GMT"
> >        rfc850-date  = weekday "," SP date2 SP time SP "GMT"
> >        asctime-date = wkday SP date3 SP time SP 4DIGIT
> >        date1        = 2DIGIT SP month SP 4DIGIT
> >                       ; day month year (e.g., 02 Jun 1982)
> >        date2        = 2DIGIT "-" month "-" 2DIGIT
> >                       ; day-month-year (e.g., 02-Jun-82)
> >        date3        = month SP ( 2DIGIT | ( SP 1DIGIT ))
> >                       ; month day (e.g., Jun  2)
> >
> > > I expect that this patch could be applied to nginx code.
> >
> > I don't think it's a good idea to relax parsing beyond what
> > RFC 2616 requires unless there is really good reason to.
> >
> > [...]
> >
> > --
> > Maxim Dounin
> > http://nginx.com/support.html
> >
> > _______________________________________________
> > nginx-devel mailing list
> > nginx-devel at nginx.org
> > http://mailman.nginx.org/mailman/listinfo/nginx-devel
> >

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


-- 
Maxim Dounin
http://nginx.com/support.html



More information about the nginx-devel mailing list