Skip to content

add proposer preferences related api specifications#593

Open
bharath-123 wants to merge 1 commit intoethereum:masterfrom
bharath-123:proposer-preferences-spec
Open

add proposer preferences related api specifications#593
bharath-123 wants to merge 1 commit intoethereum:masterfrom
bharath-123:proposer-preferences-spec

Conversation

@bharath-123
Copy link
Copy Markdown
Contributor

This PR partially resolves #570

This PR adds the following:

  1. Adds the proposer preferences type in the beacon api specs
  2. Adds the proposer preferences SSE event api spec
  3. Adds the proposer preferences pool api to get pooled preferences and to allow users to submit preferences.

@nflaig nflaig added the Gloas api's needed in Gloas fork. label Mar 24, 2026
$ref: "../../../beacon-node-oapi.yaml#/components/responses/InternalError"

post:
operationId: submitSignedProposerPreferences
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

i think this should be under the validator client folder not pool

description: The node has received a `SignedProposerPreferences` (from P2P or API) that passes gossip validation on the `proposer_preferences` topic
value: |
event: proposer_preferences
data: {"version":"gloas", "data":{"message": {"proposal_slot": "32", "validator_index": "123", "fee_recipient": "0x0000000000000000000000000000000000000000", "gas_limit": "30000000"}, "signature": "0x1b66ac1fb663c9bc59509846d6ec05345bd908eda73e670af888da41af171505cc411d61252fb6cb3fa0017b679f8bb2305b26a285fa2737f175668d0dff91cc1b66ac1fb663c9bc59509846d6ec05345bd908eda73e670af888da41af171505"}}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

you want an event for every single proposer preference? not like an array of everything that was sent?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

guess we can go with this for now and change it if it's too noisy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Gloas api's needed in Gloas fork.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add spec for proposer preferences

3 participants