opendev/gerrit
Code Issues Proposed changes

CurrentUser: Add method for getting a loggable name

Browse Source 
This is useful for including a CurrentUser into log messages.

Change-Id: I25631902e5fb7b69960b3f53d62b51de3c70e9e2
Signed-off-by: Edwin Kempin <ekempin@google.com>
This commit is contained in:
Edwin Kempin
2018-02-20 08:17:38 +01:00
parent 7aef822f5c
commit f825323de8
2 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
java/com/google/gerrit/server/CurrentUser.java
View File

@@ -95,6 +95,11 @@ public abstract class CurrentUser {
return Optional.empty(); return Optional.empty();
} }
/** @return unique name of the user for logging, never {@code null} */
public String getLoggableName() {
return getUserName().orElseGet(() -> getClass().getSimpleName());
}
/** Check if user is the IdentifiedUser */ /** Check if user is the IdentifiedUser */
public boolean isIdentifiedUser() { public boolean isIdentifiedUser() {
return false; return false;

1
java/com/google/gerrit/server/IdentifiedUser.java
View File

@@ -323,6 +323,7 @@ public class IdentifiedUser extends CurrentUser {
} }
/** @return unique name of the user for logging, never {@code null} */ /** @return unique name of the user for logging, never {@code null} */
@Override
public String getLoggableName() { public String getLoggableName() {
return getUserName() return getUserName()
.orElseGet( .orElseGet(