Page MenuHomePhabricator

ircecho should support nickserv registration
Closed, ResolvedPublic

Description

The ircecho script at operations/debs/ircecho.git does not support nickserv registration. That prevents us from having our bots registered properly (T48252: IRC bots account pending nickserv registration (tracking)).

Ideally we would want ircecho to read a configuration file instead of using command line arguments. That will let us put the password in a more or less secured place since we do not want to expose it on the command line.

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:25 AM
bzimport set Reference to bz46254.
bzimport added a subscriber: Unknown Object (MLST).

(In reply to comment #0)

Ideally we would want ircecho to read a configuration file instead of using
command line arguments. That will let us put the password in a more or less
secured place since we do not want to expose it on the command line.

(From bug 20771 comment #4)

Should probably use the certificates instead of a password, to authenticate.
https://blog.freenode.net/2012/06/upgrade-and-database-prune-completed/
http://pricechild.blogspot.co.uk/2012/04/identifying-to-freenode-testnet-
with.html

That is solely for icinga-wm and it works just fine without nick registration. I dont think anyone will ever add support for nick registration to ircecho, but if there is interest still feel free to fill ANOTHER task.

I think it's fine to reopen this without needing another task.
This is also for shinken-wm and we've recently found that the $~a ban applied everywhere via #wikimedia-bans is preventing shinken-wm from joining channels and speaking. Adding SASL auth to some NickServ account is better than assigning the instance a floating IP and getting all the relevant channel ops to add exemptions (helpfully, exempts can't be globalised like bans)

Change 455277 had a related patch set uploaded (by Alex Monk; owner: Alex Monk):
[operations/puppet@production] ircecho: Add support for authenticating with SASL

https://gerrit.wikimedia.org/r/455277

Change 455277 merged by Andrew Bogott:
[operations/puppet@production] ircecho: Add support for authenticating with SASL

https://gerrit.wikimedia.org/r/455277

Krenair claimed this task.