Skip to content

Commit a251a28

Browse files
committed
Properly extract JsonString results
toString leaves behind an extra pair of double quotes within the string, as opposed to returning the contents of the string.
1 parent 1a76ccb commit a251a28

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

workspace-server/src/main/java/gov/nasa/jpl/aerie/workspace/server/WorkspaceBindings.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -475,7 +475,7 @@ private void createFileDirectory(Context context) {
475475
}
476476

477477
if(uploadResults.response.getValueType() == JsonValue.ValueType.STRING) {
478-
context.status(uploadResults.status).result(uploadResults.response.toString());
478+
context.status(uploadResults.status).result(((JsonString) uploadResults.response()).getString());
479479
} else {
480480
context.status(uploadResults.status).json(uploadResults.response);
481481
}
@@ -540,7 +540,7 @@ && checkPermissions(context, body.destinationWorkspaceId(), WorkspaceAction.writ
540540
body.destinationWorkspaceId(),
541541
body.overwrite());
542542
if(moveResults.response.getValueType() == JsonValue.ValueType.STRING) {
543-
context.status(moveResults.status).result(moveResults.response.toString());
543+
context.status(moveResults.status).result(((JsonString) moveResults.response()).getString());
544544
} else {
545545
context.status(moveResults.status).json(moveResults.response);
546546
}
@@ -556,7 +556,7 @@ && checkPermissions(context, body.destinationWorkspaceId(), WorkspaceAction.writ
556556
body.destinationWorkspaceId(),
557557
body.overwrite());
558558
if (copyResults.response.getValueType() == JsonValue.ValueType.STRING) {
559-
context.status(copyResults.status).result(copyResults.response.toString());
559+
context.status(copyResults.status).result(((JsonString) copyResults.response()).getString());
560560
} else {
561561
context.status(copyResults.status).json(copyResults.response);
562562
}
@@ -576,7 +576,7 @@ private void deleteFileDirectory(Context context) {
576576
final var deleteResults = handleDelete(pathInfo.workspaceId, pathInfo.filePath);
577577

578578
if(deleteResults.response.getValueType() == JsonValue.ValueType.STRING) {
579-
context.status(deleteResults.status).result(deleteResults.response.toString());
579+
context.status(deleteResults.status).result(((JsonString) deleteResults.response()).getString());
580580
} else {
581581
context.status(deleteResults.status).json(deleteResults.response);
582582
}

0 commit comments

Comments
 (0)