Skip to content

Conversation

@zilm13
Copy link
Contributor

@zilm13 zilm13 commented Dec 20, 2025

PR Description

Fixed Issue(s)

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

Note

Adds optional User-Agent and standard Accept/consensus-version headers to builder REST calls and updates RestClient.getAsync to accept headers.

  • Builder client (RestBuilderClient):
    • Introduces buildHeadersWith(...) to construct headers (optional User-Agent, Accept, and Eth-Consensus-Version).
    • Applies headers to status, registerValidators (SSZ/JSON), getHeader, getPayload, and getPayloadV2 requests.
  • REST client API:
    • Changes RestClient.getAsync signature to require headers and updates OkHttpRestClient implementation accordingly.
  • Tests:
    • Update OkHttpRestClientTest to pass headers for GET calls.
    • Update RestBuilderClientTest to reflect new postAsync signature and verify presence/absence of User-Agent and other headers.

Written by Cursor Bugbot for commit ca75078. This will update automatically on new commits. Configure here.

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