Make it work using comma separated string

Author: sandermvanvliet

src/Serilog.Enrichers.Sensitive/MaskPropertyCollection.cs

@@ -1,8 +1,9 @@
-using System.Collections.Generic;
+using System.Collections;
+using System.Collections.Generic;
 
 namespace Serilog.Enrichers.Sensitive;
 
-public class MaskPropertyCollection : List<MaskProperty>
+public class MaskPropertyCollection : IEnumerable<string>
 {
     private readonly Dictionary<string, MaskOptions> _properties = new();
 
@@ -32,4 +33,14 @@ public static MaskPropertyCollection From(IEnumerable<string> enricherOptionsMas
 
         return collection;
     }
+
+    public IEnumerator<string> GetEnumerator()
+    {
+        return _properties.Keys.GetEnumerator();
+    }
+
+    IEnumerator IEnumerable.GetEnumerator()
+    {
+        return GetEnumerator();
+    }
 }

src/Serilog.Enrichers.Sensitive/SensitiveDataEnricher.cs

@@ -33,7 +33,7 @@ public SensitiveDataEnricher(
                 MaskingMode.Globally, 
                 DefaultMaskValue, 
                 DefaultOperators.Select(o => o.GetType().AssemblyQualifiedName),
-                new List<string>(),
+                _maskProperties != null ? string.Join(",", _maskProperties) : null,
                 new List<string>());
 
             if (options != null)

src/Serilog.Enrichers.Sensitive/SensitiveDataEnricherOptions.cs

@@ -18,15 +18,15 @@ public SensitiveDataEnricherOptions(
             MaskingMode mode = MaskingMode.Globally, 
             string maskValue = SensitiveDataEnricher.DefaultMaskValue, 
             IEnumerable<string>? maskingOperators = null,
-            IEnumerable<string>? maskProperties = null, 
+            string? maskProperties = null, 
             IEnumerable<string>? excludeProperties = null,
             // ReSharper disable once UnusedParameter.Local as this only exists to support JSON configuration, see the Operators property below 
             IEnumerable<string>? operators = null)
         {
             Mode = mode;
             MaskValue = maskValue;
             MaskingOperators = maskingOperators == null ? new List<IMaskingOperator>() : ResolveMaskingOperators(maskingOperators);
-            MaskProperties = maskProperties == null ? new MaskPropertyCollection() : MaskPropertyCollection.From(maskProperties);
+            MaskProperties = maskProperties == null ? new MaskPropertyCollection() : MaskPropertyCollection.From(maskProperties?.Split(',') ?? []);
             ExcludeProperties = excludeProperties?.ToList() ?? new List<string>();
         }
 
@@ -130,19 +130,4 @@ public void Apply(SensitiveDataEnricherOptions other)
             other.Operators = Operators;
         }
     }
-
-    public class MaskProperty
-    {
-        public MaskProperty()
-        {
-        }
-
-        public MaskProperty(string propertyName)
-        {
-            Name = propertyName;
-        }
-
-        public string Name { get; set; }
-        public MaskOptions Options { get; set; } = MaskOptions.Default;
-    }
 }

test/Serilog.Enrichers.Sensitive.Tests.Unit/enricher-config.json

@@ -9,9 +9,7 @@
         "Args": {
           "options": {
             "MaskValue": "**SECRET**",
-            "MaskProperties": [
-              "secret"
-            ]
+            "MaskProperties": "secret"
           }
         }
       }