[Java] fix question mark issue when getting available feedback widget list (#168)

* fix: change location of question mark

* fix: feedback widget qmark

* Update CHANGELOG.md

---------

Co-authored-by: Artūrs Kadiķis <[email protected]>

Author: arifBurakDemiray

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 23.10.1
+
+* Fixed a bug where getting the feedback widget list would fail if "salt" was enabled.
+
 ## 23.10.0
 
 * ! Minor breaking change ! Calling "init" twice will now not reinitialize the SDK. The call will be ignored

sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleFeedback.java

@@ -59,7 +59,7 @@ private void getAvailableFeedbackWidgetsInternal(CallbackOnFinish<List<CountlyFe
         String params = ModuleRequests.prepareRequiredParamsAsString(internalConfig, "method", "feedback");
         ImmediateRequestGenerator iRGenerator = internalConfig.immediateRequestGenerator;
 
-        iRGenerator.createImmediateRequestMaker().doWork("?" + params, "/o/sdk", transport, false, networkingIsEnabled, checkResponse -> {
+        iRGenerator.createImmediateRequestMaker().doWork(params, "/o/sdk?", transport, false, networkingIsEnabled, checkResponse -> {
             if (checkResponse == null) {
                 L.d("[ModuleFeedback] getAvailableFeedbackWidgetsInternal, Not possible to retrieve widget list. Probably due to lack of connection to the server");
                 callback.onFinished(null, "Not possible to retrieve widget list. Probably due to lack of connection to the server");

sdk-java/src/test/java/ly/count/sdk/java/internal/ModuleFeedbackTests.java

@@ -197,7 +197,7 @@ public void getAvailableFeedbackWidgets_base(List<CountlyFeedbackWidget> expecte
         ImmediateRequestI requestMaker = (requestData, customEndpoint, cp, requestShouldBeDelayed, networkingIsEnabled, callback, log) -> {
             Map<String, String> params = TestUtils.parseQueryParams(requestData);
             Assert.assertEquals("feedback", params.get("method"));
-            TestUtils.validateRequestMakerRequiredParams("/o/sdk", customEndpoint, requestShouldBeDelayed, networkingIsEnabled);
+            TestUtils.validateRequestMakerRequiredParams("/o/sdk?", customEndpoint, requestShouldBeDelayed, networkingIsEnabled);
             TestUtils.validateRequiredParams(params);
             callback.callback(returnedResponse);
         };