Skip to content

Conversation

@michaelortmann
Copy link
Member

@michaelortmann michaelortmann commented Oct 19, 2025

Found by: https://github.com/michaelortmann/
Patch by: https://github.com/michaelortmann/
Fixes:

One-line summary:
Also log port for failed connect

Additional description (if needed):
Useful to relate log messages to each other and make it very clear what exactly failed

Test cases demonstrating functionality (if applicable):
Before:

[17:29:12] Trying server 127.0.0.1:6667
[...]
[17:29:12] Failed connect to 127.0.0.1 (Connection refused)

After:

[22:35:43] Trying server 127.0.0.1:6667
[...]
[22:35:43] Failed connect to 127.0.0.1:6667 (Connection refused)
.jump ::6 +7
[22:41:25] #-HQ# jump ::6 +7 
Jumping servers......
[22:41:26] Trying server [::6]:+7
[22:41:26] Failed connect to [::6]:+7 (Network is unreachable)

@michaelortmann michaelortmann changed the title Also log port for failed connect (WIP) Also log port for failed connect Oct 19, 2025
@michaelortmann michaelortmann changed the title (WIP) Also log port for failed connect Also log port for failed connect Oct 19, 2025
@vanosg vanosg added this to the v1.10.2 milestone Dec 20, 2025
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