Skip to content

Commit

Permalink
fix serialization of strings in properties
Browse files Browse the repository at this point in the history
  • Loading branch information
smokedlinq committed Jul 21, 2022
1 parent 6f87abe commit 764ce66
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
1 change: 0 additions & 1 deletion .dccache

This file was deleted.

3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
.dccache

# Created by https://www.toptal.com/developers/gitignore/api/visualstudio,visualstudiocode
# Edit at https://www.toptal.com/developers/gitignore?templates=visualstudio,visualstudiocode
Expand Down Expand Up @@ -370,4 +371,4 @@ MigrationBackup/
# Ionide (cross platform F# VS Code tools) working folder
.ionide/

# End of https://www.toptal.com/developers/gitignore/api/visualstudio,visualstudiocode
# End of https://www.toptal.com/developers/gitignore/api/visualstudio,visualstudiocode
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,10 @@ private void TrackAvailability(KeyValuePair<string, HealthReportEntry> entry)
properties.Add($"{data.Key}:{dataEntry.Key}", dataEntry.Value.Status.ToString());
}
}
else if (data.Value is string value)
{
properties.Add(data.Key, value);
}
else if (data.Value is IEnumerable enumerable)
{
properties.Add(data.Key, JsonSerializer.Serialize(enumerable));
Expand Down

0 comments on commit 764ce66

Please sign in to comment.