Skip to content

Fix and enhance error logging#1743

Merged
vanosg merged 2 commits intoeggheads:developfrom
michaelortmann:ident.error
Oct 23, 2025
Merged

Fix and enhance error logging#1743
vanosg merged 2 commits intoeggheads:developfrom
michaelortmann:ident.error

Conversation

@michaelortmann
Copy link
Copy Markdown
Member

@michaelortmann michaelortmann commented Jan 20, 2025

Found by: tuvok81
Patch by: michaelortmann
Fixes: #1742

One-line summary:

Additional description (if needed):
if .oidentd.conf doesnt exist (yet), eggdrop will create it. This case is not an error and should not be logged as one.

(oops, i removed some additional description here by accident post merge)

Test cases demonstrating functionality (if applicable):

loadmodule ident
set ident-method 0

Before:

rm ~/.oidentd.conf
$ ./eggdrop -t BotA.conf
[...]
[14:11:46] net: open_telnet_raw(): idx 4 host 127.0.0.1 ip 127.0.0.1 port 6697 ssl 1
[14:11:46] IDENT: oident.conf missing, or error opening for reading
[14:11:46] Failed connect to 127.0.0.1 (No such file or directory)

After:

rm ~/.oidentd.conf
$ ./eggdrop -t BotA.conf
[...]
[14:12:32] net: open_telnet_raw(): idx 4 host 127.0.0.1 ip 127.0.0.1 port 6697 ssl 1
[14:12:32] net: attempted socket connection refused: 127.0.0.1:6697
[14:12:32] Failed connect to 127.0.0.1 (Connection refused)

@michaelortmann michaelortmann changed the title (WIP) Enhance error logging Enhance error logging Jan 20, 2025
@michaelortmann michaelortmann changed the title Enhance error logging Fix and enhance error logging Jan 20, 2025
@vanosg vanosg added this to the v1.10.2 milestone Jun 14, 2025
@vanosg vanosg merged commit 890e7e4 into eggheads:develop Oct 23, 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.

wired error message on first start

2 participants