Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: Use EnableBuffering to keep request payload around #250

Merged
merged 1 commit into from
Jul 24, 2019

Conversation

bruno-garcia
Copy link
Member

@bruno-garcia bruno-garcia commented Jul 24, 2019

Resolves #249

@codecov-io
Copy link

codecov-io commented Jul 24, 2019

Codecov Report

Merging #250 into master will decrease coverage by 0.09%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #250     +/-   ##
=========================================
- Coverage   89.28%   89.18%   -0.1%     
=========================================
  Files          81       81             
  Lines        2035     2035             
=========================================
- Hits         1817     1815      -2     
- Misses        218      220      +2
Impacted Files Coverage Δ
src/Sentry.AspNetCore/SentryMiddleware.cs 98.38% <100%> (ø) ⬆️
src/Sentry/Internal/BackgroundWorker.cs 94.26% <0%> (-1.64%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 16ec03e...520bb78. Read the comment docs.

@bruno-garcia bruno-garcia merged commit f2ddaa1 into master Jul 24, 2019
@bruno-garcia bruno-garcia deleted the fix/aspnetcore-enable-buffering branch July 24, 2019 10:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sentry.AspNetCore don't work on .NET Core 3 preview 7
2 participants