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

Yichun Zhang (agentzh) agentzh at gmail.com
Mon Jun 30 23:24:58 UTC 2014


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 :)

Best regards,
-agentzh



More information about the nginx-devel mailing list