Proposed Patch - url encoding all upper case


#1

I’d like to propose that URI encoding be done using upper case
characters instead of lower case characters. Both seem to be acceptable;
I’m only requesting this since most of the other libraries that we use
that do URI encoding use upper case characters.

nginx 0.7.6 src/core/ngx_string.c

diff ngx_string.c~ ngx_string.c
1096c1096
< static u_char hex[] = “0123456789abcdef”;

static u_char   hex[] = "0123456789ABCDEF";

#2

å¯ä»¥æ”¹ä»£ç ç¼–è¯‘åŽç›´æŽ¥ä½¿ç”¨ï¼›ä¹Ÿå¯ä»¥ç”¨å†…åµŒçš„perl灵活设置,效率也非常高

Posted at Nginx Forum:
http://forum.nginx.org/read.php?2,1918,1920#msg-1920


#3

坏人 wrote:

å¯ä»¥æ”¹ä»£ç ç¼–è¯‘åŽç›´æŽ¥ä½¿ç”¨ï¼›ä¹Ÿå¯ä»¥ç”¨å†…åµŒçš„perl灵活设置,效率也非常高

Posted at Nginx Forum: http://forum.nginx.org/read.php?2,1918,1920#msg-1920

The reason that I’m asking for the change in NGINX is that in one
particular application (X-Accel-Redirect to AWS S3), calculation of a
server variable is done on URL encodings done with UPPER CASE
characters.

We supply NON-URL encoded URLs to X-Accel-Redirect - these are
re-encoded by nginx using lower case characters for any hex values, and
consequently the server variable calculation (already done before
X-Accel-Redirect) fails.


#4

Please excuse me for the necroposting, but what’s the decision for the
proposed patch?
My Nginx reverse proxy malfunctions due to this problem.(
Is there any workaround for the problem?

Posted at Nginx Forum:
http://forum.nginx.org/read.php?2,1918,217439#msg-217439