opendev/gerrit
Code Issues Proposed changes

SshDaemon: Add a metric for authentication failures

Browse Source 
Change-Id: I02b1f87a1b9e00ed59857f506c394705e005e739
This commit is contained in:
David Pursehouse
2015-11-11 11:28:30 -08:00
parent 01e08a2154
commit 40f980e67f
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
gerrit-sshd/src/main/java/com/google/gerrit/sshd/SshDaemon.java
View File

@@ -272,6 +272,12 @@ public class SshDaemon extends SshServer implements SshInfo, LifecycleListener {
.setRate() .setRate()
.setUnit("sessions")); .setUnit("sessions"));
final Counter0 authFailures = metricMaker.newCounter(
"sshd/sessions/authentication-failed",
new Description("Rate of SSH authentication failures")
.setRate()
.setUnit("failures"));
setSessionFactory(new SessionFactory() { setSessionFactory(new SessionFactory() {
@Override @Override
protected AbstractSession createSession(final IoSession io) protected AbstractSession createSession(final IoSession io)
@@ -300,6 +306,7 @@ public class SshDaemon extends SshServer implements SshInfo, LifecycleListener {
public void operationComplete(CloseFuture future) { public void operationComplete(CloseFuture future) {
connected.decrementAndGet(); connected.decrementAndGet();
if (sd.isAuthenticationError()) { if (sd.isAuthenticationError()) {
authFailures.increment();
sshLog.onAuthFail(sd); sshLog.onAuthFail(sd);
} }
} }