Skip to content

[bugfix/telegram]: fix shortening of long messages and sending the queue - #162

Merged
Schrolli91 merged 1 commit into
BOSWatch:developfrom
KoenigMjr:bugfix/telegram
Oct 21, 2026
Merged

Schrolli91 merged 1 commit into
BOSWatch:developfrom
KoenigMjr:bugfix/telegram

Conversation

@KoenigMjr

Copy link
Copy Markdown
Contributor

Textkürzung (_shorten_text) korrigiert:

  • MarkdownV2: Suffixe und Escape-Sequenzen werden nun fehlerfrei behandelt, um Syntaxfehler bei langen Nachrichten zu verhindern.
  • HTML: Mitten in einer Entity abgeschnittene HTML-Tags am Textende werden sauber erkannt und bereinigt, damit Telegram die Nachricht nicht ablehnt.

Sauberer Shutdown & Queue-Entleerung:

  • Die Reihenfolge beim Herunterfahren (shutdown) wurde optimiert: Der Stopp-Event wird erst gesetzt, nachdem die Nachrichtenschlange (msg_queue) leer ist oder das Timeout (5 Sekunden) abgelaufen ist.
  • Dadurch wird verhindert, dass verbleibende Nachrichten während des Herunterfahrens verworfen werden – sie werden stattdessen noch regulär versendet.

…eue on shutdown

- Long messages are shortened without cutting an escape sequence or an
  HTML entity, and the "[...]" suffix is escaped for MarkdownV2.
  Before, Telegram rejected these messages with HTTP 400 and they were
  discarded without retry.
- shutdown() lets the worker send the remaining messages before it is
  stopped. Before, the stop signal was set first, so all queued
  messages were discarded after waiting 5 seconds.

@Schrolli91 Schrolli91 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Schrolli91
Schrolli91 merged commit dfbac87 into BOSWatch:develop Oct 21, 2026
7 checks passed
@KoenigMjr
KoenigMjr deleted the bugfix/telegram branch October 24, 2026 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants