Skip to content

Improve SentryHttpFailedRequestHandler issue grouping #4724

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jamescrosswell merged 21 commits into from
Nov 19, 2025
Merged

Improve SentryHttpFailedRequestHandler issue grouping #4724

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
19c0b76
Add new HttpStatusCodeRangeExtensions and tests.
ericsampson Nov 11, 2025
8b3ea27
Improve tests.
ericsampson Nov 11, 2025
56e3bf8
Add test that currently fails to prove the fix works.
ericsampson Nov 11, 2025
b97a6bd
Enhance the HttpFailedRequestHandler so that it captures a full excep…
ericsampson Nov 11, 2025
fd65e67
Merge branch 'main' into SentryHttpMessageHandler-issue-grouping
ericsampson Nov 17, 2025
02e69b1
Delete tests for deleted functionality.
ericsampson Nov 17, 2025
c9da3c7
Update test/Sentry.Tests/HttpStatusCodeRangeExtensionsTests.cs
ericsampson Nov 17, 2025
a07ed5d
Switch to using ExceptionDispatchInfo.SetCurrentStackTrace
ericsampson Nov 17, 2025
c4b981d
Update re PR feedback
ericsampson Nov 17, 2025
ced8585
Update with PR feedback
ericsampson Nov 17, 2025
7a1573a
PR feedback
ericsampson Nov 17, 2025
02a16b4
PR feedback
ericsampson Nov 17, 2025
77132f5
Add note to changelog
ericsampson Nov 17, 2025
46bcc41
Update src/Sentry/SentryHttpFailedRequestHandler.cs
ericsampson Nov 17, 2025
70232c3
Update src/Sentry/SentryHttpFailedRequestHandler.cs re method availab…
ericsampson Nov 18, 2025
7a90dac
format test/Sentry.Tests/SentryHttpFailedRequestHandlerTests.cs
ericsampson Nov 18, 2025
f8edf37
Update test for running on .NET5+
ericsampson Nov 18, 2025
e3fc2bc
Merge branch 'main' into SentryHttpMessageHandler-issue-grouping
ericsampson Nov 18, 2025
c76afd1
update Changelog re PR feedback
ericsampson Nov 18, 2025
885d728
Update CHANGELOG.md
ericsampson Nov 19, 2025
3901dd0
Merge branch 'main' into SentryHttpMessageHandler-issue-grouping
ericsampson Nov 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
format test/Sentry.Tests/SentryHttpFailedRequestHandlerTests.cs 
Co-authored-by: Stefan Pölz <[email protected]>
  • Loading branch information
@ericsampson @Flash0ver
ericsampson and Flash0ver authored Nov 18, 2025
commit 7a90dac1c8993fd9d39ace948ac6b43c9e6e84ed
2 changes: 1 addition & 1 deletion test/Sentry.Tests/SentryHttpFailedRequestHandlerTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -337,7 +337,7 @@ public void HandleResponse_FailedRequestWithHint_HasHint()

// Act
SentryHint hint = null;
_hub.CaptureEvent( Arg.Any<SentryEvent>(), hint: Arg.Do<SentryHint>(h => hint = h));
_hub.CaptureEvent(Arg.Any<SentryEvent>(), hint: Arg.Do<SentryHint>(h => hint = h));
sut.HandleResponse(response);

// Assert
Expand Down