Skip to content

Enhance tls error logging - #1841

Merged
vanosg merged 2 commits into
eggheads:developfrom
michaelortmann:ssl.fatal.recordoverflow
Nov 28, 2025
Merged

vanosg merged 2 commits into
eggheads:developfrom
michaelortmann:ssl.fatal.recordoverflow

Conversation

@michaelortmann

Copy link
Copy Markdown
Member

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

One-line summary:
Enhance tls error logging in case of connecting via +port to a non ssl port

Additional description (if needed):

Test cases demonstrating functionality (if applicable):
The last line is new and logged to LOG_MISC instead of tls->loglevel, which could hide such message

[15:02:17] Trying server 127.0.0.1:+6667
[15:02:17] [!m] CAP LS 302
[15:02:17] TLS: alert during write: fatal (record overflow).
[15:02:17] ERROR: No tls port?

@vanosg vanosg added this to the v1.10.2 milestone Oct 23, 2025
@vanosg
vanosg merged commit d13fbc1 into eggheads:develop Nov 28, 2025
26 checks passed
@michaelortmann
michaelortmann deleted the ssl.fatal.recordoverflow branch November 28, 2025 18:45
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