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.

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

.rehash could let bot die #1479

Open
michaelortmann opened this issue Oct 7, 2023 · 3 comments · May be fixed by #1634
Open

.rehash could let bot die #1479

michaelortmann opened this issue Oct 7, 2023 · 3 comments · May be fixed by #1634

Comments

@michaelortmann
Copy link
Member

michaelortmann commented Oct 7, 2023 •
edited
Loading

eggdrop version: current git develop: 901c3a1

.rehash
[23:40:30] tcl: builtin dcc call: *dcc:rehash -HQ 1 
[23:40:30] #-HQ# rehash
Rehashing.
[23:40:30] Writing user file...
[23:40:30] Writing channel file...
[23:40:30] Rehashing ...
[23:40:30] Writing channel file...
[23:40:30] Loading dccwhois.tcl...
[23:40:30] Loaded dccwhois.tcl
[23:40:30] Userinfo TCL v1.08 loaded (URL BF GF IRL EMAIL DOB PHONE ICQ YOUTUBE TWITCH).
[23:40:30] use '.help userinfo' for commands.
[23:40:30] Loaded quotepong.tcl
[23:40:30] tcldcc: setlisten(): getaddrinfo(): ip virtual.host.com not known
[23:40:30] Tcl error in file 'BotA.conf':
[23:40:30] this port is already bound to 0.0.0.0 on this machine, remove it before trying to bind to this IP
    while executing
"listen +3343 all"
    (file "BotA.conf" line 1618)
[23:40:30] * CONFIG FILE NOT LOADED (NOT FOUND, OR ERROR)

could be related to the garbage config setting set listen-addr "virtual.host.com"
but that should not lead to the bot dying on .rehash
@michaelortmann michaelortmann mentioned this issue Oct 8, 2023
Open
@ia2115
Copy link

ia2115 commented Jul 3, 2024

this also happends to me for some reason.

.rehash
[17:38:17] #user# rehash
Rehashing.
[17:38:17] Rehashing ...
[17:38:17] Tcl error in file 'eggdrop.conf':
[17:38:17] this port is already bound to a specific IP on this machine, remove it before trying to bind to all interfaces
while executing
"listen 4040 all"
(file "eggdrop.conf" line 37)
[17:38:17] * CONFIG FILE NOT LOADED (NOT FOUND, OR ERROR)
Connection closed by foreign host.

anyone found out why this is happening?

@michaelortmann michaelortmann linked a pull request Jul 3, 2024 that will close this issue
Open
@michaelortmann
Copy link
Member Author

michaelortmann commented Jul 3, 2024 •
edited
Loading

This is happening, when on rehash eggdrop tries to set a listen port to ANYIP (0.0.0.0 / ::) while eggdrop is already listening to that port, but on a specific IP.

listen 4040 all doesnt specify an IP, so that becomes ANYIP.

When listen ports are changed during eggdrops runtime via tcl listen that error is reported, but when the same change would happen during rehash eggdrop will die after reporting it. rehash should not change listen ports. so far so good. but i think eggdrop should not die during rehash either. Logging a warning instead should suffice.

.dccstat will show current listen IP/ports

.tcl listen [ip] <port> off removes a current listen IP/port

@ia2115
Copy link

ia2115 commented Jul 3, 2024

ye thats actually a good point, also i think documentation should point out how important it is in listen statement to include ip adress, without it you experience wierd stuff as i did, finally i found this and fixed my error.

im happy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

(WIP) Fix and enhance logging for port already bound michaelortmann/eggdrop
2 participants
@ia2115 @michaelortmann