Merge pull request #45 from hansdude/master

markpaulson · markpaulson · commit b6979d3dfb17 · 2014-09-30T15:33:41.000-06:00
Updated the Java client to escape the url.
diff --git a/sdk/src/main/java/com/spectralogic/ds3client/NetworkClientImpl.java b/sdk/src/main/java/com/spectralogic/ds3client/NetworkClientImpl.java
@@ -137,7 +137,7 @@ private HttpRequest buildHttpRequest() throws IOException {
         }
 
         private String buildPath() {
-            String path = this.ds3Request.getPath();
+            String path = UrlEscapers.urlFragmentEscaper().escape(this.ds3Request.getPath());
             final Map<String, String> queryParams = this.ds3Request.getQueryParams();
             if (!queryParams.isEmpty()) {
                 path += "?" + NetUtils.buildQueryString(queryParams);

Original file line number	Diff line number	Diff line change
`@@ -137,7 +137,7 @@ private HttpRequest buildHttpRequest() throws IOException {`
`137`	`137`	`}`
`138`	`138`
`139`	`139`	`private String buildPath() {`
`140`		`- String path = this.ds3Request.getPath();`
	`140`	`+ String path = UrlEscapers.urlFragmentEscaper().escape(this.ds3Request.getPath());`
`141`	`141`	`final Map<String, String> queryParams = this.ds3Request.getQueryParams();`
`142`	`142`	`if (!queryParams.isEmpty()) {`
`143`	`143`	`path += "?" + NetUtils.buildQueryString(queryParams);`