[Notifier] Improve Telegrams markdown escaping

codedge · codedge · commit aa38cf6d21ef · 2024-10-23T01:43:24.000+02:00
diff --git a/src/Symfony/Component/Notifier/Bridge/Telegram/TelegramTransport.php b/src/Symfony/Component/Notifier/Bridge/Telegram/TelegramTransport.php
@@ -89,7 +89,19 @@ protected function doSend(MessageInterface $message): SentMessage
 
         if (!isset($options['parse_mode']) || TelegramOptions::PARSE_MODE_MARKDOWN_V2 === $options['parse_mode']) {
             $options['parse_mode'] = TelegramOptions::PARSE_MODE_MARKDOWN_V2;
-            $text = preg_replace('/([_*\[\]()~`>#+\-=|{}.!\\\\])/', '\\\\$1', $text);
+            /*
+             * Just replace the obvious chars according to Telegram documentation.
+             * Do not try to find pairs or replace chars, that occur in pairs like
+             *  - *bold text*
+             *  - _italic text_
+             *  - __underlined text__
+             *  - various notations of images, f. ex. [title](url)
+             *  - `code samples`
+             * These formats should be takes care of, when the message is constructed.
+             *
+             * @see https://core.telegram.org/bots/api#markdownv2-style
+             */
+            $text = preg_replace('/([.!#>+-=|{}~])/', '\\\\$1', $text);
         }
 
         if (isset($options['upload'])) {
diff --git a/src/Symfony/Component/Notifier/Bridge/Telegram/Tests/TelegramTransportTest.php b/src/Symfony/Component/Notifier/Bridge/Telegram/Tests/TelegramTransportTest.php
@@ -265,7 +265,7 @@ public function testSendWithMarkdownShouldEscapeSpecialCharacters()
 
         $expectedBody = [
             'chat_id' => 'testChannel',
-            'text' => 'I contain special characters \_ \* \[ \] \( \) \~ \` \> \# \+ \- \= \| \{ \} \. \! \\\\ to send\.',
+            'text' => 'I contain special characters _ * [ ] ( ) \~ ` \> \# \+ \- \= \| \{ \} \. \! \ to send\.',
             'parse_mode' => 'MarkdownV2',
         ];
 
