Review

elastic · May 4, 2021 · 71aa0a3 · 71aa0a3
1 parent 4001fdd
commit 71aa0a3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/Elastic.Apm/Filters/ErrorContextSanitizerFilter.cs b/src/Elastic.Apm/Filters/ErrorContextSanitizerFilter.cs
@@ -22,10 +22,9 @@ public IError Filter(IError error)
 			{
 				if (realError.Context.Request?.Headers != null && realError.ConfigSnapshot != null)
 				{
-					foreach (var key in realError.Context?.Request?.Headers?.Keys.ToList())
+					foreach (var key in realError.Context?.Request?.Headers?.Keys)
 					{
-						if (WildcardMatcher.IsAnyMatch(realError.ConfigSnapshot.SanitizeFieldNames, key)
-							&& realError.Context.Request.Headers[key] != Consts.Redacted)
+						if (WildcardMatcher.IsAnyMatch(realError.ConfigSnapshot.SanitizeFieldNames, key))
 							realError.Context.Request.Headers[key] = Consts.Redacted;
 					}
 				}

diff --git a/src/Elastic.Apm/Filters/HeaderDictionarySanitizerFilter.cs b/src/Elastic.Apm/Filters/HeaderDictionarySanitizerFilter.cs
@@ -22,10 +22,9 @@ public ITransaction Filter(ITransaction transaction)
 			{
 				if (realTransaction.IsContextCreated && realTransaction.Context.Request?.Headers != null)
 				{
-					foreach (var key in realTransaction.Context?.Request?.Headers?.Keys.ToList())
+					foreach (var key in realTransaction.Context?.Request?.Headers?.Keys)
 					{
-						if (WildcardMatcher.IsAnyMatch(realTransaction.ConfigSnapshot.SanitizeFieldNames, key)
-							&& realTransaction.Context.Request.Headers[key] != Consts.Redacted)
+						if (WildcardMatcher.IsAnyMatch(realTransaction.ConfigSnapshot.SanitizeFieldNames, key))
 							realTransaction.Context.Request.Headers[key] = Consts.Redacted;
 					}
 				}