From 4b02f9c9a28e1c687c32a51cc6b68a6480b6e2eb Mon Sep 17 00:00:00 2001 From: Jon Jensen Date: Sat, 14 Nov 2009 10:26:25 -0700 Subject: [PATCH] Change log level 'auth' (which doesn't exist) to 'warning' 'auth' is a facility, not a level. The erroneous 'auth' level setting was formerly a no-op because 'auth' wouldn't exist in a correctly-configured SysLog hash, but with the new logging code it actually passed through as intended and raised an error both in /usr/bin/logger and Sys::Syslog calls. --- lib/Vend/File.pm | 2 +- lib/Vend/Util.pm | 11 ++++++----- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/lib/Vend/File.pm b/lib/Vend/File.pm index e456bf65e..f88040523 100644 --- a/lib/Vend/File.pm +++ b/lib/Vend/File.pm @@ -758,7 +758,7 @@ sub log_file_violation { } ::logError($msg); - ::logGlobal({ level => 'auth'}, $msg); + ::logGlobal({ level => 'warning' }, $msg); } 1; diff --git a/lib/Vend/Util.pm b/lib/Vend/Util.pm index e3f270da4..dae65731a 100644 --- a/lib/Vend/Util.pm +++ b/lib/Vend/Util.pm @@ -1527,7 +1527,8 @@ ALERT: Attempt to %s at %s from: SCRIPT_NAME %s PATH_INFO %s EOF - logGlobal ({level => 'auth'}, $fmt, + logGlobal({ level => 'warning' }, + $fmt, $msg, $CGI::script_name, $CGI::host, @@ -1548,7 +1549,7 @@ EOF ne $Vend::Cfg->{Password}) { ::logGlobal( - {level => 'auth'}, + { level => 'warning' }, "ALERT: Password mismatch, attempt to %s at %s from %s", $msg, $CGI::script_name, @@ -1573,7 +1574,7 @@ ALERT: Attempt to %s %s per user name: EOF ::logGlobal( - {level => 'auth'}, + { level => 'warning' }, $fmt, $CGI::script_name, $msg, @@ -1601,8 +1602,8 @@ Attempt to %s on %s, secure operations disabled. SCRIPT_NAME %s PATH_INFO %s EOF - ::logGlobal ( - {level => 'auth'}, + ::logGlobal( + { level => 'warning' }, $fmt, $msg, $CGI::script_name,