Fix issue with TokenCredential (#1261)

Fix an issue where we forgot to set the SymbolTokenCredential property. Additionally make this an optional paramter.
microsoft · Jun 6, 2024 · 34c4600 · 34c4600
1 parent 135c7e8
commit 34c4600
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/Microsoft.Diagnostics.Runtime/CustomDataTarget.cs b/src/Microsoft.Diagnostics.Runtime/CustomDataTarget.cs
@@ -44,9 +44,10 @@ public class CustomDataTarget : IDisposable
         /// </summary>
         /// <param name="reader">A non-null IDataReader.</param>
         /// <param name="symbolCredential">The TokenCredential to use for any Azure based symbol servers (set to null if not using one).</param>
-        public CustomDataTarget(IDataReader reader, TokenCredential? symbolCredential)
+        public CustomDataTarget(IDataReader reader, TokenCredential? symbolCredential = null)
         {
             DataReader = reader ?? throw new ArgumentNullException(nameof(reader));
+            SymbolTokenCredential = symbolCredential;
         }
 
         /// <summary>