You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by be...@apache.org on 2004/01/15 21:46:21 UTC
cvs commit: apache-1.3/src/support check_forensic
ben 2004/01/15 12:46:21
Modified: src CHANGES
src/modules/standard mod_log_forensic.c
src/support check_forensic
Log:
Don't log twice.
Revision Changes Path
1.1921 +3 -0 apache-1.3/src/CHANGES
Index: CHANGES
===================================================================
RCS file: /home/cvs/apache-1.3/src/CHANGES,v
retrieving revision 1.1920
retrieving revision 1.1921
diff -u -r1.1920 -r1.1921
--- CHANGES 13 Jan 2004 21:03:06 -0000 1.1920
+++ CHANGES 15 Jan 2004 20:46:20 -0000 1.1921
@@ -1,5 +1,8 @@
Changes with Apache 1.3.30
+ *) Forensic logging shouldn't log internal redirects.
+ [Ivan Ristic <iv...@webkreator.com>]
+
*) Some syntax errors in mod_mime_magic's magic file can result
in a 500 error, which previously was unlogged. Now we log the
error. [Jeff Trawick]
1.4 +1 -1 apache-1.3/src/modules/standard/mod_log_forensic.c
Index: mod_log_forensic.c
===================================================================
RCS file: /home/cvs/apache-1.3/src/modules/standard/mod_log_forensic.c,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- mod_log_forensic.c 3 Jan 2004 16:53:30 -0000 1.3
+++ mod_log_forensic.c 15 Jan 2004 20:46:20 -0000 1.4
@@ -219,7 +219,7 @@
const char *id;
hlog h;
- if (cfg->fd < 0)
+ if (cfg->fd < 0 || r->prev)
return DECLINED;
if (!(id = ap_table_get(r->subprocess_env, "UNIQUE_ID"))) {
1.2 +1 -0 apache-1.3/src/support/check_forensic
Index: check_forensic
===================================================================
RCS file: /home/cvs/apache-1.3/src/support/check_forensic,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- check_forensic 31 Dec 2003 11:58:21 -0000 1.1
+++ check_forensic 15 Jan 2004 20:46:20 -0000 1.2
@@ -10,5 +10,6 @@
cut -f 1 -d '|' $F > /tmp/fc-all.$$
grep + < /tmp/fc-all.$$ | cut -c2- | sort > /tmp/fc-in.$$
grep -- - < /tmp/fc-all.$$ | cut -c2- | sort > /tmp/fc-out.$$
+# use -i instead of -I for GNU xargs
join -v 1 /tmp/fc-in.$$ /tmp/fc-out.$$ | xargs -I xx egrep "^\\+xx" $F
rm /tmp/fc-all.$$ /tmp/fc-in.$$ /tmp/fc-out.$$