I'm planning to submit a patch for Apache 1.x and 2.x that would add
real URI escaping to ap_escape_uri the same way as URI::Escape does.  To
prepare for this and IMHO anyway, Apache::Util::escape_uri should use
ap_escape_uri instead of ap_os_escape_path.  ap_escape_uri has been
around since Apache 1.3.4.

Patch against the current modperl 1 CVS follows, I think it should be in
2.0 too but the implementation there is a bit different and I couldn't
figure it out in 2 minutes... :)

Index: src/modules/perl/Util.xs
===================================================================
RCS file: /home/cvspublic/modperl/src/modules/perl/Util.xs,v
retrieving revision 1.11
diff -a -u -r1.11 Util.xs
--- src/modules/perl/Util.xs    25 Mar 2002 18:45:23 -0000      1.11
+++ src/modules/perl/Util.xs    9 Jun 2002 14:18:10 -0000
@@ -95,7 +95,7 @@
     const char *segment

     CODE:
-    RETVAL = ap_os_escape_path(util_pool(), segment, TRUE);
+    RETVAL = ap_escape_uri(util_pool(), segment);

     OUTPUT:
     RETVAL

-- 
Ville Skytt�
ville.skytta at iki.fi


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to