Fix OIdentdConfigGenerator double-printing on ssl
authoresainane <esainane+github@gmail.com>
Sat, 21 Nov 2015 08:38:11 +0000 (21:38 +1300)
committerManuel Nickschas <sputnick@quassel-irc.org>
Sat, 2 Jan 2016 16:59:32 +0000 (17:59 +0100)
commite5b84abb18c6419d24d807bd6f84ce44be172385
treeec14219305b5eb6422968000ace6e478515d2e68
parent72d822f3583f67252fed8544b76f8501cda7531f
Fix OIdentdConfigGenerator double-printing on ssl

connected() is always connected to socketInitialized().
encrypted() is also connected to socketInitialized() when ssl is enabled.

socketInitialized() fires socketOpen(...) unconditionally, but does not fire socketInitialized(...) if we're using SSL without encryption yet - see line 464.

With socketOpen(...) connected to OIdentdConfigurationGenerator::addSocket, OIdentdConfigurationGenerator writes to .oidentd.conf twice for every connection.

Changing this to have socketInitialized(...) connected instead prevents this duplication.
src/core/corenetwork.cpp