From dcda5304ff4881802357ad8a9d3ea114398e6429 Mon Sep 17 00:00:00 2001 From: Christopher Poile Date: Tue, 20 Jan 2026 14:58:08 -0500 Subject: [PATCH] Suppress SiteURL log error when in CI or local_testing mode (#34982) --- server/channels/app/server.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/server/channels/app/server.go b/server/channels/app/server.go index 0f3eeb5287e..109822c4aa3 100644 --- a/server/channels/app/server.go +++ b/server/channels/app/server.go @@ -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