Skip to content

Commit 8059366

Browse files
authored
Fix fromjson serialization of breadcrumbs from android (#3993)
* Fix fromjson serialization of breadcrumbs from android * Update CHANGELOG.md
1 parent 2a798a8 commit 8059366

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22

33
## Unreleased
44

5+
- Fix mismapped breadcrumb levels coming in from native to dotnet SDK ([#3993](https://github.com/getsentry/sentry-dotnet/pull/3993))
6+
57
### Features
68

79
- Serilog scope properties are now sent with Sentry events ([#3976](https://github.com/getsentry/sentry-dotnet/pull/3976))

src/Sentry/Breadcrumb.cs

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,8 +157,18 @@ public static Breadcrumb FromJson(JsonElement json)
157157
var type = json.GetPropertyOrNull("type")?.GetString();
158158
var data = json.GetPropertyOrNull("data")?.GetStringDictionaryOrNull();
159159
var category = json.GetPropertyOrNull("category")?.GetString();
160-
var level = json.GetPropertyOrNull("level")?.GetString()?.ParseEnum<BreadcrumbLevel>() ?? default;
161160

161+
var levelString = json.GetPropertyOrNull("level")?.GetString();
162+
var level = levelString?.ToUpper() switch
163+
{
164+
"DEBUG" => BreadcrumbLevel.Debug,
165+
"INFO" => BreadcrumbLevel.Info,
166+
"WARNING" => BreadcrumbLevel.Warning,
167+
"ERROR" => BreadcrumbLevel.Error,
168+
"CRITICAL" => BreadcrumbLevel.Critical,
169+
"FATAL" => BreadcrumbLevel.Critical,
170+
_ => default
171+
};
162172
return new Breadcrumb(timestamp, message, type, data!, category, level);
163173
}
164174
}

0 commit comments

Comments
 (0)