search

getsentry / team-sdks

A meta repository for tracking work across all SDK teams.
0 stars 0 forks source link

Set HTTP client breadcrumbs log level based on response status code #100

Open cleptric opened 1 month ago

cleptric commented 1 month ago

Description

We should align the log level for HTTP client requests and make sure we set appropriate levels based on the response status code.

5xx should be error 4xx should be warning

Stakeholder(s)

@lforst @cleptric

SDKs

## SDKs
- [ ] JavaScript SDK https://github.com/getsentry/sentry-javascript/issues/13359
- [ ] Laravel SDK https://github.com/getsentry/sentry-laravel/pull/940
- [ ] PHP SDK https://github.com/getsentry/sentry-php/pull/1773
- [ ] Electron SDK
- [ ] Python SDK
- [ ] Symfony SDK
- [ ] Go SDK
- [ ] Ruby SDK
- [ ] Elixir SDK
- [ ] Rust SDK
- [ ] Java SDK
- [ ] .NET SDK
- [ ] Unity SDK
- [ ] Unreal SDK
- [ ] PowerShell SDK
- [ ] Android SDK
- [ ] Apple/Cocoa SDK
- [ ] Capacitor SDK
- [ ] Cordova SDK
- [ ] Dart SDK
- [ ] Flutter SDK
- [ ] Kotlin Multiplatform SDK
- [ ] React Native SDK
- [ ] Native SDK
- [ ] Sentry CLI
lforst commented 1 month ago

@cleptric would it make sense to put 4xx as error too? Is there a deeper thought putting it as warn?