diff options
author | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2019-05-28 12:41:22 +0200 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2019-05-28 12:41:22 +0200 |
commit | faf72206464b5cb9fe5ad45871a1921ae6c5d77a (patch) | |
tree | 7f88225997c4548d66d813e5f2b38d264d95aafc /configserver | |
parent | b5d61f400ad8bb6b24dc368a0de6652409a41f2b (diff) |
Improve logging in MultiTenantRpcAuthorizer
Diffstat (limited to 'configserver')
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/config/server/rpc/security/MultiTenantRpcAuthorizer.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/security/MultiTenantRpcAuthorizer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/security/MultiTenantRpcAuthorizer.java index 9e015e73dde..2527569bea4 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/security/MultiTenantRpcAuthorizer.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/security/MultiTenantRpcAuthorizer.java @@ -88,6 +88,8 @@ public class MultiTenantRpcAuthorizer implements RpcAuthorizer { try { getPeerIdentity(request) .ifPresent(peerIdentity -> authorizer.accept(request, peerIdentity)); + log.log(LogLevel.DEBUG, () -> String.format("Authorization succeeded for request '%s' from '%s'", + request.methodName(), request.target().toString())); } catch (Throwable t) { handleAuthorizationFailure(request, t); } @@ -155,7 +157,7 @@ public class MultiTenantRpcAuthorizer implements RpcAuthorizer { } private void handleAuthorizationFailure(Request request, Throwable throwable) { - String errorMessage = String.format("For request from '%s': %s", request.target().toString(), throwable.getMessage()); + String errorMessage = String.format("For request '%s' from '%s': %s", request.methodName(), request.target().toString(), throwable.getMessage()); log.log(LogLevel.WARNING, errorMessage, throwable); if (mode == Mode.ENFORCE) { JrtErrorCode error = throwable instanceof AuthorizationException ? JrtErrorCode.UNAUTHORIZED : JrtErrorCode.AUTHORIZATION_FAILED; |