From 1bd30955b37acce2e81ccc871c7642ebf4344e03 Mon Sep 17 00:00:00 2001 From: Rens Groothuijsen Date: Sun, 31 Jan 2021 19:56:36 +0100 Subject: [PATCH] Log missing environment variables warning to TaskListener instead of system log --- .../plugins/envinject/EnvInjectEnvVarsContributor.java | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/main/java/org/jenkinsci/plugins/envinject/EnvInjectEnvVarsContributor.java b/src/main/java/org/jenkinsci/plugins/envinject/EnvInjectEnvVarsContributor.java index a7dd522e..7e9729d1 100644 --- a/src/main/java/org/jenkinsci/plugins/envinject/EnvInjectEnvVarsContributor.java +++ b/src/main/java/org/jenkinsci/plugins/envinject/EnvInjectEnvVarsContributor.java @@ -6,7 +6,6 @@ import java.io.IOException; import java.util.Map; -import java.util.logging.Logger; /** * @since 1.92 @@ -14,8 +13,6 @@ @Extension public class EnvInjectEnvVarsContributor extends EnvironmentContributor { - private final static Logger LOGGER = Logger.getLogger(EnvInjectEnvVarsContributor.class.getName()); - @Override @SuppressWarnings("rawtypes") public void buildEnvironmentFor(Job job, EnvVars env, TaskListener listener) throws IOException, InterruptedException { @@ -30,7 +27,7 @@ public void buildEnvironmentFor(Job job, EnvVars env, TaskListener listener) thr int expectedEnvSize = env.size() + result.size(); env.putAll(result); if (env.size() != expectedEnvSize) { - LOGGER.warning("Not all environment variables could be successfully injected. " + + listener.error("Not all environment variables could be successfully injected. " + "Check for similarly-named environment variables."); } }