mirror of https://github.com/apache/cloudstack.git
bug 6653: Extract Api's - No need to encode the url's forward slash with %2F as we have the latest version of gson which doesnt err out on forward slash anymore.
status 6653: resolved fixed
This commit is contained in:
parent
f6e2669a6e
commit
c009b56a60
|
|
@ -141,8 +141,7 @@ public class ExtractVolumeCmd extends BaseAsyncCmd {
|
|||
response.setUploadId(uploadId);
|
||||
response.setState(uploadInfo.getUploadState().toString());
|
||||
response.setAccountId(getEntityOwnerId());
|
||||
//FIX ME - Need to set the url once the gson jar is upgraded since it is throwing an error right now.
|
||||
response.setUrl(uploadInfo.getUploadUrl().replaceAll("/", "%2F"));
|
||||
response.setUrl(uploadInfo.getUploadUrl());
|
||||
this.setResponseObject(response);
|
||||
} else {
|
||||
throw new ServerApiException(BaseCmd.INTERNAL_ERROR, "Failed to extract volume");
|
||||
|
|
|
|||
|
|
@ -1631,9 +1631,7 @@ public class ApiResponseHelper implements ResponseGenerator {
|
|||
response.setUploadId(uploadId);
|
||||
response.setState(uploadInfo.getUploadState().toString());
|
||||
response.setAccountId(accountId);
|
||||
// FIX ME - Need to set the url once the gson jar is upgraded since it is throwing an error right now.
|
||||
// response.setUrl(uploadInfo.getUploadUrl());
|
||||
response.setUrl(uploadInfo.getUploadUrl().replaceAll("/", "%2F"));
|
||||
response.setUrl(uploadInfo.getUploadUrl());
|
||||
return response;
|
||||
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue