mirror of
https://github.com/Mikaela/Limnoria.git
synced 2024-12-23 19:22:45 +01:00
Services: fix problem with some channels being mixed up between networks on startup, when noJoinsUntilIdentified is true.
When noJoinsUntilIdentified config is true, the bot holds join messages in a 'waitingJoins' list, and processes them once nickserv identification comes through. The problem was that when the bot is configured to join multiple networks, join messages from different networks would get appended to the same list, without any differentiation by which message belongs to which network. Thus, if there are messages waiting for multiple networks, it would often be the case that whichever network got identification done first, would 'pick up' other network's join messages. This fix stores the network name along with the join messages in the list, and has each network pick out only its own join messages. Conflicts: src/version.py Signed-off-by: James McCoy <jamessan@users.sourceforge.net>
This commit is contained in:
parent
71bcc2bc7c
commit
d5648ab534
@ -30,6 +30,7 @@
|
||||
|
||||
import re
|
||||
import time
|
||||
import copy
|
||||
|
||||
import config
|
||||
|
||||
@ -76,7 +77,7 @@ class Services(callbacks.Plugin):
|
||||
if self.registryValue('noJoinsUntilIdentified'):
|
||||
self.log.info('Holding JOIN to %s until identified.',
|
||||
msg.args[0])
|
||||
self.waitingJoins.append(msg)
|
||||
self.waitingJoins.append((irc.network, msg,))
|
||||
return None
|
||||
return msg
|
||||
|
||||
@ -314,9 +315,11 @@ class Services(callbacks.Plugin):
|
||||
for channel in self.channels:
|
||||
irc.queueMsg(networkGroup.channels.join(channel))
|
||||
if self.waitingJoins:
|
||||
for m in self.waitingJoins:
|
||||
irc.sendMsg(m)
|
||||
self.waitingJoins = []
|
||||
tmp_wj = copy.deepcopy(self.waitingJoins) # can't iterate over list if we're modifying it
|
||||
for netname, m in tmp_wj:
|
||||
if netname == irc.network:
|
||||
irc.sendMsg(m)
|
||||
self.waitingJoins.remove((netname, m,))
|
||||
elif 'not yet authenticated' in s:
|
||||
# zirc.org has this, it requires an auth code.
|
||||
email = s.split()[-1]
|
||||
|
Loading…
Reference in New Issue
Block a user