summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBjørn Christian Seime <bjorncs@verizonmedia.com>2019-05-31 16:33:04 +0200
committerGitHub <noreply@github.com>2019-05-31 16:33:04 +0200
commita8290d04f124fdfefc6dbfba245c4605f7722b65 (patch)
tree19684f33688e7865334992f9104af45ede9128ae
parentfdb31c2917af662bad4ab7a92e59698df72998ba (diff)
parent5b3647e88a81cd4e3d22cac104eded6e14150b79 (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.java4
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);