summaryrefslogtreecommitdiffstats
path: root/configserver
diff options
context:
space:
mode:
authorBjørn Christian Seime <bjorncs@verizonmedia.com>2019-05-28 12:41:22 +0200
committerBjørn Christian Seime <bjorncs@verizonmedia.com>2019-05-28 12:41:22 +0200
commitfaf72206464b5cb9fe5ad45871a1921ae6c5d77a (patch)
tree7f88225997c4548d66d813e5f2b38d264d95aafc /configserver
parentb5d61f400ad8bb6b24dc368a0de6652409a41f2b (diff)
Improve logging in MultiTenantRpcAuthorizer
Diffstat (limited to 'configserver')
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/rpc/security/MultiTenantRpcAuthorizer.java4
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;