Skip to content

Commit 750fc0e

Browse files
yasinmirankjellp
authored andcommitted
fix(tsd-file-api-client): order of execution for code and message set
moved the statements to set code and message in the response after the fromJson method, so that it won't override the values set previously.
1 parent 7aa5ac4 commit 750fc0e

File tree

2 files changed

+15
-16
lines changed

2 files changed

+15
-16
lines changed

lib/tsd-file-api-client/src/main/java/no/uio/ifi/tc/TSDFileAPIClient.java

Lines changed: 12 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -142,13 +142,12 @@ public Message deleteFile(String token, String appId, String fileName) throws IO
142142
Message message = new Message();
143143

144144
try (Response response = client.newCall(request).execute()) {
145-
message.setStatusCode(response.code());
146-
message.setStatusText(response.message());
147-
148145
if (response.isSuccessful() && response.body() != null) {
149146
String responseBody = response.body().string();
150147
message = gson.fromJson(responseBody, Message.class);
151148
}
149+
message.setStatusCode(response.code());
150+
message.setStatusText(response.message());
152151
} catch (JsonSyntaxException e) {
153152
log.error(e.getMessage(), e);
154153
}
@@ -173,13 +172,12 @@ public ResumableUploads listResumableUploads(String token, String appId) throws
173172
ResumableUploads resumableUploads = new ResumableUploads();
174173

175174
try (Response response = client.newCall(request).execute()) {
176-
resumableUploads.setStatusCode(response.code());
177-
resumableUploads.setStatusText(response.message());
178-
179175
if (response.isSuccessful() && response.body() != null) {
180176
String responseBody = response.body().string();
181177
resumableUploads = gson.fromJson(responseBody, ResumableUploads.class);
182178
}
179+
resumableUploads.setStatusCode(response.code());
180+
resumableUploads.setStatusText(response.message());
183181
} catch (JsonSyntaxException e) {
184182
log.error(e.getMessage(), e);
185183
}
@@ -233,12 +231,12 @@ public Chunk initializeResumableUpload(
233231
Chunk chunkResponse = new Chunk();
234232

235233
try (Response response = client.newCall(request).execute()) {
236-
chunkResponse.setStatusCode(response.code());
237-
chunkResponse.setStatusText(response.message());
238234
String _body = Objects.requireNonNull(response.body()).string();
239235
if (response.isSuccessful() && _body != null) {
240236
chunkResponse = gson.fromJson(_body, Chunk.class);
241237
}
238+
chunkResponse.setStatusCode(response.code());
239+
chunkResponse.setStatusText(response.message());
242240
} catch (JsonSyntaxException e) {
243241
log.error(e.getMessage(), e);
244242
}
@@ -285,13 +283,12 @@ public Chunk uploadChunk(
285283
Chunk chunkResponse = new Chunk();
286284

287285
try (Response response = client.newCall(request).execute()) {
288-
chunkResponse.setStatusCode(response.code());
289-
chunkResponse.setStatusText(response.message());
290-
291286
if (response.isSuccessful() && response.body() != null) {
292287
String responseBody = response.body().string();
293288
chunkResponse = gson.fromJson(responseBody, Chunk.class);
294289
}
290+
chunkResponse.setStatusCode(response.code());
291+
chunkResponse.setStatusText(response.message());
295292
} catch (JsonSyntaxException e) {
296293
log.error(e.getMessage(), e);
297294
}
@@ -364,13 +361,12 @@ public Message deleteResumableUpload(String token, String appId, String uploadId
364361
Message message = new Message();
365362

366363
try (Response response = client.newCall(request).execute()) {
367-
message.setStatusCode(response.code());
368-
message.setStatusText(response.message());
369-
370364
if (response.isSuccessful() && response.body() != null) {
371365
String responseBody = response.body().string();
372366
message = gson.fromJson(responseBody, Message.class);
373367
}
368+
message.setStatusCode(response.code());
369+
message.setStatusText(response.message());
374370
} catch (JsonSyntaxException e) {
375371
log.error(e.getMessage(), e);
376372
}
@@ -442,14 +438,14 @@ public Token getToken(String tokenType, String oidcProvider, String idToken) thr
442438

443439
try {
444440
response = client.newCall(request).execute();
445-
token.setStatusCode(response.code());
446-
token.setStatusText(response.message());
447441
ResponseBody responseBody = response.body();
448442
if (response.isSuccessful() && responseBody != null) {
449443
String bodyString = responseBody.string();
450444
token.setToken(
451445
JsonParser.parseString(bodyString).getAsJsonObject().get("token").getAsString());
452446
}
447+
token.setStatusCode(response.code());
448+
token.setStatusText(response.message());
453449
response.close();
454450
} catch (Exception e) {
455451
log.error(e.getMessage(), e);

lib/tsd-file-api-client/src/main/java/no/uio/ifi/tc/model/pojo/TSDFileAPIResponse.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
package no.uio.ifi.tc.model.pojo;
22

3+
import com.fasterxml.jackson.annotation.JsonProperty;
34
import com.google.gson.annotations.SerializedName;
45
import lombok.Data;
56
import lombok.ToString;
@@ -8,9 +9,11 @@
89
@ToString
910
public class TSDFileAPIResponse {
1011

12+
@JsonProperty("statusCode")
1113
@SerializedName("statusCode")
1214
private int statusCode;
1315

16+
@JsonProperty("statusText")
1417
@SerializedName("statusText")
1518
private String statusText;
1619
}

0 commit comments

Comments
 (0)