Fix: API Key added in the body instead of param (#142)

* Fix: Add the missing API Key on send chat calls.

* Fix: Wrongly added the apiKey in the body instead of the param

* Update TikTokLiveHttpClient.java

We do not need to null check as Eulerstream verifies anyway so we can just pass whatever the value is.

* Update TikTokLiveHttpClient.java

Revert to your method, but using Header instead of Param, since HttpClient does not allow null values!

Co-authored-by: mbayou <mathieu@novasquare.io>
Co-authored-by: kohlerpop1 <70915561+kohlerpop1@users.noreply.github.com>

Author: 3 people

Client/src/main/java/io/github/jwdeveloper/tiktok/TikTokLiveHttpClient.java

@@ -204,13 +204,11 @@ public boolean requestSendChat(LiveRoomInfo roomInfo, String content) {
         body.addProperty("sessionId", clientSettings.getSessionId());
         body.addProperty("ttTargetIdc", clientSettings.getTtTargetIdc());
         body.addProperty("roomId", roomInfo.getRoomId());
+        HttpClientBuilder builder = httpFactory.client(TIKTOK_CHAT_URL)
+            .withHeader("Content-Type", "application/json");
         if (clientSettings.getApiKey() != null)
-            body.addProperty("apiKey", clientSettings.getApiKey());
-        var result = httpFactory.client(TIKTOK_CHAT_URL)
-            .withHeader("Content-Type", "application/json")
-            .withBody(HttpRequest.BodyPublishers.ofString(body.toString()))
-            .build()
-            .toJsonResponse();
+            builder.withHeader("apiKey", clientSettings.getApiKey());
+        var result = builder.withBody(HttpRequest.BodyPublishers.ofString(body.toString())).build().toJsonResponse();
         return result.isSuccess();
     }
 
@@ -243,4 +241,4 @@ protected ActionResult<HttpResponse<byte[]>> getByteResponse(String room_id) {
 
         return result;
     }
-}
+}