upstream/mattermost
1
0
Fork 0
mirror of https://github.com/mattermost/mattermost.git synced 2026-02-18 18:18:23 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

Suppress SiteURL log error when in CI or local_testing mode (#34982)

Browse source
This commit is contained in:
Christopher Poile 2026-01-20 14:58:08 -05:00 committed by GitHub
parent fde4393144
commit dcda5304ff
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
server/channels/app/server.go
View file

@ -384,7 +384,10 @@ func NewServer(options ...Option) (*Server, error) {
}
if _, err = url.ParseRequestURI(*s.platform.Config().ServiceSettings.SiteURL); err != nil {
mlog.Error("SiteURL must be set. Some features will operate incorrectly if the SiteURL is not set. See documentation for details: https://mattermost.com/pl/configure-site-url")
// Don't spam the logs when in CI or local testing mode
if !(os.Getenv("IS_CI") == "true" || os.Getenv("IS_LOCAL_TESTING") == "true") {
mlog.Error("SiteURL must be set. Some features will operate incorrectly if the SiteURL is not set. See documentation for details: https://mattermost.com/pl/configure-site-url")
}
}
// Start email batching because it's not like the other jobs