[PATCH] Core: use uppercase hexadecimal digits for percent-encoding

Maxim Dounin mdounin at mdounin.ru
Tue Jul 1 01:13:18 UTC 2014


On Mon, Jun 30, 2014 at 04:24:58PM -0700, Yichun Zhang (agentzh) wrote:

> Hello!
> On Fri, Jun 27, 2014 at 10:57 AM, Maxim Dounin wrote:
> > I can't say I like this change.  I've considered this a while ago,
> > and decided to keep it as is.  This preserve compatibility with
> > what nginx used to do for years.  And it also looks like Apache
> > does the same.
> >
> > Any other opinions?
> >
> I agree with Piotr Sikora and hope that nginx uses %DD instead of %dd
> for URI escaping. Right now ngx_lua also uses %dd for the consistency
> with the nginx core, which has already yielded several complaints from
> ngx_lua's user community (the users also pointed me to the RFC). Once
> the nginx core switches over to %DD, I can make a similar change to
> ngx_lua accordingly :)

Thanks, counted.

Maxim Dounin

More information about the nginx-devel mailing list