Skip to content

fix: bind consensus discovery to published UDP port - #1178

Open
zakazaka95 wants to merge 2 commits into
base:mainfrom
zakazaka95:fix-1177-consensus-udp-port
Open

fix: bind consensus discovery to published UDP port#1178
zakazaka95 wants to merge 2 commits into
base:mainfrom
zakazaka95:fix-1177-consensus-udp-port

Conversation

@zakazaka95

Copy link
Copy Markdown

Summary

  • configure the consensus UDP listen port as 9222 for both mainnet and Sepolia
  • align the listener with the Docker 9222:9222/udp mapping and advertised UDP port
  • avoid fresh nodes silently binding discovery to a different default port

Fixes #1177

Testing

  • verified both network env files use UDP port 9222 consistently
  • verified the compose configuration publishes UDP 9222
  • git diff --check

Docker Compose runtime validation was not available in the local environment.

Align the base-consensus UDP listen port with the port exposed and advertised by Docker.
Apply the same explicit UDP listen port to the Sepolia configuration.
@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

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.

docker-compose.yml missing UDP/9200 mapping causes silent 0-peers on fresh installs (discv5 discovery blocked)

2 participants