From 91c753ad2f625b1b417371ef8622191995c41b83 Mon Sep 17 00:00:00 2001 From: Andrey Smirnov Date: Thu, 2 Oct 2014 14:12:43 +0400 Subject: [PATCH] Add URL to all download errrors, so that they're easier to understand. #26 --- http/download.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/http/download.go b/http/download.go index 155f08fe..4332d56f 100644 --- a/http/download.go +++ b/http/download.go @@ -124,7 +124,7 @@ func (downloader *downloaderImpl) handleTask(task *downloadTask) { resp, err := downloader.client.Get(task.url) if err != nil { - task.result <- err + task.result <- fmt.Errorf("%s: %s", task.url, err) return } if resp.Body != nil { @@ -138,7 +138,7 @@ func (downloader *downloaderImpl) handleTask(task *downloadTask) { err = os.MkdirAll(filepath.Dir(task.destination), 0755) if err != nil { - task.result <- err + task.result <- fmt.Errorf("%s: %s", task.url, err) return } @@ -146,7 +146,7 @@ func (downloader *downloaderImpl) handleTask(task *downloadTask) { outfile, err := os.Create(temppath) if err != nil { - task.result <- err + task.result <- fmt.Errorf("%s: %s", task.url, err) return } defer outfile.Close() @@ -163,7 +163,7 @@ func (downloader *downloaderImpl) handleTask(task *downloadTask) { _, err = io.Copy(w, resp.Body) if err != nil { os.Remove(temppath) - task.result <- err + task.result <- fmt.Errorf("%s: %s", task.url, err) return } @@ -194,7 +194,7 @@ func (downloader *downloaderImpl) handleTask(task *downloadTask) { err = os.Rename(temppath, task.destination) if err != nil { os.Remove(temppath) - task.result <- err + task.result <- fmt.Errorf("%s: %s", task.url, err) return }