Merge pull request #745 from juliar/android-file-path-encoding-fix

rpenfold · web-flow · commit 71fcebb48b92 · 2022-02-22T21:04:17.000-07:00
Fix addAndroidDownloads for paths containing %
diff --git a/android/src/main/java/com/RNFetchBlob/RNFetchBlobReq.java b/android/src/main/java/com/RNFetchBlob/RNFetchBlobReq.java
@@ -177,7 +177,7 @@ public void run() {
                     req.setDescription(options.addAndroidDownloads.getString("description"));
                 }
                 if(options.addAndroidDownloads.hasKey("path")) {
-                    req.setDestinationUri(Uri.parse("file://" + options.addAndroidDownloads.getString("path")));
+                    req.setDestinationUri(Uri.fromFile(new File(options.addAndroidDownloads.getString("path"))));
                 }
                 // #391 Add MIME type to the request
                 if(options.addAndroidDownloads.hasKey("mime")) {

Original file line number	Diff line number	Diff line change
`@@ -177,7 +177,7 @@ public void run() {`
`177`	`177`	`req.setDescription(options.addAndroidDownloads.getString("description"));`
`178`	`178`	`}`
`179`	`179`	`if(options.addAndroidDownloads.hasKey("path")) {`
`180`		`- req.setDestinationUri(Uri.parse("file://" + options.addAndroidDownloads.getString("path")));`
	`180`	`+ req.setDestinationUri(Uri.fromFile(new File(options.addAndroidDownloads.getString("path"))));`
`181`	`181`	`}`
`182`	`182`	`// #391 Add MIME type to the request`
`183`	`183`	`if(options.addAndroidDownloads.hasKey("mime")) {`