GitHub/PyLink
3
0
Fork 0
mirror of https://github.com/jlu5/PyLink.git synced 2024-11-27 13:09:23 +01:00
Code Issues Projects Releases Wiki Activity

clientbot: only split /names replies by spaces

Browse Source 
This fixes issues with colored hostnames because \x1f is treated as a whitespace char by str.split().

Closes #641.
This commit is contained in:
James Lu 2019-04-08 22:33:55 -07:00
parent 5ca57cb3c1
commit 9056799633
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
protocols/clientbot.py
View File

@ -740,14 +740,19 @@ class ClientbotWrapperProtocol(ClientbotBaseProtocol, IRCCommonProtocol):
prefix_to_mode = {v:k for k, v in self.prefixmodes.items()}
prefixes = ''.join(self.prefixmodes.values())
for name in args[-1].split():
# N.B. only split on spaces because of color in hosts nonsense...
# str.split() by default treats \x1f as whitespace
for name in args[-1].split(' '):
nick = name.lstrip(prefixes)
# Handle userhost-in-names where available.
if 'userhost-in-names' in self.ircv3_caps:
nick, ident, host = utils.split_hostmask(nick)
else:
ident = host = None
if 'userhost-in-names' in self.ircv3_caps:
try:
nick, ident, host = utils.split_hostmask(nick)
except ValueError:
log.exception('(%s) Failed to split hostmask %r from /names reply on %s', self.name, nick, channel)
# If error, leave nick unsplit
# Get the PUID for the given nick. If one doesn't exist, spawn
# a new virtual user.