diff options
author | Valerij Fredriksen <valerijf@verizonmedia.com> | 2019-11-01 16:23:51 +0100 |
---|---|---|
committer | Valerij Fredriksen <valerijf@verizonmedia.com> | 2019-11-01 16:23:51 +0100 |
commit | 8d96fe5afad6fe64e0b85bda34f6d7bad8a8e5d2 (patch) | |
tree | 0ce71bf990700ebb82aa38060c41ef5847241921 /controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java | |
parent | 058812ebc1baec63de17c700b22ff806809e69e0 (diff) |
Create client once in ConfigServerRestExecutorImpl
Diffstat (limited to 'controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java | 12 |
1 files changed, 1 insertions, 11 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java index f23161664a2..99cc78a2614 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java @@ -10,7 +10,6 @@ import com.yahoo.restapi.Path; import com.yahoo.restapi.SlimeJsonResponse; import com.yahoo.slime.Cursor; import com.yahoo.slime.Slime; -import com.yahoo.vespa.athenz.tls.AthenzIdentityVerifier; import com.yahoo.vespa.hosted.controller.Controller; import com.yahoo.vespa.hosted.controller.api.integration.zone.ZoneRegistry; import com.yahoo.vespa.hosted.controller.auditlog.AuditLoggingRequestHandler; @@ -18,12 +17,9 @@ import com.yahoo.vespa.hosted.controller.proxy.ConfigServerRestExecutor; import com.yahoo.vespa.hosted.controller.proxy.ProxyException; import com.yahoo.vespa.hosted.controller.proxy.ProxyRequest; import com.yahoo.yolean.Exceptions; -import org.apache.http.conn.ssl.DefaultHostnameVerifier; -import javax.net.ssl.HostnameVerifier; import java.net.URI; import java.util.List; -import java.util.Set; import java.util.logging.Level; import java.util.stream.Stream; @@ -98,8 +94,7 @@ public class ConfigServerApiHandler extends AuditLoggingRequestHandler { } try { - return proxy.handle(new ProxyRequest( - request, List.of(getEndpoint(zoneId)), createHostnameVerifier(zoneId), cfgPath)); + return proxy.handle(new ProxyRequest(request, List.of(getEndpoint(zoneId)), cfgPath)); } catch (ProxyException e) { throw new RuntimeException(e); } @@ -129,9 +124,4 @@ public class ConfigServerApiHandler extends AuditLoggingRequestHandler { private URI getEndpoint(ZoneId zoneId) { return CONTROLLER_ZONE.equals(zoneId) ? zoneRegistry.apiUrl() : zoneRegistry.getConfigServerVipUri(zoneId); } - - private HostnameVerifier createHostnameVerifier(ZoneId zoneId) { - if (CONTROLLER_ZONE.equals(zoneId)) return new DefaultHostnameVerifier(); - return new AthenzIdentityVerifier(Set.of(zoneRegistry.getConfigServerHttpsIdentity(zoneId))); - } } |