Skip to content
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Commit 416909a

Browse files
committed
Print error to the log, don't send it to user
It could leak the bot token in case of a network error. Credits for the bug go to @Meidum.
1 parent 7e76b3c commit 416909a

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

run.go

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,7 @@ func (c *Client) Run() {
2828
log.Printf("username: %s, id: %d, text: %s", update.CallbackQuery.Message.From.UserName, update.CallbackQuery.Message.From.ID, update.CallbackQuery.Message.Text)
2929
go func() {
3030
if err := c.processCallback(update.CallbackQuery); err != nil {
31-
msg := tgbotapi.NewMessage(update.CallbackQuery.Message.Chat.ID, err.Error())
32-
msg.ReplyToMessageID = update.CallbackQuery.Message.MessageID
33-
go c.Bot.Send(msg)
31+
log.Println(err)
3432
}
3533
}()
3634
} else {
@@ -49,8 +47,7 @@ func (c *Client) Run() {
4947
if c.CheckAuth(update.Message.From) {
5048
go func() {
5149
if err := c.processMessage(update.Message); err != nil {
52-
msg.Text = err.Error()
53-
go c.Bot.Send(msg)
50+
log.Println(err)
5451
}
5552
}()
5653
} else {

0 commit comments

Comments
 (0)