Skip to content

Commit

Permalink
Change log level 'auth' (which doesn't exist) to 'warning'
Browse files Browse the repository at this point in the history
'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.
  • Loading branch information
jonjensen committed Nov 14, 2009
1 parent 545c1cf commit 4b02f9c
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
2 changes: 1 addition & 1 deletion lib/Vend/File.pm
Expand Up @@ -758,7 +758,7 @@ sub log_file_violation {
}

::logError($msg);
::logGlobal({ level => 'auth'}, $msg);
::logGlobal({ level => 'warning' }, $msg);
}

1;
Expand Down
11 changes: 6 additions & 5 deletions lib/Vend/Util.pm
Expand Up @@ -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,
Expand All @@ -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,
Expand All @@ -1573,7 +1574,7 @@ ALERT: Attempt to %s %s per user name:
EOF

::logGlobal(
{level => 'auth'},
{ level => 'warning' },
$fmt,
$CGI::script_name,
$msg,
Expand Down Expand Up @@ -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,
Expand Down

0 comments on commit 4b02f9c

Please sign in to comment.