[Notifier] Escape . char for Telegram transport

symfony · derrabus · Jun 10, 2021 · Jun 8, 2021 · Jun 10, 2021 · 0aa0fcbbbcf049950ca3418c59e81cb0c7a1544b
commit 0aa0fcbbbcf049950ca3418c59e81cb0c7a1544b
@@ -79,8 +79,9 @@ protected function doSend(MessageInterface $message): SentMessage
 
         $options['text'] = $message->getSubject();
 
-        if (!isset($options['parse_mode'])) {
+        if (!isset($options['parse_mode']) || TelegramOptions::PARSE_MODE_MARKDOWN_V2 === $options['parse_mode']) {
             $options['parse_mode'] = TelegramOptions::PARSE_MODE_MARKDOWN_V2;
+            $options['text'] = str_replace('.', '\.', $message->getSubject());
         }
 
         $response = $this->client->request('POST', $endpoint, [

@@ -185,4 +185,56 @@ public function testSendWithChannelOverride()
         $this->assertEquals(1, $sentMessage->getMessageId());
         $this->assertEquals('telegram://api.telegram.org?channel=defaultChannel', $sentMessage->getTransport());
     }
+
+    public function testSendWithMarkdownShouldEscapeDots()
+    {
+        $response = $this->createMock(ResponseInterface::class);
+        $response->expects($this->exactly(2))
+            ->method('getStatusCode')
+            ->willReturn(200);
+
+        $content = <<<JSON
+            {
+                "ok": true,
+                "result": {
+                    "message_id": 1,
+                    "from": {
+                        "id": 12345678,
+                        "first_name": "YourBot",
+                        "username": "YourBot"
+                    },
+                    "chat": {
+                        "id": 1234567890,
+                        "first_name": "John",
+                        "last_name": "Doe",
+                        "username": "JohnDoe",
+                        "type": "private"
+                    },
+                    "date": 1459958199,
+                    "text": "Hello from Bot!"
+                }
+            }
+JSON;
+
+        $response->expects($this->once())
+            ->method('getContent')
+            ->willReturn($content)
+        ;
+
+        $expectedBody = [
+            'chat_id' => 'testChannel',
+            'text' => 'I contain a \.',
+            'parse_mode' => 'MarkdownV2',
+        ];
+
+        $client = new MockHttpClient(function (string $method, string $url, array $options = []) use ($response, $expectedBody): ResponseInterface {
+            $this->assertSame($expectedBody, json_decode($options['body'], true));
+
+            return $response;
+        });
+
+        $transport = $this->createTransport($client, 'testChannel');
+
+        $transport->send(new ChatMessage('I contain a .'));
+    }
 }