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

fix: place getNodeAddress inside try catches #247

Closed

maschad wants to merge 1 commit into ChainSafe:master from maschad:fix/discv5-error-handling-fixes

maschad commented May 10, 2023 •

edited

Loading

In running the discv5 crawler there were a few bugs @wemeetagain and I discovered. This PR addresses those:

Handle timeout errors
Handle udp doesn't exist in multiaddr from error


          fix: place getNodeAddress inside try catches

f94d836

wemeetagain force-pushed the master branch 2 times, most recently from b3e0a17 to 938e0fb Compare

November 29, 2023 15:59

Member

wemeetagain commented Jan 25, 2024

closing in favor of fix: improve error handling #284

wemeetagain closed this

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

Labels

None yet