Skip to content

Commit 7c9f6ed

Browse files
committed
Minor refactoring and adding 429 log line
1 parent f8fa0d0 commit 7c9f6ed

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

Adjust/adjust/src/main/java/com/adjust/sdk/RequestHandler.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -97,13 +97,13 @@ private void sendI(ActivityPackage activityPackage, int queueSize) {
9797
return;
9898
}
9999

100-
if (responseData.jsonResponse == null) {
101-
packageHandler.closeFirstPackage(responseData, activityPackage);
100+
if (responseData.trackingState == TrackingState.OPTED_OUT) {
101+
activityHandler.gotOptOutResponse();
102102
return;
103103
}
104104

105-
if (responseData.trackingState == TrackingState.OPTED_OUT) {
106-
activityHandler.gotOptOutResponse();
105+
if (responseData.jsonResponse == null) {
106+
packageHandler.closeFirstPackage(responseData, activityPackage);
107107
return;
108108
}
109109

Adjust/adjust/src/main/java/com/adjust/sdk/UtilNetworking.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -117,10 +117,6 @@ private static ResponseData readHttpResponse(HttpsURLConnection connection, Acti
117117
responseCode = connection.getResponseCode();
118118
InputStream inputStream;
119119

120-
if (responseCode == 429) {
121-
return responseData;
122-
}
123-
124120
if (responseCode >= 400) {
125121
inputStream = connection.getErrorStream();
126122
} else {
@@ -148,6 +144,10 @@ private static ResponseData readHttpResponse(HttpsURLConnection connection, Acti
148144
String stringResponse = sb.toString();
149145
logger.verbose("Response: %s", stringResponse);
150146

147+
if (responseCode == 429) {
148+
logger.error("Too frequent requests to the endpoint (429)");
149+
return responseData;
150+
}
151151
if (stringResponse == null || stringResponse.length() == 0) {
152152
return responseData;
153153
}

0 commit comments

Comments
 (0)