Merge pull request Squirrel#249 from willdean/patch-1

Log URLs correctly in FileDownloader
akrisiun · Feb 28, 2015 · 0e1487e · 0e1487e
2 parents 5b0f9fd + c944f9f
commit 0e1487e
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/Squirrel/FileDownloader.cs b/src/Squirrel/FileDownloader.cs
@@ -22,10 +22,10 @@ public async Task DownloadFile(string url, string targetFile, Action<int> progre
 
         retry:
             try {
-                this.Log().Info("Downloading file: " + failedUrl ?? url);
+                this.Log().Info("Downloading file: " + (failedUrl ?? url));
 
                 await this.WarnIfThrows(() => wc.DownloadFileTaskAsync(failedUrl ?? url, targetFile),
-                    "Failed downloading URL: " + failedUrl ?? url);
+                    "Failed downloading URL: " + (failedUrl ?? url));
             } catch (Exception) {
                 // NB: Some super brain-dead services are case-sensitive yet 
                 // corrupt case on upload. I can't even.
@@ -45,10 +45,10 @@ public async Task<byte[]> DownloadUrl(string url)
 
         retry:
             try {
-                this.Log().Info("Downloading url: " + failedUrl ?? url);
+                this.Log().Info("Downloading url: " + (failedUrl ?? url));
 
                 return await this.WarnIfThrows(() => wc.DownloadDataTaskAsync(failedUrl ?? url),
-                    "Failed to download url: " + failedUrl ?? url);
+                    "Failed to download url: " + (failedUrl ?? url));
             } catch (Exception) {
                 // NB: Some super brain-dead services are case-sensitive yet 
                 // corrupt case on upload. I can't even.