GitHub/ergo
3
0
Fork 0
mirror of https://github.com/ergochat/ergo.git synced 2024-12-22 10:42:52 +01:00
Code Issues Projects Releases Wiki Activity

add a warning for the #1634 case

Browse Source 
Configurations with require-sasl and open account registration are valid,
but likely unintended. Show a warning about them.
This commit is contained in:
Shivaram Lingamneni 2021-04-27 08:44:54 -04:00
parent 5c157adf45
commit aa27ad98a8
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
irc/server.go
View File

@ -716,6 +716,10 @@ func (server *Server) applyConfig(config *Config) (err error) {
// we are now open for business
err = server.setupListeners(config)
// send other config warnings
if config.Accounts.RequireSasl.Enabled && config.Accounts.Registration.Enabled {
server.logger.Warning("server", "Warning: although require-sasl is enabled, users can still register accounts. If your server is not intended to be public, you must set accounts.registration.enabled to false.")
}
if !initial {
// push new info to all of our clients
@ -852,7 +856,7 @@ func (server *Server) setupListeners(config *Config) (err error) {
}
if publicPlaintextListener != "" {
server.logger.Warning("listeners", fmt.Sprintf("Your server is configured with public plaintext listener %s. Consider disabling it for improved security and privacy.", publicPlaintextListener))
server.logger.Warning("listeners", fmt.Sprintf("Warning: your server is configured with public plaintext listener %s. Consider disabling it for improved security and privacy.", publicPlaintextListener))
}
return