fix: Log Warning instead of Error when ratelimited

[bitsandfoxes]

Surfaced through getsentry/sentry-unity#2538

This PR addresses an inconsistency in expected, recoverable behaviour that has - as per log message - no effect on the user's application. Same as dropping events with too large attachments.

[github-actions]

Semver Impact of This PR

⚪ None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

---

Fixes 🐛

• fix: Log Warning instead of Error when ratelimited by bitsandfoxes in #4927

• fix: Log Warning instead of Error when ratelimited by bitsandfoxes in #4927

Dependencies ⬆️

Deps

• chore(deps): update Native SDK to v0.12.8 by github-actions in #4929
• Apps built using the Sentry SDK for .NET must now target iOS version 15 or higher. Previously only version 13 or higher was required. (#4781) by github-actions in #4781
• Bump Cocoa SDK from v8.57.3 to v9.2.0 (#4781) by github-actions in #4781
  • changelog
  • diff
• chore(deps): update Native SDK to v0.12.7 by github-actions in #4920

Other

• test(blazor): Add Playwright E2E tests for navigation breadcrumbs by bruno-garcia in #4908
• test(android): Use volatile to produce SIGSEGV in native crash test by jpnurmi in #4919

---

_{🤖 This preview updates automatically when you update the PR.}

[codecov]

Codecov Report

❌ Patch coverage is 86.36364% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.88%. Comparing base (7c80a90) to head (d31988a).
⚠️ Report is 5 commits behind head on main.

Files with missing lines	Patch %	Lines
src/Sentry/Http/HttpTransportBase.cs	86.36%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4927      +/-   ##
==========================================
+ Coverage   73.86%   73.88%   +0.01%     
==========================================
  Files         496      496              
  Lines       17927    17947      +20     
  Branches     3511     3516       +5     
==========================================
+ Hits        13242    13260      +18     
- Misses       3824     3825       +1     
- Partials      861      862       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jamescrosswell]

Is this consistent with what the other SDKs do for 429 response codes?

Arguably the user's tests could use mocks rather than a real Sentry instance... so I'm not sure if we want to change the SDK here to make one user's tests pass.

[bitsandfoxes]

change the SDK here to make one user's tests pass

This is a red herring. It's not about user tests, they are just surfacing an inconsistency. The other SDKs are also all over the place, checking a few: Python and Dart logs a warning, Cocoa debug, Java does it like .NET right now, warns on dropped attachments and errors on 429.

My argument is still: If this is handled and expected behaviour, why is this an error.

[jamescrosswell]

Looks fine.

btw: You shouldn't need to explicitly add a changelog anymore... these get generated automatically from the PR descriptions now (for future reference).