Skip to content

LokiTarget - Expect100Continue for HttpClient#158

Merged
corentinaltepe merged 1 commit intocorentinaltepe:masterfrom
snakefoot:Expect100Continue
Mar 26, 2026
Merged

LokiTarget - Expect100Continue for HttpClient#158
corentinaltepe merged 1 commit intocorentinaltepe:masterfrom
snakefoot:Expect100Continue

Conversation

@snakefoot
Copy link
Copy Markdown

@snakefoot snakefoot commented Mar 26, 2026

.NET Framework has Expect100Continue = true by default, so it is mostly an optimization for legacy platform.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 26, 2026

Codecov Report

❌ Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.10%. Comparing base (b1c346b) to head (4ad01dc).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
src/NLog.Loki/LokiTarget.cs 33.33% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
- Coverage   76.42%   76.10%   -0.32%     
==========================================
  Files          12       12              
  Lines         403      406       +3     
  Branches       57       58       +1     
==========================================
+ Hits          308      309       +1     
- Misses         69       70       +1     
- Partials       26       27       +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.

@snakefoot snakefoot force-pushed the Expect100Continue branch 2 times, most recently from 9ca1d28 to 4ad01dc Compare March 26, 2026 19:28
@corentinaltepe corentinaltepe merged commit 28e721b into corentinaltepe:master Mar 26, 2026
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants