diff options
| author | Anton Tananaev <anton.tananaev@gmail.com> | 2020-01-15 21:02:16 -0800 |
|---|---|---|
| committer | Anton Tananaev <anton.tananaev@gmail.com> | 2020-01-15 21:02:16 -0800 |
| commit | 10ce85ef9eed15276534938c7a18c04cb18ca7f2 (patch) | |
| tree | 53ae265c2fda1def1d41c1bf96898a60c9081d58 /src/main/java/org/traccar/helper/LogAction.java | |
| parent | 729021a19086d982c1b4ec5661d9140c42f030f8 (diff) | |
| download | trackermap-server-10ce85ef9eed15276534938c7a18c04cb18ca7f2.tar.gz trackermap-server-10ce85ef9eed15276534938c7a18c04cb18ca7f2.tar.bz2 trackermap-server-10ce85ef9eed15276534938c7a18c04cb18ca7f2.zip | |
Clean login failure logging
Diffstat (limited to 'src/main/java/org/traccar/helper/LogAction.java')
| -rw-r--r-- | src/main/java/org/traccar/helper/LogAction.java | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/src/main/java/org/traccar/helper/LogAction.java b/src/main/java/org/traccar/helper/LogAction.java index 21fe4c3fa..16d55ec60 100644 --- a/src/main/java/org/traccar/helper/LogAction.java +++ b/src/main/java/org/traccar/helper/LogAction.java @@ -1,5 +1,5 @@ /* - * Copyright 2017 Anton Tananaev (anton@traccar.org) + * Copyright 2017 - 2020 Anton Tananaev (anton@traccar.org) * Copyright 2017 Andrey Kunitsyn (andrey@traccar.org) * * Licensed under the Apache License, Version 2.0 (the "License"); @@ -38,14 +38,13 @@ public final class LogAction { private static final String ACTION_LOGIN = "login"; private static final String ACTION_LOGOUT = "logout"; - private static final String ACTION_FAILED_LOGIN_NO_IP = "Failed Login Attempt. IP address: failed to retrieve"; private static final String ACTION_DEVICE_ACCUMULATORS = "resetDeviceAccumulators"; private static final String PATTERN_OBJECT = "user: %d, action: %s, object: %s, id: %d"; private static final String PATTERN_LINK = "user: %d, action: %s, owner: %s, id: %d, property: %s, id: %d"; private static final String PATTERN_LOGIN = "user: %d, action: %s"; - private static final String PATTERN_FAILED_LOGIN = "Failed Login Attempt. IP address: %s"; + private static final String PATTERN_LOGIN_FAILED = "login failed from: %s"; private static final String PATTERN_DEVICE_ACCUMULATORS = "user: %d, action: %s, deviceId: %d"; public static void create(long userId, BaseModel object) { @@ -76,18 +75,13 @@ public final class LogAction { logLoginAction(ACTION_LOGOUT, userId); } - public static void failedLogin(String ipAddress) { - - if (ipAddress == null || ipAddress.isEmpty()) { - LOGGER.info(ACTION_FAILED_LOGIN_NO_IP); - } else { - LOGGER.info(String.format( - PATTERN_FAILED_LOGIN, ipAddress)); + public static void failedLogin(String remoteAddress) { + if (remoteAddress == null || remoteAddress.isEmpty()) { + remoteAddress = "unknown"; } - + LOGGER.info(String.format(PATTERN_LOGIN_FAILED, remoteAddress)); } - public static void resetDeviceAccumulators(long userId, long deviceId) { LOGGER.info(String.format( PATTERN_DEVICE_ACCUMULATORS, userId, ACTION_DEVICE_ACCUMULATORS, deviceId)); |