From 96254b52cdc3759962d199a8763c589c0992e504 Mon Sep 17 00:00:00 2001
From: Paul Schneider <paul@pschneider.fr>
Date: Wed, 16 Sep 2026 19:48:13 +0100
Subject: [PATCH] Supprime un faux message d'erreur

---
 src/PostIt/PostIt/Services/YavscApiClient.cs | 34 +++++++++++++++-----
 1 file changed, 26 insertions(+), 8 deletions(-)

diff --git a/src/PostIt/PostIt/Services/YavscApiClient.cs b/src/PostIt/PostIt/Services/YavscApiClient.cs
index 7e3187d8b..04dbce1fd 100644
--- a/src/PostIt/PostIt/Services/YavscApiClient.cs
+++ b/src/PostIt/PostIt/Services/YavscApiClient.cs
@@ -194,10 +194,7 @@ public class YavscApiClient : IYavscApiClient, IAsyncDisposable
         using var response = await SendAsync(method, path, body is null ? null : () => JsonContent.Create(body), ct).ConfigureAwait(false);
         await EnsureSuccessOrThrowAsync(response, ct).ConfigureAwait(false);
 
-        var stream = await response.Content.ReadAsStreamAsync(ct).ConfigureAwait(false);
-        var dto = await JsonSerializer.DeserializeAsync<T>(stream,
-            new JsonSerializerOptions { PropertyNameCaseInsensitive = true }, ct).ConfigureAwait(false);
-        return dto!;
+        return await ReadJsonAsync<T>(response, ct).ConfigureAwait(false);
     }
 
     /// <summary>
@@ -215,10 +212,31 @@ public class YavscApiClient : IYavscApiClient, IAsyncDisposable
         using var response = await SendAsync(method, path, contentFactory, ct).ConfigureAwait(false);
         await EnsureSuccessOrThrowAsync(response, ct).ConfigureAwait(false);
 
-        var stream = await response.Content.ReadAsStreamAsync(ct).ConfigureAwait(false);
-        var dto = await JsonSerializer.DeserializeAsync<T>(stream,
-            new JsonSerializerOptions { PropertyNameCaseInsensitive = true }, ct).ConfigureAwait(false);
-        return dto!;
+        return await ReadJsonAsync<T>(response, ct).ConfigureAwait(false);
+    }
+
+    /// <summary>
+    /// Deserialise the response body as <typeparamref name="T"/>,
+    /// tolerating an empty body. <c>PUT /blogspot/{id}</c> answers
+    /// <c>204 No Content</c>: without this guard,
+    /// <c>JsonSerializer.DeserializeAsync</c> throws
+    /// <c>JsonException: The input does not contain any JSON
+    /// tokens</c> and the VM surfaces a successful save as an
+    /// error. Callers that genuinely need a payload
+    /// (<c>GetPostAsync</c>, <c>CreatePostAsync</c>) are unaffected:
+    /// their endpoints always return a body.
+    /// </summary>
+    private static async Task<T> ReadJsonAsync<T>(HttpResponseMessage response, CancellationToken ct)
+    {
+        if (response.StatusCode == HttpStatusCode.NoContent)
+            return default!;
+
+        var payload = await response.Content.ReadAsStringAsync(ct).ConfigureAwait(false);
+        if (string.IsNullOrWhiteSpace(payload))
+            return default!;
+
+        return JsonSerializer.Deserialize<T>(payload,
+            new JsonSerializerOptions { PropertyNameCaseInsensitive = true })!;
     }
 
     /// <summary>