Re: cvs commit: squid/lib rfc1738.c

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: cvs commit: squid/lib rfc1738.c

Adrian Chadd-3
On Wed, May 23, 2007, Henrik Nordstrom wrote:
> hno         2007/05/23 16:00:02 MDT
>
>   Modified files:
>     lib                  rfc1738.c
>   Log:
>   MFC: URI-escape using the recommended upper case
>  

Ta for doing this; the only reason I didn't touch it when I saw
the discrepancy between the code and RFC (3986 though)
is what it might break. I was going to talk to you about it today.



Adrian

Reply | Threaded
Open this post in threaded view
|

Re: cvs commit: squid/lib rfc1738.c

Henrik Nordström
ons 2007-05-23 klockan 19:35 -0600 skrev Adrian Chadd:

> Ta for doing this; the only reason I didn't touch it when I saw
> the discrepancy between the code and RFC (3986 though)
> is what it might break. I was going to talk to you about it today.

I'm not very worried about breakage here.

The only questionmark is on helpers, but I think all uses case-neutral
parsers..  (queries to helpers is uri-escaped).

Regards
Henrik



signature.asc (316 bytes) Download Attachment