You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Ian Holsman <ia...@cnet.com> on 2001/08/17 23:13:15 UTC

[PATCH] a different bug in mod-include fixed

Path_info is not set when the request is proxied.
(can this patch make the 2.0.24) ?
..Ian

Index: mod_include.c
===================================================================
RCS file: /home/cvspublic/httpd-2.0/modules/filters/mod_include.c,v
retrieving revision 1.124
diff -u -u -r1.124 mod_include.c
--- mod_include.c    2001/08/17 20:41:15    1.124
+++ mod_include.c    2001/08/17 21:07:34
@@ -113,7 +113,10 @@
     apr_table_setn(e, "LAST_MODIFIED",
               ap_ht_time(r->pool, r->finfo.mtime, timefmt, 0));
     apr_table_setn(e, "DOCUMENT_URI", r->uri);
-    apr_table_setn(e, "DOCUMENT_PATH_INFO", r->path_info);
+    /* path info is not set if the pages has come from mod-proxy */
+    if (r->path_info && *r->path_info ) {
+        apr_table_setn(e, "DOCUMENT_PATH_INFO", r->path_info);
+    }
     if (apr_get_username(&pwname, r->finfo.user, r->pool) == APR_SUCCESS) {
         apr_table_setn(e, "USER_NAME", pwname);
     }