diff options
author | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2019-05-31 16:33:04 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-05-31 16:33:04 +0200 |
commit | a8290d04f124fdfefc6dbfba245c4605f7722b65 (patch) | |
tree | 19684f33688e7865334992f9104af45ede9128ae | |
parent | fdb31c2917af662bad4ab7a92e59698df72998ba (diff) | |
parent | 5b3647e88a81cd4e3d22cac104eded6e14150b79 (diff) |
Merge pull request #9635 from vespa-engine/bjorncs/configserver-rpc-authz-fix
Fix logic for unknown hosts
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/config/server/rpc/security/MultiTenantRpcAuthorizer.java | 4 |
1 files changed, 2 insertions, 2 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 f9361b43045..b7bc70ff648 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 @@ -112,10 +112,10 @@ public class MultiTenantRpcAuthorizer implements RpcAuthorizer { } else { String hostname = configRequest.getClientHostName(); Optional<RequestHandler> tenantHandler = - Optional.of(hostRegistry.getKeyForHost(hostname)) + Optional.ofNullable(hostRegistry.getKeyForHost(hostname)) .flatMap(this::getTenantHandler); if (tenantHandler.isEmpty()) { - return; // unknown tenant + return; // unknown host } ApplicationId resolvedApplication = tenantHandler.get().resolveApplicationId(hostname); ApplicationId peerOwner = applicationId(peerIdentity); |