Skip to content

Ignore instead of reject on preferences - #5429

Merged
jtraglia merged 3 commits into
ethereum:masterfrom
potuz:ignore_on_preferences
Jul 6, 2026
Merged

Ignore instead of reject on preferences#5429
jtraglia merged 3 commits into
ethereum:masterfrom
potuz:ignore_on_preferences

Conversation

@potuz

@potuz potuz commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

This is a counter PR to #5428.

Proposer preferences are not checked for equivocations. A proposer that equivocates on the proposer preferences could in principle cause honest peer downscoring by equivocating and making them forward invalid bids over P2P. We should not reject and downscore for non-provable malicious behavior.

@jtraglia
jtraglia enabled auto-merge (squash) July 6, 2026 14:45
@jtraglia
jtraglia merged commit 853c585 into ethereum:master Jul 6, 2026
15 checks passed
ensi321 added a commit to ChainSafe/lodestar that referenced this pull request Jul 7, 2026
ethereum/consensus-specs#5429

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
pull Bot pushed a commit to All-Blockchains/prysm that referenced this pull request Jul 8, 2026
…17128)

- Downgrades the bid gossip `fee_recipient` check from REJECT to IGNORE
per ethereum/consensus-specs#5429, proposer
preferences are not checked for equivocations so a mismatch is not
provably malicious
ensi321 added a commit to ChainSafe/lodestar that referenced this pull request Jul 21, 2026
ethereum/consensus-specs#5429

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants