Skip to content

fix: incorrect handling http 103 - #280

Merged
rustatian merged 3 commits into
masterfrom
fix/incorrect-101-frames
Aug 21, 2026
Merged

fix: incorrect handling http 103#280
rustatian merged 3 commits into
masterfrom
fix/incorrect-101-frames

Conversation

@rustatian

Copy link
Copy Markdown
Member

Reason for This PR

closes: roadrunner-server/roadrunner#2381

Description of Changes

  • Do not apply any status in log's middleware ResponseWriter interceptor.

License Acceptance

By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.

PR Checklist

[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

  • All commits in this PR are signed (git commit -s).
  • The reason for this PR is clearly provided (issue no. or explanation).
  • The description of changes is clear and encompassing.
  • Any required documentation changes (code and docs) are included in this PR.
  • Any user-facing changes are mentioned in CHANGELOG.md.
  • All added/changed functionality is tested.

Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
@rustatian rustatian self-assigned this Aug 21, 2026
Copilot AI lite review requested due to automatic review settings August 21, 2026 05:57
@rustatian rustatian added the bug Something isn't working label Aug 21, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.61%. Comparing base (6ec48a3) to head (f5d191d).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #280      +/-   ##
==========================================
+ Coverage   87.38%   87.61%   +0.23%     
==========================================
  Files          28       28              
  Lines        1268     1292      +24     
==========================================
+ Hits         1108     1132      +24     
  Misses        160      160              

☔ View full report in Codecov by Harness.
📢 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.

@rustatian
rustatian merged commit d13c0b9 into master Aug 21, 2026
8 checks passed
@rustatian
rustatian deleted the fix/incorrect-101-frames branch August 21, 2026 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🐛 BUG]: 103 Early Hints frame replaces the final response status with 200

2 participants