fix: Cast search attributes to string for proper tagging in activity

hangy · hangy · commit bf004d8cba74 · 2025-09-04T00:48:39.000+02:00
diff --git a/src/AlbusKavaliro.TraceableLdapClient/TraceableLdapConnection.cs b/src/AlbusKavaliro.TraceableLdapClient/TraceableLdapConnection.cs
@@ -355,7 +355,7 @@ protected virtual void Dispose(bool disposing)
             activity.SetTag("ldap.search.base", sr.DistinguishedName);
             activity.SetTag("ldap.search.scope", sr.Scope.ToString());
             activity.SetTag("ldap.search.filter", sr.Filter);
-            activity.SetTag("ldap.search.attributes", string.Join(',', sr.Attributes));
+            activity.SetTag("ldap.search.attributes", string.Join(',', sr.Attributes.Cast<string>()));
             activity.SetTag("ldap.search.size_limit", sr.SizeLimit);
             activity.SetTag("ldap.search.time_limit", sr.TimeLimit);
         }

Original file line number	Diff line number	Diff line change
`@@ -355,7 +355,7 @@ protected virtual void Dispose(bool disposing)`
`355`	`355`	`activity.SetTag("ldap.search.base", sr.DistinguishedName);`
`356`	`356`	`activity.SetTag("ldap.search.scope", sr.Scope.ToString());`
`357`	`357`	`activity.SetTag("ldap.search.filter", sr.Filter);`
`358`		`- activity.SetTag("ldap.search.attributes", string.Join(',', sr.Attributes));`
	`358`	`+ activity.SetTag("ldap.search.attributes", string.Join(',', sr.Attributes.Cast<string>()));`
`359`	`359`	`activity.SetTag("ldap.search.size_limit", sr.SizeLimit);`
`360`	`360`	`activity.SetTag("ldap.search.time_limit", sr.TimeLimit);`
`361`	`361`	`}`