Channel key support

[timypcr]

I'd like to use logbot for an internal IRC server at work but the channels we have setup require a key, is there quick modification that can be made so logbot can support this?

Thanks,
Tim

[FiXato]

the c.join command at

logbot/logbot.py

Lines 359 to 360 in db89264

	for chan in self.chans:
	c.join(chan)

theoretically supports a second parameter for a channel key:

logbot/irclib.py

Lines 689 to 691 in db89264

	def join(self, channel, key=""):
	"""Send a JOIN command."""
	self.send_raw("JOIN %s%s" % (channel, (key and (" " + key))))

Channels are set at

logbot/logbot.py

Line 219 in db89264

self.chans = [x.lower() for x in channels]

So, it doesn't look like there is a clean way to quickly do it without rewriting things.

Since channel names should not contain spaces, you could add a space + channel key to your channels defined at

logbot/logbot.py

Line 72 in db89264

CHANNELS=["#excid3","#keryx"]

Then extract the channel keys with a split before they are lowercased and assigned to self.chan, and then use that as a second parameter at the join command.

I don't really use this library myself anymore, so to be fair, I don't really have the motivation to submit a pull request with working code for this feature request, but hopefully this helps you (or @excid3) add a workaround yourself. :)

[FiXato]

If someone does end up rewriting it, I would suggest you make use of the IRC protocol's format of the JOIN command: https://tools.ietf.org/html/rfc2812#page-16
And just have the user define the channels and keys in the argument format the server expects; the actual channel names could then be extracted (and downcased if you want) from the actual join replies from the server. That way you don't have to match up the channel keys with each channel, and just rely on the server to do the matching (as long as the keys are entered in the right order as the channels; AFAIK it doesn't matter though if there are channels without keys in between those that require one; the server should be smart enough).
Basically use comma delimiters for the channel names, followed by a space, followed by comma-separated channel keys.

[excid3]

Since keys is just a second parameter, you can create a KEYS array next to channels and then just pass those in during this loop:

logbot/logbot.py

Lines 359 to 360 in db89264

	for chan in self.chans:
	c.join(chan)

Should be really simple to fix.