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.

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

Write less log spam if host is unreachable. #91

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

moridius
Copy link

@moridius moridius commented Feb 8, 2024

If a network host is unreachable for respondd it now writes a one-line error message instead of several lines of stacktrace.

@jplitza
Copy link
Member

jplitza commented Jul 9, 2024

I'd suggest a message like "Address unreachable while sending response to {address}". IMO it's easier to understand and easier to produce (since I don't like poking around stack frames).

Also, please use errno.ENETUNREACH instead of 101 as a magic number.

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.

2 participants