Skip to content

Commit a4941be

Browse files
committed
Warn on missing serverURL
We see some issues that are based on not properly configured `config.serverURL`. This patch adds a warning when `config.serverURL` is an empty value. This should provide users direct feedback about how to improve their configs. Signed-off-by: Sheogorath <sheogorath@shivering-isles.com>
1 parent 769a1c4 commit a4941be

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

lib/config/index.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,10 @@ config.serverURL = (function getserverurl () {
101101
return url
102102
})()
103103

104+
if (config.serverURL === '') {
105+
logger.warn('Neither \'domain\' nor \'CMD_DOMAIN\' is configured. This can cause issues with various components.\nHint: Make sure \'protocolUseSSL\' and \'urlAddPort\' or \'CMD_PROTOCOL_USESSL\' and \'CMD_URL_ADDPORT\' are configured properly.')
106+
}
107+
104108
config.Environment = Environment
105109

106110
// auth method

0 commit comments

Comments
 (0)