Added Suppression for CA5394 (insecure randomness) in RandomStringPatternConverter

FreeAndNil · FreeAndNil · commit c40eb18d102a · 2024-10-22T16:37:51.000+02:00
diff --git a/src/log4net/Util/PatternStringConverters/RandomStringPatternConverter.cs b/src/log4net/Util/PatternStringConverters/RandomStringPatternConverter.cs
@@ -90,6 +90,8 @@ public void ActivateOptions()
   /// </summary>
   /// <param name="writer">the writer to write to</param>
   /// <param name="state">null, state is not set</param>
+  [System.Diagnostics.CodeAnalysis.SuppressMessage("Security", "CA5394:Do not use insecure randomness", 
+    Justification = "Random value does not need to be cryptographically secure")]
   public override void Convert(TextWriter writer, object? state)
   {
     try

Original file line number	Diff line number	Diff line change
`@@ -90,6 +90,8 @@ public void ActivateOptions()`
`90`	`90`	`/// </summary>`
`91`	`91`	`/// <param name="writer">the writer to write to</param>`
`92`	`92`	`/// <param name="state">null, state is not set</param>`
	`93`	`+ [System.Diagnostics.CodeAnalysis.SuppressMessage("Security", "CA5394:Do not use insecure randomness",`
	`94`	`+ Justification = "Random value does not need to be cryptographically secure")]`
`93`	`95`	`public override void Convert(TextWriter writer, object? state)`
`94`	`96`	`{`
`95`	`97`	`try`