Skip to content

Reject malformed Beacon blob hash filters - #77

Draft
0x00101010 wants to merge 3 commits into
fix/ci-dependencies-fixturesfrom
fix/beacon-hash-filters
Draft

0x00101010 wants to merge 3 commits into
fix/ci-dependencies-fixturesfrom
fix/beacon-hash-filters

Conversation

@0x00101010

Copy link
Copy Markdown

Why

Invalid blob hashes were silently discarded, broadening filtered requests. Reject the entire malformed filter instead.

What

  • Return HTTP 400 for empty, malformed, or partly invalid hash filters.
  • Preserve valid filtering and requests without filters.

Notes

  • Implementation, tests, and review assisted by Amp.

Invalid hashes were dropped, so a bad filter returned more blobs than requested or every blob in the block.

Amp-Thread-ID: https://ampcode.com/threads/T-01a10d69-00b6-7378-855f-af2b1a148692
Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: T-01a10d69-00b6-7378-855f-af2b1a148692
Amp-Thread-ID: T-01a10d69-00b6-7378-855f-af2b1a148692
@0x00101010
0x00101010 changed the base branch from base-anvil-fork to fix/ci-dependencies-fixtures October 6, 2026 04:26
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.

1 participant