Merge pull request #232 from Microsoft/users/angoya/regex

Minor fix for masking regex from logs. We were filtering the regex based on the variables. Removing that check
microsoft · Feb 26, 2016 · 9593218 · 9593218
2 parents e99df48 + 5ac3e4e
commit 9593218
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/agent/common.ts b/src/agent/common.ts
@@ -430,11 +430,14 @@ function createMaskFunction(jobEnvironment: agentifm.JobEnvironment): Replacemen
     var envMasks = jobEnvironment.mask || [];
     var maskHints = [];
     envMasks.forEach((maskHint: agentifm.MaskHint) => {
-        if ((maskHint.type === agentifm.MaskType.Variable || maskHint.type === agentifm.MaskType.Regex) && maskHint.value) {
+        if (maskHint.type === agentifm.MaskType.Variable && maskHint.value) {
             if (jobEnvironment.variables[maskHint.value]) {
                 maskHints.push(maskHint);
             }
         }
+        else if (maskHint.type === agentifm.MaskType.Regex && maskHint.value) {
+            maskHints.push(maskHint);
+        }
     });
 
     if (maskHints.length > 0) {