From 28db5c80d686d98a18ba2cc71c36e1479e5f10ff Mon Sep 17 00:00:00 2001 From: adbenitez Date: Thu, 13 Apr 2023 10:46:13 -0400 Subject: [PATCH] bugfix: don't modifiy original message in Gateway.SendMessage() --- gateway/gateway.go | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/gateway/gateway.go b/gateway/gateway.go index a2d572be..b91d08ca 100644 --- a/gateway/gateway.go +++ b/gateway/gateway.go @@ -323,7 +323,7 @@ func (gw *Gateway) ignoreFilesComment(extra map[string][]interface{}, igMessages return false } -func (gw *Gateway) modifyUsername(msg *config.Message, dest *bridge.Bridge) string { +func (gw *Gateway) modifyUsername(msg *config.Message, dest *bridge.Bridge) { if dest.GetBool("StripNick") { re := regexp.MustCompile("[^a-zA-Z0-9]+") msg.Username = re.ReplaceAllString(msg.Username, "") @@ -369,16 +369,15 @@ func (gw *Gateway) modifyUsername(msg *config.Message, dest *bridge.Bridge) stri gw.logger.Errorf("modifyUsernameTengo error: %s", err) } nick = strings.ReplaceAll(nick, "{TENGO}", tengoNick) - return nick + msg.Username = nick } -func (gw *Gateway) modifyAvatar(msg *config.Message, dest *bridge.Bridge) string { +func (gw *Gateway) modifyAvatar(msg *config.Message, dest *bridge.Bridge) { iconurl := dest.GetString("IconURL") iconurl = strings.Replace(iconurl, "{NICK}", msg.Username, -1) if msg.Avatar == "" { msg.Avatar = iconurl } - return msg.Avatar } func (gw *Gateway) modifyMessage(msg *config.Message) { @@ -461,8 +460,8 @@ func (gw *Gateway) SendMessage( } msg.Channel = channel.Name - msg.Avatar = gw.modifyAvatar(rmsg, dest) - msg.Username = gw.modifyUsername(rmsg, dest) + gw.modifyAvatar(&msg, dest) + gw.modifyUsername(&msg, dest) // exclude file delete event as the msg ID here is the native file ID that needs to be deleted if msg.Event != config.EventFileDelete {