diff options
author | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2019-05-23 20:51:25 +0200 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2019-05-23 20:51:25 +0200 |
commit | a5053473a8905ab129664eef30487191ec6da079 (patch) | |
tree | 1bb81ef54f174ba4abc3fb03390f3744fc39f68c | |
parent | a717bfeecf8a0c0f237685a4b2b37035d4cad553 (diff) |
Call getPeerIdentity() from inside try-catch block
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/config/server/rpc/security/MultiTenantRpcAuthorizer.java | 16 |
1 files changed, 8 insertions, 8 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 bed9ca3eba3..9e015e73dde 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 @@ -84,14 +84,14 @@ public class MultiTenantRpcAuthorizer implements RpcAuthorizer { private CompletableFuture<Void> doAsyncAuthorization(Request request, BiConsumer<Request, NodeIdentity> authorizer) { return CompletableFuture.runAsync( - () -> getPeerIdentity(request) - .ifPresent(peerIdentity -> { - try { - authorizer.accept(request, peerIdentity); - } catch (Throwable t) { - handleAuthorizationFailure(request, t); - } - }), + () -> { + try { + getPeerIdentity(request) + .ifPresent(peerIdentity -> authorizer.accept(request, peerIdentity)); + } catch (Throwable t) { + handleAuthorizationFailure(request, t); + } + }, executor); } |