Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[JENKINS-27363] - Annotate sensibleVariables (may be null)
Patch to the core: jenkinsci/jenkins#1601

Signed-off-by: Oleg Nenashev <o.v.nenashev@gmail.com>
  • Loading branch information
oleg-nenashev committed Mar 11, 2015
1 parent 0033cdf commit d70408a
Showing 1 changed file with 11 additions and 6 deletions.
17 changes: 11 additions & 6 deletions src/main/java/org/jenkinsci/lib/envinject/EnvInjectAction.java
Expand Up @@ -13,6 +13,7 @@
import java.util.Collections;
import java.util.Map;
import java.util.Set;
import javax.annotation.CheckForNull;

/**
* @author Gregory Boissinot
Expand All @@ -21,7 +22,7 @@ public class EnvInjectAction implements Action, StaplerProxy {

public static final String URL_NAME = "injectedEnvVars";

protected transient Map<String, String> envMap;
protected transient @CheckForNull Map<String, String> envMap;

private AbstractBuild build;

Expand All @@ -30,9 +31,10 @@ public class EnvInjectAction implements Action, StaplerProxy {
*/
private transient Map<String, String> resultVariables;
private transient File rootDir;
private transient Set<String> sensibleVariables;
private transient @CheckForNull Set<String> sensibleVariables;

public EnvInjectAction(AbstractBuild build, Map<String, String> envMap) {
public EnvInjectAction(AbstractBuild build,
@CheckForNull Map<String, String> envMap) {
this.build = build;
this.envMap = envMap;
}
Expand All @@ -41,7 +43,9 @@ public void overrideAll(Map<String, String> all) {
overrideAll(Collections.<String>emptySet(), all);
}

public void overrideAll(final Set<String> sensibleVariables, Map<String, String> all) {
public void overrideAll(
final @CheckForNull Set<String> sensibleVariables,
@CheckForNull Map<String, String> all) {
if (envMap == null) {
return;
}
Expand Down Expand Up @@ -91,7 +95,8 @@ private Object writeReplace() throws ObjectStreamException {
dao.saveEnvironment(build.getRootDir(), Maps.transformEntries(envMap,
new Maps.EntryTransformer<String, String, String>() {
public String transformEntry(String key, String value) {
return sensibleVariables.contains(key) ? "********" : value;
return (sensibleVariables != null && sensibleVariables.contains(key))
? "********" : value;
}
}));
return this;
Expand Down Expand Up @@ -155,7 +160,7 @@ public Object getTarget() {
throw new UnsupportedOperationException();
}

public Set<String> getSensibleVariables() {
public @CheckForNull Set<String> getSensibleVariables() {
return sensibleVariables;
}
}

0 comments on commit d70408a

Please sign in to comment.