From 43c94d8a5f4fdbf6847c119b98cb5d4089e38edf Mon Sep 17 00:00:00 2001 From: jonmv Date: Wed, 6 Sep 2023 16:01:53 +0200 Subject: Avoid verbose complaint about missing keys when testing locally --- .../cd/commons/DefaultEndpointAuthenticator.java | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) (limited to 'tenant-cd-commons/src') diff --git a/tenant-cd-commons/src/main/java/ai/vespa/hosted/cd/commons/DefaultEndpointAuthenticator.java b/tenant-cd-commons/src/main/java/ai/vespa/hosted/cd/commons/DefaultEndpointAuthenticator.java index b8030afd901..a39253e00e3 100644 --- a/tenant-cd-commons/src/main/java/ai/vespa/hosted/cd/commons/DefaultEndpointAuthenticator.java +++ b/tenant-cd-commons/src/main/java/ai/vespa/hosted/cd/commons/DefaultEndpointAuthenticator.java @@ -31,8 +31,12 @@ public class DefaultEndpointAuthenticator implements EndpointAuthenticator { private static final Logger logger = Logger.getLogger(DefaultEndpointAuthenticator.class.getName()); + private final boolean hasLocalTestConfig; + /** Don't touch. */ - public DefaultEndpointAuthenticator(@SuppressWarnings("unused") SystemName __) { } + public DefaultEndpointAuthenticator(SystemName system) { + hasLocalTestConfig = system == SystemName.dev; + } /** * If {@code System.getProperty("vespa.test.credentials.root")} is set, key and certificate files @@ -65,12 +69,14 @@ public class DefaultEndpointAuthenticator implements EndpointAuthenticator { PrivateKey privateKey = KeyUtils.fromPemEncodedPrivateKey(new String(Files.readAllBytes(privateKeyFile))); return new SslContextBuilder().withKeyStore(privateKey, certificate).build(); } - logger.warning( "##################################################################################\n" - + "# Data plane key and/or certificate missing; please specify #\n" - + "# '-DdataPlaneCertificateFile=/path/to/certificate' and #\n" - + "# '-DdataPlaneKeyFile=/path/to/private_key'. #\n" - + "# Trying the default SSLContext, but this will most likely cause HTTP error 401. #\n" - + "##################################################################################"); + if ( ! hasLocalTestConfig) + logger.warning(""" + ################################################################################## + # Data plane key and/or certificate missing; please specify # + # '-DdataPlaneCertificateFile=/path/to/certificate' and # + # '-DdataPlaneKeyFile=/path/to/private_key'. # + # Trying the default SSLContext, but this will most likely cause HTTP error 401. # + ##################################################################################"""); return SSLContext.getDefault(); } catch (IOException e) { throw new UncheckedIOException(e); -- cgit v1.2.3