From 97cfc0b62983c432bfb0f6b3401e1453396be6e4 Mon Sep 17 00:00:00 2001 From: Daniel Oaks Date: Thu, 27 Jul 2017 23:29:08 +1000 Subject: [PATCH] Print warnings to heavily encourage exposing port 6697 as a TLS port --- irc/server.go | 17 ++++++++++++++++- 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/irc/server.go b/irc/server.go index a714cd4a..5be7a09e 100644 --- a/irc/server.go +++ b/irc/server.go @@ -312,8 +312,23 @@ func NewServer(configFilename string, config *Config, logger *logger.Manager) (* server.password = config.Server.PasswordBytes() } + tlsListeners := config.TLSListeners() for _, addr := range config.Server.Listen { - server.createListener(addr, config.TLSListeners()) + server.createListener(addr, tlsListeners) + } + + if len(tlsListeners) == 0 { + server.logger.Warning("startup", "You are not exposing an SSL/TLS listening port. You should expose at least one port (typically 6697) to accept TLS connections") + } + var usesStandardTLSPort bool + for addr := range config.TLSListeners() { + if strings.Contains(addr, "6697") { + usesStandardTLSPort = true + break + } + } + if 0 < len(tlsListeners) && !usesStandardTLSPort { + server.logger.Warning("startup", "Port 6697 is the standard TLS port for IRC. You should (also) expose port 6697 as a TLS port to ensure clients can connect securely") } if config.Server.Wslisten != "" {