Removed message splitting for large messages (Doesn't work well with HTML formatting), fixes #33.

Author: SeppPenner

Changelog.md

@@ -1,7 +1,7 @@
 Change history
 --------------
 
-* **Version 1.4.3.0 (2024-12-02)** : Removed support for Net6.0, added support for Net9.0, updated NuGet packages.
+* **Version 1.4.3.0 (2024-12-02)** : Removed support for Net6.0, added support for Net9.0, updated NuGet packages, removed message splitting for large messages (Doesn't work well with HTML formatting), fixes https://github.com/serilog-contrib/Serilog.Sinks.Telegram.Alternative/issues/33.
 * **Version 1.4.2.0 (2024-05-16)** : Removed support for Net7.0.
 * **Version 1.4.1.0 (2024-03-03)**: Updated NuGet packages.
 * **Version 1.4.0.0 (2023-11-21)**: Updated NuGet packages, removed support for netstandard, added support for Net8.0.

src/Serilog.Sinks.Telegram.Alternative/Serilog.Sinks.Telegram.Alternative.csproj

@@ -16,7 +16,7 @@
         <RepositoryUrl>https://github.com/serilog-contrib/Serilog.Sinks.Telegram.Alternative</RepositoryUrl>
         <PackageIcon>Icon.png</PackageIcon>
         <RepositoryType>Github</RepositoryType>
-        <PackageReleaseNotes>Version 1.4.3.0 (2024-12-02): Removed support for Net6.0, added support for Net9.0, updated NuGet packages.</PackageReleaseNotes>
+        <PackageReleaseNotes>Version 1.4.3.0 (2024-12-02): Removed support for Net6.0, added support for Net9.0, updated NuGet packages, removed message splitting for large messages (Doesn't work well with HTML formatting), fixes https://github.com/serilog-contrib/Serilog.Sinks.Telegram.Alternative/issues/33.</PackageReleaseNotes>
         <PackageLicenseExpression>MIT</PackageLicenseExpression>
         <LangVersion>latest</LangVersion>
         <Nullable>enable</Nullable>

src/Serilog.Sinks.Telegram.Alternative/Sinks/Telegram/Alternative/TelegramSink.cs

@@ -211,14 +211,17 @@ private async Task SendMessage(HttpClient httpClient, string? botApiUrl, string
 
         try
         {
-            foreach (var messageChunk in GetMessageChunks(message))
+            if (message.Length > MaxMessageLength)
             {
-                var response = await client.PostMessage(messageChunk, chatId, topicId);
+                this.TryWriteToSelflog($"Message is too long to be sent, it must not exceed {MaxMessageLength} characters.");
+                return;
+            }
 
-                if (response is not null)
-                {
-                    this.TryWriteToSelflog($"Message sent to chatId '{chatId}': '{response.StatusCode}'.");
-                }
+            var response = await client.PostMessage(message, chatId, topicId);
+
+            if (response is not null)
+            {
+                this.TryWriteToSelflog($"Message sent to chatId '{chatId}': '{response.StatusCode}'.");
             }
         }
         catch (Exception ex)
@@ -227,31 +230,7 @@ private async Task SendMessage(HttpClient httpClient, string? botApiUrl, string
             this.options.FailureCallback?.Invoke(ex);
         }
     }
-    
-    /// <summary>
-    /// Gets the message chunks.
-    /// </summary>
-    /// <param name="message">The message.</param>
-    /// <returns>A <see cref="IEnumerable{T}"/> of <see cref="string"/> containing the chunks.</returns>
-    private static IEnumerable<string> GetMessageChunks(string message)
-    {
-        if (message.Length <= MaxMessageLength)
-        {
-            return new[] { message };
-        }
-
-        var chunksCount = (message.Length + MaxMessageLength - 1) / MaxMessageLength;
-        var result = new string[chunksCount];
-        
-        for (var i = 0; i < chunksCount; i++)
-        {
-            var isLastChunk = chunksCount - i == 1;
-            var chunkLength = isLastChunk ? message.Length % MaxMessageLength : MaxMessageLength;
-            result[i] = message.Substring(i * MaxMessageLength, chunkLength);
-        }
-
-        return result;
-    }
+   
 
     /// <summary>
     /// Tries to write to Selflog and throws an exception if a formatting error occurred.