Skip to content

Commit 4079ca9

Browse files
committed
handle code 204 as success
1 parent 4e23c46 commit 4079ca9

1 file changed

Lines changed: 14 additions & 2 deletions

File tree

source/gameanalytics/GAHTTPApi.cpp

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -199,7 +199,7 @@ namespace gameanalytics
199199
res = curl_easy_perform(curl);
200200
if (res != CURLE_OK)
201201
{
202-
logging::GALogger::d(curl_easy_strerror(res));
202+
logging::GALogger::d("%s", curl_easy_strerror(res));
203203
return NoResponse;
204204
}
205205

@@ -211,13 +211,21 @@ namespace gameanalytics
211211

212212
EGAHTTPApiResponse requestResponseEnum = processRequestResponse(response_code, s.packet.data(), "Events");
213213

214+
const bool isValidResponse =
215+
requestResponseEnum == Ok || requestResponseEnum == Created || requestResponseEnum == NoContent;
216+
214217
// if not 200 result
215-
if (requestResponseEnum != Ok && requestResponseEnum != Created && requestResponseEnum != BadRequest)
218+
if (!isValidResponse && requestResponseEnum != BadRequest)
216219
{
217220
logging::GALogger::d("Failed Events Call. URL: %s, JSONString: %s, Authorization: %s", url.c_str(), jsonString.c_str(), authorization.data());
218221
return requestResponseEnum;
219222
}
220223

224+
if(requestResponseEnum == NoContent)
225+
{
226+
return requestResponseEnum;
227+
}
228+
221229
// decode JSON
222230
json requestJsonDict = json::parse(s.toString());
223231
if (requestJsonDict.is_null())
@@ -438,6 +446,10 @@ namespace gameanalytics
438446
{
439447
return Created;
440448
}
449+
if(statusCode == 204)
450+
{
451+
return NoContent;
452+
}
441453

442454
// 401 can return 0 status
443455
if (statusCode == 0 || statusCode == 401)

0 commit comments

Comments
 (0)