commit a33f008fa154ac1fbad057653b452d94f784ca35 Author: Konstantin Ryabitsev mricon@kernel.org Date: Thu Apr 19 11:17:20 2012 -0400
Only do summary views for failures.
This fixes the logic -- we were showing summaries for both failed and successful logins, which isn't what we want.
modules/logins_mod.py | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) --- diff --git a/modules/logins_mod.py b/modules/logins_mod.py index 2d2b8a8..b46df71 100644 --- a/modules/logins_mod.py +++ b/modules/logins_mod.py @@ -639,7 +639,7 @@ class logins_mod(InternalModule): action += 10 remote = self._mk_userat(byuser, rhost)
- if self.failed_summary_only != '0': + if action == self.root_failure and self.failed_summary_only != '0': restuple = (action, service, 'total failed', '%s%s' % (system, remote)) else: @@ -655,7 +655,7 @@ class logins_mod(InternalModule): tmp = {'system': system, 'rhost': rhost} system = self.untrusted_host % tmp
- if self.failed_summary_only != '0': + if action == self.failure and self.failed_summary_only != '0': restuple = (action, service, 'total failed', system) else: # Sanitize user