Skip to content

feat: proxyd update consensus mode out of range requests to be api-spec compliant#552

Merged
jelias2 merged 2 commits intomainfrom
seb/fix-block-out-of-range-do-not-forward
Feb 12, 2026
Merged

feat: proxyd update consensus mode out of range requests to be api-spec compliant#552
jelias2 merged 2 commits intomainfrom
seb/fix-block-out-of-range-do-not-forward

Conversation

@jelias2
Copy link
Contributor

@jelias2 jelias2 commented Feb 12, 2026

Description

  • In consensus mode proxyd if requests were out of range for the consensus backend they would return an http 400 status code and error message. This PR updates proxyd to be json-rpc spec complaint by returning a compatible response to the client. This should ensure the client can properly function regardless of proxyd intercepting requests or not

Tests

Please describe any tests you've added. If you've added no tests, or left important behavior untested, please explain why not.

Additional context

Add any other context about the problem you're solving.

Metadata

  • Fixes #[Link to Issue]

@jelias2 jelias2 requested a review from a team as a code owner February 12, 2026 15:40
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 47.61905% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.59%. Comparing base (f9bf450) to head (a8a0851).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
proxyd/backend.go 47.61% 11 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #552      +/-   ##
==========================================
- Coverage   57.61%   57.59%   -0.02%     
==========================================
  Files          94       94              
  Lines       14204    14224      +20     
==========================================
+ Hits         8184     8193       +9     
- Misses       5522     5534      +12     
+ Partials      498      497       -1     
Flag Coverage Δ
proxyd 71.22% <47.61%> (-0.11%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
proxyd/backend.go 79.26% <47.61%> (-0.57%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@geoknee
Copy link
Contributor

geoknee commented Feb 12, 2026

replaces #547

@jelias2 jelias2 merged commit 9e3b5d0 into main Feb 12, 2026
61 checks passed
@jelias2 jelias2 deleted the seb/fix-block-out-of-range-do-not-forward branch February 12, 2026 21:29
@sebastianst sebastianst linked an issue Feb 13, 2026 that may be closed by this pull request
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.

proxyd: Wrongly returns HTTP and JSON error if block not found

5 participants