From 95f784a47588672867d462ac0294674e664024a2 Mon Sep 17 00:00:00 2001 From: Agniva De Sarker Date: Fri, 25 Feb 2022 13:04:18 +0530 Subject: [PATCH] MM-36668: Trim large slash commands before logging (#19621) Automatic Merge --- app/command.go | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/app/command.go b/app/command.go index 640103064b..956c6013c6 100644 --- a/app/command.go +++ b/app/command.go @@ -25,6 +25,7 @@ import ( const ( CmdCustomStatusTrigger = "status" usernameSpecialChars = ".-_" + maxTriggerLen = 512 ) var atMentionRegexp = regexp.MustCompile(`\B@[[:alnum:]][[:alnum:]\.\-_:]*`) @@ -225,6 +226,10 @@ func (a *App) ExecuteCommand(c *request.Context, args *model.CommandArgs) (*mode return a.HandleCommandResponse(c, cmd, args, response, true) } + if len(trigger) > maxTriggerLen { + trigger = trigger[:maxTriggerLen] + trigger += "..." + } return nil, model.NewAppError("command", "api.command.execute_command.not_found.app_error", map[string]interface{}{"Trigger": trigger}, "", http.StatusNotFound) }