mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Move authentication failed logging to checkPassword 

Fixes https://github.com/owncloud/core/issues/10366
This commit is contained in:
Lukas Reschke 2014-08-15 12:13:00 +02:00
parent 93f5e90c9f
commit 5bb4772858
3 changed files with 2 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
config/config.sample.php
View File

@ -185,9 +185,6 @@ $CONFIG = array(
(watch out, this option can increase the size of your log file)*/ (watch out, this option can increase the size of your log file)*/
"log_query" => false, "log_query" => false,
/* Enable or disable the logging of IP addresses in case of webform auth failures */
"log_authfailip" => false,
/* Whether ownCloud should log the last successfull cron exec */ /* Whether ownCloud should log the last successfull cron exec */
"cron_log" => true, "cron_log" => true,

7
lib/base.php
View File

@ -856,13 +856,6 @@ class OC {
} // logon via web form } // logon via web form
elseif (OC::tryFormLogin()) { elseif (OC::tryFormLogin()) {
$error[] = 'invalidpassword'; $error[] = 'invalidpassword';
if ( OC_Config::getValue('log_authfailip', false) ) {
OC_Log::write('core', 'Login failed: user \''.$_POST["user"].'\' , wrong password, IP:'.$_SERVER['REMOTE_ADDR'],
OC_Log::WARN);
} else {
OC_Log::write('core', 'Login failed: user \''.$_POST["user"].'\' , wrong password, IP:set log_authfailip=true in conf',
OC_Log::WARN);
}
} }
OC_Util::displayLoginPage(array_unique($error)); OC_Util::displayLoginPage(array_unique($error));

2
lib/private/user/manager.php
View File

@ -164,6 +164,8 @@ class Manager extends PublicEmitter implements IUserManager {
} }
} }
} }
\OC::$server->getLogger()->warning('Login failed: \''. $loginname .'\' (Remote IP: \''. $_SERVER['REMOTE_ADDR'] .'\', X-Forwarded-For: \''. $_SERVER['HTTP_X_FORWARDED_FOR'] .'\')', array('app' => 'core'));
return false; return false;
} }