Telegram Bot на Powershell вылетает с ошибкой Bad Request: message to edit not found. Почему?

Question

[trlhrn]

Добрый день.
Делаю Телеграм Бот на PowerShell на основе этого поста https://habr.com/ru/post/483660/
У меня завязно на секции с callback_query (добавлены различные функции, отправка данных и отправка inline keyboard).
Когда только я пользуюсь Ботом - все ОК. Если кто-то еще жмет в нем кнопки, то в моей сессии после нажатия любой кнопки в PowerShell консоли появляется сообщение "Bad Request: message to edit not found"
Помогите разобраться в чем может быть дело.
Также иногда сообщения после нажатия кнопки уходят не тому пользователю, кто нажимал.

Comments

[MaxKozlov]

Скорее всего у вас там глобальные переменные заюзаны для callback, что является ошибкой.

Но не видя кода это невозможно ни подтвердить, ни опровергнуть.

Полные логи запросов и ответов на консоль или в файл пишите, может увидите что не так.

[trlhrn]

MaxKozlov, подскажите, что имеется в виду под глобальными переменными?

[MaxKozlov]

trlhrn, Ну, например, в статье
все переменные выше функций - глобальные для скрипта в том смысле, что доступны из любой из этих функций.

Причём они существуют в одном экземпляре для всех запросов, которые обрабатывает скрипт.

И если какая-то функция будет туда писать данные именно для обработки текущего callback'а от одного юзера, то при обработке этого же или другого callback'а от другого юзера эти данные затрутся или будут использованы неверно.

Данные при обработке кнопок надо передавать или привязывая к user_id или гоняя их внутри запросов через callback_data

[trlhrn]

MaxKozlov, ниже часть моего кода. Я конечно подозревал, что все это не очень хорошо выглядит и где-то выбран не верный подход для работы с переменными. Остальная часть в работе с кнопками в общем и целом аналогична этой.

while ($true) {

    $return = getUpdates $URL_get

    #If the MSG is in default text
    if ($return.text) {

        #write-host "$($return["chat_id"])"
        #Write-Host "$($return["msg_id"])"
        switch -Wildcard ($return["text"]) {
            "*" {
                $text = "Hello, *$($return["f_name"])*! Please, choose the option you need:"
                $chat_id = $return.chat_id
                $keybrd = sendKeyboardMain $URL_set $buttons $chat_id $text
                
                #Get MSG ID from MAIN keybrd
                $last_bot_msg = $keybrd.result.message_id
            }

        }  

    }
       
    #If button has been pressed
    elseif ($return.callback_data) {

        #If GET REPORT button pressed
        If ($return.callback_data -eq "SQLPredReport") {

            #Check if user has access (subscription)
            $useraccessbtns = UserAccessToButtons

            If ($useraccessbtns -eq $true) {
           
                $keybrd = Invoke-RestMethod -Uri "https://api.telegram.org/bot$($token)/editMessageText?chat_id=$($return.chat_id)&message_id=$($last_bot_msg)&text=Choose period:&reply_markup=$buttonreport" -ContentType "application/json; charset=utf-8"
                $last_bot_msg = $keybrd.result.message_id

                #Loop until Back button will be pressed
                Do {

                    $return = getUpdates $URL_get

                    #If TODAY btn pressed
                    If ($return.callback_data -eq "reptoday") {

                        $reportdate = (Get-Date).ToString('yyyy-MM-dd')

                        #Run SQL Report (TODAY) function
                        $SQLReport = GetReportTodayYesterday
                        sendMessage $URL_set $($return.chat_id) $SQLReport

                        #Delete previous MSG with keyboard
                        Invoke-RestMethod -Uri "https://api.telegram.org/bot$($token)/deleteMessage?chat_id=$($return.chat_id)&message_id=$($last_bot_msg)"

                        #Send Report keybrd again
                        $text = "Choose period:"
                        $keybrd = sendKeyboardReport $URL_set $buttonreport $chat_id $text
                        $last_bot_msg = $keybrd.result.message_id

                    }

[MaxKozlov]

trlhrn, Ну в общем не всё стало яснее :)
Но кое-что видно уже. Тот же $last_bot_msg вызывает очень большие сомнения.
Присваивается он во внешнем цикле, причём внутри IF, а что будет если text='' ? он останется от предыдущего захода.
Что делает внутренний цикл

#If button has been pressed
    elseif ($return.callback_data) {

не понятно, но если где-то в нём есть ещё раз getUpdates, то именно здесь у вас и косяк. потому что при получении обновлений сообщения приходят от разных пользователей и lastmsgid может оказаться не от того пользователя.
Сообщения привязаны к чату, насколько я знаю, то есть если в чате №1 у вас есть сообщение №12345, то в чате №2 у вас сообщение может быть №3456, а вы пытаетесь редактировать 12345.

Ещё, судя по всему, вы рассчитываете принимать только одно сообщение за раз, а за один getupdates может прийти несколько сообщений, если вы специально не попросили отдавать по одному.

[trlhrn]

MaxKozlov, спасибо за комментарии. А как Вы определили, что принимается только одно сообщение за раз? Касательно ещё одного get updates. Он действительно есть. Попробую его убрать.

[MaxKozlov]

trlhrn, Что там реально принимается - зависит от того что успели юзеры написать с момента предыдушего запроса
а вы рассчитываете только на одно потому что в первой же строке у вас
if ($return.text) {
то есть вы рассчитываете не на массив

А вот что там может оказаться:

> $result = Invoke-RestMethod "https://api.telegram.org/bot$code/getUpdates"
> $result.result
update_id message
--------- -------
224433664 @{message_id=1122; from=; chat=; date=1670697880; text=qqq}
224433665 @{message_id=1123; from=; chat=; date=1670697881; text=aaa}

[trlhrn]

MaxKozlov, как только уже ни пробовал. В примере ниже на текст B уходит сообщение от бота с верным first_name пользователя. Все ОК вроде бы. При нажатии на кнопку на клавиатуре, код пользователя в первый раз называет корректно по никнейму, а затем, один из пользователей отваливается - то есть при нажатии кнопки, сообщения летят другому пользователю.

$URL_get = "https://api.telegram.org/bot$token/getUpdates"
$URL_set = "https://api.telegram.org/bot$token/sendMessage"
$token = "TOKEN"

# timeout sec
$timeout = 1

function getUpdates($URL) {
    $json = Invoke-RestMethod -Uri $URL
    $data = $json.result | Select-Object -Last 1

    #Make variable nulliable
    $text = $null
    $callback_data = $null
    $chat_id = $null


    # Нажатие на кнопку
    if ($data.callback_query) {  
        $callback_data = $data.callback_query.data
        $chat_id = $data.callback_query.chat.id
        $f_name = $data.callback_query.from.first_name
        $l_name = $data.callback_query.from.last_name
        $username = $data.callback_query.chat.username
    }
    # Обычное сообщение
    elseif ($data.message) {
        $chat_id = $data.message.chat.id
        $text = $data.message.text
        $f_name = $data.message.chat.first_name
        $l_name = $data.message.chat.last_name
        $type = $data.message.chat.type
        $username = $data.message.chat.username

    }

    $ht = @{}
    $ht["chat_id"] = $chat_id
    $ht["text"] = $text
    $ht["f_name"] = $f_name
    $ht["l_name"] = $l_name
    $ht["username"] = $username
    $ht["callback_data"] = $callback_data
    # confirm
    Invoke-RestMethod "$($URL)?offset=$($($data.update_id)+1)" -Method Get | Out-Null
    
    return $ht
}


function sendMessage($URL, $chat_id, $text) {

    #Create a Hash Table
    $ht = @{
        text       = $text
        parse_mode = "Markdown"
        chat_id    = $chat_id
    }
    #Convert data to JSON
    $json = $ht | ConvertTo-Json

    Invoke-RestMethod $URL -Method Post -ContentType 'application/json; charset=utf-8' -Body $json | Out-Null   

}

function sendMessagePr($URL, $chat_id, $text) {

    #Create a Hash Table
    $ht = @{
        text       = $text
        parse_mode = "HTML"
        chat_id    = $chat_id
    }
    #Convert data to JSON
    $json = $ht | ConvertTo-Json

    Invoke-RestMethod $URL -Method Post -ContentType 'application/json; charset=utf-8' -Body $json | Out-Null   

}

#Buttons (MAIN)
###############
$buttonsmain = @()
$button0 = @{ "text" = "Text1"; callback_data = "Data1" }
$button1 = @{ "text" = "Text2"; callback_data = "Data2" }
$button2 = @{ "text" = "Website"; "url" = 'https://google.com/' }
$button3 = @{ "text" = "Text3"; callback_data = "Data3" }

$rowmain1 = @($button0, $button1)
$rowmain2 = @($button3, $button2)

$buttons = ($rowmain1, $rowmain2)
#$buttons = ($button0, $button1, $button2, $button3)

$buttonsmain = @{"inline_keyboard" = $buttons } | ConvertTo-Json -Depth 10

while ($true) { # вечный цикл
    # вызываем функцию
    $returns = @()
    $returns = getUpdates $URL_get
    foreach ($return in $returns) {

    If ($return.text -eq "B") {
        $textt = "Hello, *$($return["f_name"])*! Please, choose the option you need:"
        $chat_id = $return.chat_id
        $chat_id
        $username = $return.username
        $keybrd = sendKeyboardMain $URL_set $buttonsmain $chat_id $textt
        }

        ElseIf ($return.callback_data -eq "Data2") {

        $text = "Please, $username choose the option you need:"
                $keybrd = sendKeyboardMain $URL_set $buttonsmain $chat_id $text

                }
    
    #Else {}
    Start-Sleep -s $timeout    
}
}

[MaxKozlov]

trlhrn,
Во первых, вот тут вы получаете только последнее сообщение - это то о чём я говорил выше
$data = $json.result | Select-Object -Last 1
Именно вместо Select-Object надо делать цикл подготовки ваших данных, а не готовить их в одном экземпляре, как сейчас

А во вторых, ошибка с $username у вас потому что вы его устанавливаете внутри If ($return.text -eq "B") {
То есть обработался один юзер, переменная установилась на него. Потом прислал "B" другой - переменная на него. А она одна на двоих. Об этом я тоже говорил, но немного другими словами

Вообще, чтобы всё получилось, вам надо построить свой код так, чтобы он не рассчитывал ни на какие циклы.
Получили в этот заход user_id, username - тут же его использовали, не считая что вы сможете это же использовать при следующем заходе.
Либо, если вам вот прям необходимо хранить какие-то данные между разными заходами, вам надо хранить их в hashtable.
типа

$userdata = @{}
while(1) {
 .....
if (-not $userdata.containskey($user_id)) { # Этот пользователь ещё не писал
  $userdata[$user_id] = [pscustomobject]@{ data = '12345'; data2 =  'lalala'}  # инициализация
}
...
if ($callback_data -eq 'want_answer') {
  $userdata[$user_id].data2 = $user_pressed_key # Записали что-нибудь
  $answer =  $userdata[$user_id].data # Считали что-нибудь
  sendMessage $user_id $answer
}
...

Естественно, нужно понимать, что при рестарте они потеряются