From 34c46005c396800f72db703e66baff7d67a73bac Mon Sep 17 00:00:00 2001
From: Lee Culver <leculver@microsoft.com>
Date: Thu, 6 Jun 2024 13:53:38 -0700
Subject: [PATCH] Fix issue with TokenCredential (#1261)

Fix an issue where we forgot to set the SymbolTokenCredential property.  Additionally make this an optional paramter.
---
 src/Microsoft.Diagnostics.Runtime/CustomDataTarget.cs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/Microsoft.Diagnostics.Runtime/CustomDataTarget.cs b/src/Microsoft.Diagnostics.Runtime/CustomDataTarget.cs
index 14cb34c98..44a5f9293 100644
--- 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>