From d9002f050eb5034cbe69df5d7bff067a2ec9e8a1 Mon Sep 17 00:00:00 2001 From: greatroar <61184462+greatroar@users.noreply.github.com> Date: Sun, 27 Nov 2022 20:03:22 +0100 Subject: [PATCH] backend: Don't Wrap errors from url.Parse The messages from url.Error.Error already start with the word "parse". --- internal/backend/rest/config.go | 3 +-- internal/backend/s3/config.go | 2 +- internal/backend/sftp/config.go | 2 +- 3 files changed, 3 insertions(+), 4 deletions(-) diff --git a/internal/backend/rest/config.go b/internal/backend/rest/config.go index 51ff3b27c..388153fce 100644 --- a/internal/backend/rest/config.go +++ b/internal/backend/rest/config.go @@ -34,9 +34,8 @@ func ParseConfig(s string) (interface{}, error) { s = prepareURL(s) u, err := url.Parse(s) - if err != nil { - return nil, errors.Wrap(err, "url.Parse") + return nil, errors.WithStack(err) } cfg := NewConfig() diff --git a/internal/backend/s3/config.go b/internal/backend/s3/config.go index 775580450..9050e20f4 100644 --- a/internal/backend/s3/config.go +++ b/internal/backend/s3/config.go @@ -52,7 +52,7 @@ func ParseConfig(s string) (interface{}, error) { // bucket name and prefix url, err := url.Parse(s[3:]) if err != nil { - return nil, errors.Wrap(err, "url.Parse") + return nil, errors.WithStack(err) } if url.Path == "" { diff --git a/internal/backend/sftp/config.go b/internal/backend/sftp/config.go index 2f3b3697c..76d6d145d 100644 --- a/internal/backend/sftp/config.go +++ b/internal/backend/sftp/config.go @@ -42,7 +42,7 @@ func ParseConfig(s string) (interface{}, error) { // parse the "sftp://user@host/path" url format url, err := url.Parse(s) if err != nil { - return nil, errors.Wrap(err, "url.Parse") + return nil, errors.WithStack(err) } if url.User != nil { user = url.User.Username()