eggdrop icon indicating copy to clipboard operation
eggdrop copied to clipboard

KICK optimization interfers with queued messages even if optimize-kicks is turned off

Open thommey opened this issue 2 years ago • 0 comments

It is somewhat okay that putkick, like pushmode, check channel membership before executing their action. They are documented as such.

The optimize-kicks setting promises to group kicks together (max-kicks) but I would expect it only to be optimized with putkick, like pushmode.

However, eggdrop is far more invasive with kicks. It actively goes through queued messages (putserv/putquick/puthelp), parses those, finds the ones that KICK someone and applies logic to that, for example purge_kicks().

IMHO Eggdrop should not parse and mess with putserv messages at all, kick optimization should be restricted to putkick (like pushmode). Alternatively, optimize-kicks 0 should at least turn this invasive operation off.

Background is that networks with hidden users (ircu +D delayed join) want to execute that kick if scripts take care of tracking users themselves.

Found by: Empus

thommey avatar Jul 18 '22 18:07 thommey

Duplicate of #969

thommey avatar Sep 12 '22 07:09 thommey