-
Notifications
You must be signed in to change notification settings - Fork 746
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.
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
Panic in UPnP search #4171
Comments
also on 4d17fb3 |
Which OS? Any virtualisation or containerisation in play? Looks like a weird error from the OS being unable to do UPnP. You can probably work around it with |
Thanks, works with |
No virtualization or containers |
Our UPnP support is best-effort due to the vast number of routers and networking setups that are possible, but I think we could avoid panicking in this case (a clean failure would be better). Which router do you have, if you don't mind me asking? You can also DM the answer on Discord, I'm |
In terms of a fix, looks like maybe we'd have to go to the |
I don't know it was on public wi-fi. I will try to ask owner of router in few days |
I had a look at the |
I have filed a PR to rust-igd. sbstp/rust-igd#56 |
Sorry, I can't reproduce it anymore. Just prints |
## Issue Addressed #4171 ## Proposed Changes Through [this PR](sbstp/rust-igd#56) in rust-igd, `igd` v0.12.1 no longer panics if there is an issue while searching for a gateway. So updating igd makes lighthouse emit a helpful log instead of panicking. ## Additional Info No CHANGELOG exists in rust-igd. 👀 Here is the commit history between v0.11.1 and v0.12.1. No breaking changes. sbstp/rust-igd@v0.11.1...v0.12.1
## Issue Addressed #4171 ## Proposed Changes Through [this PR](sbstp/rust-igd#56) in rust-igd, `igd` v0.12.1 no longer panics if there is an issue while searching for a gateway. So updating igd makes lighthouse emit a helpful log instead of panicking. ## Additional Info No CHANGELOG exists in rust-igd. 👀 Here is the commit history between v0.11.1 and v0.12.1. No breaking changes. sbstp/rust-igd@v0.11.1...v0.12.1
In current |
Closed by #4257 |
## Issue Addressed sigp#4171 ## Proposed Changes Through [this PR](sbstp/rust-igd#56) in rust-igd, `igd` v0.12.1 no longer panics if there is an issue while searching for a gateway. So updating igd makes lighthouse emit a helpful log instead of panicking. ## Additional Info No CHANGELOG exists in rust-igd. 👀 Here is the commit history between v0.11.1 and v0.12.1. No breaking changes. sbstp/rust-igd@v0.11.1...v0.12.1
## Issue Addressed sigp#4171 ## Proposed Changes Through [this PR](sbstp/rust-igd#56) in rust-igd, `igd` v0.12.1 no longer panics if there is an issue while searching for a gateway. So updating igd makes lighthouse emit a helpful log instead of panicking. ## Additional Info No CHANGELOG exists in rust-igd. 👀 Here is the commit history between v0.11.1 and v0.12.1. No breaking changes. sbstp/rust-igd@v0.11.1...v0.12.1
## Issue Addressed sigp#4171 ## Proposed Changes Through [this PR](sbstp/rust-igd#56) in rust-igd, `igd` v0.12.1 no longer panics if there is an issue while searching for a gateway. So updating igd makes lighthouse emit a helpful log instead of panicking. ## Additional Info No CHANGELOG exists in rust-igd. 👀 Here is the commit history between v0.11.1 and v0.12.1. No breaking changes. sbstp/rust-igd@v0.11.1...v0.12.1
Description
Lighthouse crashes during startup
Version
a53830f (current master)
Present Behaviour
lighthouse bn --network sepolia --execution-endpoint http://localhost:8551 --execution-jwt <path> --checkpoint-sync-url https://sepolia.checkpoint-sync.ethpandaops.io --disable-deposit-contract-sync
Expected Behaviour
No exception
The text was updated successfully, but these errors were encountered: