-
-
Notifications
You must be signed in to change notification settings - Fork 372
Closed
Labels
Breaking-changeshould go in a major release (breaks apps, changes default configs in a major way)should go in a major release (breaks apps, changes default configs in a major way)BugCocoa
Description
Description
According to our spec in the develop docs the http.response.status_code must be an int, but we send it as a string.
sentry-cocoa/Sources/Sentry/SentryNetworkTracker.m
Lines 360 to 361 in 1a4afd3
| [netSpan setDataValue:[NSString stringWithFormat:@"%@", statusCode] | |
| forKey:@"http.response.status_code"]; |
Maybe worth double checking if we have other places in the SDK, that use NSString as well for status_code.
Metadata
Metadata
Assignees
Labels
Breaking-changeshould go in a major release (breaks apps, changes default configs in a major way)should go in a major release (breaks apps, changes default configs in a major way)BugCocoa