Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Error: ENR has no udp multiaddr #6320

Closed
nflaig opened this issue Jan 17, 2024 · 1 comment
Closed

Error: ENR has no udp multiaddr #6320

nflaig opened this issue Jan 17, 2024 · 1 comment
Assignees
@wemeetagain
Labels
meta-bug Issues that identify a bug and require a fix. meta-investigate Issues found that require further investigation and may not have a specific resolution/fix scope-networking All issues related to networking, gossip, and libp2p.

Comments

@nflaig
Copy link
Member

nflaig commented Jan 17, 2024

Describe the bug

There is an uncaught expection in discv5

Jan-17 09:17:47.140[network]         error: uncaughtException: ENR has no udp multiaddr
Error: ENR has no udp multiaddr
    at getNodeAddress (file:///usr/app/node_modules/@chainsafe/discv5/lib/session/nodeInfo.js:71:23)
    at Discv5.sendRpcRequest (file:///usr/app/node_modules/@chainsafe/discv5/lib/service/service.js:402:26)
    at Discv5.requestEnr (file:///usr/app/node_modules/@chainsafe/discv5/lib/service/service.js:376:14)
    at Discv5.handlePing (file:///usr/app/node_modules/@chainsafe/discv5/lib/service/service.js:613:22)
    at SessionService.handleRpcRequest (file:///usr/app/node_modules/@chainsafe/discv5/lib/service/service.js:597:29)
    at SessionService.emit (node:events:514:28)
    at SessionService.handleMessage (file:///usr/app/node_modules/@chainsafe/discv5/lib/session/service.js:489:18)
    at UDPTransportService.processInboundPacket (file:///usr/app/node_modules/@chainsafe/discv5/lib/session/service.js:234:29)
    at UDPTransportService.emit (node:events:514:28)
    at Socket.handleIncoming (file:///usr/app/node_modules/@chainsafe/discv5/lib/transport/udp.js:121:14) - ENR has no udp multiaddr
Error: ENR has no udp multiaddr
    at getNodeAddress (file:///usr/app/node_modules/@chainsafe/discv5/lib/session/nodeInfo.js:71:23)
    at Discv5.sendRpcRequest (file:///usr/app/node_modules/@chainsafe/discv5/lib/service/service.js:402:26)
    at Discv5.requestEnr (file:///usr/app/node_modules/@chainsafe/discv5/lib/service/service.js:376:14)
    at Discv5.handlePing (file:///usr/app/node_modules/@chainsafe/discv5/lib/service/service.js:613:22)
    at SessionService.handleRpcRequest (file:///usr/app/node_modules/@chainsafe/discv5/lib/service/service.js:597:29)
    at SessionService.emit (node:events:514:28)
    at SessionService.handleMessage (file:///usr/app/node_modules/@chainsafe/discv5/lib/session/service.js:489:18)
    at UDPTransportService.processInboundPacket (file:///usr/app/node_modules/@chainsafe/discv5/lib/session/service.js:234:29)
    at UDPTransportService.emit (node:events:514:28)
    at Socket.handleIncoming (file:///usr/app/node_modules/@chainsafe/discv5/lib/transport/udp.js:121:14)

Expected behavior

No uncaught exceptions, all errors should be handled

Steps to reproduce

Run Lodestar beacon node

Additional context

No response

Operating system

Linux

Lodestar version or commit hash

v1.14.0-rc.1
@nflaig nflaig added the meta-bug Issues that identify a bug and require a fix. label Jan 17, 2024
@wemeetagain wemeetagain self-assigned this Jan 19, 2024
@philknows philknows added scope-networking All issues related to networking, gossip, and libp2p. meta-investigate Issues found that require further investigation and may not have a specific resolution/fix labels Jan 19, 2024
@wemeetagain wemeetagain mentioned this issue Jan 19, 2024
Merged
@nflaig
Copy link
Member Author

nflaig commented Feb 1, 2024

This error rarely happens and I have not confirmed it is resolved but confident that it is fixed by upgrading discv5 in #6380 which includes a fix to improve error handling (ChainSafe/discv5#284).

Will reopen in case the error still happens on latest unstable.

@nflaig nflaig closed this as completed Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wemeetagain wemeetagain

Labels
meta-bug Issues that identify a bug and require a fix. meta-investigate Issues found that require further investigation and may not have a specific resolution/fix scope-networking All issues related to networking, gossip, and libp2p.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@wemeetagain @nflaig @philknows