From 597de7a083c329bdaed7fc469555a4142f71bcb8 Mon Sep 17 00:00:00 2001
From: Sascha Grunert <sgrunert@redhat.com>
Date: Thu, 30 Jun 2022 16:47:21 +0200
Subject: libpod/runtime: switch to golang native error wrapping

We now use the golang error wrapping format specifier `%w` instead of
the deprecated github.com/pkg/errors package.

[NO NEW TESTS NEEDED]

Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
---
 pkg/api/handlers/utils/errors.go | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

(limited to 'pkg/api/handlers/utils')

diff --git a/pkg/api/handlers/utils/errors.go b/pkg/api/handlers/utils/errors.go
index bf60b2c84..ab1b6f227 100644
--- a/pkg/api/handlers/utils/errors.go
+++ b/pkg/api/handlers/utils/errors.go
@@ -1,17 +1,18 @@
 package utils
 
 import (
+	"errors"
+	"fmt"
 	"net/http"
 
 	"github.com/containers/podman/v4/libpod/define"
 	"github.com/containers/podman/v4/pkg/errorhandling"
 	"github.com/containers/storage"
-	"github.com/pkg/errors"
 	log "github.com/sirupsen/logrus"
 )
 
 var (
-	ErrLinkNotSupport = errors.New("Link is not supported")
+	ErrLinkNotSupport = errors.New("link is not supported")
 )
 
 // TODO: document the exported functions in this file and make them more
@@ -25,7 +26,7 @@ func Error(w http.ResponseWriter, code int, err error) {
 	// Log detailed message of what happened to machine running podman service
 	log.Infof("Request Failed(%s): %s", http.StatusText(code), err.Error())
 	em := errorhandling.ErrorModel{
-		Because:      (errors.Cause(err)).Error(),
+		Because:      errorhandling.Cause(err).Error(),
 		Message:      err.Error(),
 		ResponseCode: code,
 	}
@@ -33,51 +34,50 @@ func Error(w http.ResponseWriter, code int, err error) {
 }
 
 func VolumeNotFound(w http.ResponseWriter, name string, err error) {
-	if errors.Cause(err) != define.ErrNoSuchVolume {
+	if !errors.Is(err, define.ErrNoSuchVolume) {
 		InternalServerError(w, err)
 	}
 	Error(w, http.StatusNotFound, err)
 }
 
 func ContainerNotFound(w http.ResponseWriter, name string, err error) {
-	switch errors.Cause(err) {
-	case define.ErrNoSuchCtr, define.ErrCtrExists:
+	if errors.Is(err, define.ErrNoSuchCtr) || errors.Is(err, define.ErrCtrExists) {
 		Error(w, http.StatusNotFound, err)
-	default:
+	} else {
 		InternalServerError(w, err)
 	}
 }
 
 func ImageNotFound(w http.ResponseWriter, name string, err error) {
-	if errors.Cause(err) != storage.ErrImageUnknown {
+	if !errors.Is(err, storage.ErrImageUnknown) {
 		InternalServerError(w, err)
 	}
 	Error(w, http.StatusNotFound, err)
 }
 
 func NetworkNotFound(w http.ResponseWriter, name string, err error) {
-	if errors.Cause(err) != define.ErrNoSuchNetwork {
+	if !errors.Is(err, define.ErrNoSuchNetwork) {
 		InternalServerError(w, err)
 	}
 	Error(w, http.StatusNotFound, err)
 }
 
 func PodNotFound(w http.ResponseWriter, name string, err error) {
-	if errors.Cause(err) != define.ErrNoSuchPod {
+	if !errors.Is(err, define.ErrNoSuchPod) {
 		InternalServerError(w, err)
 	}
 	Error(w, http.StatusNotFound, err)
 }
 
 func SessionNotFound(w http.ResponseWriter, name string, err error) {
-	if errors.Cause(err) != define.ErrNoSuchExecSession {
+	if !errors.Is(err, define.ErrNoSuchExecSession) {
 		InternalServerError(w, err)
 	}
 	Error(w, http.StatusNotFound, err)
 }
 
 func SecretNotFound(w http.ResponseWriter, nameOrID string, err error) {
-	if errors.Cause(err).Error() != "no such secret" {
+	if errorhandling.Cause(err).Error() != "no such secret" {
 		InternalServerError(w, err)
 	}
 	Error(w, http.StatusNotFound, err)
@@ -92,7 +92,7 @@ func InternalServerError(w http.ResponseWriter, err error) {
 }
 
 func BadRequest(w http.ResponseWriter, key string, value string, err error) {
-	e := errors.Wrapf(err, "failed to parse query parameter '%s': %q", key, value)
+	e := fmt.Errorf("failed to parse query parameter '%s': %q: %w", key, value, err)
 	Error(w, http.StatusBadRequest, e)
 }
 
-- 
cgit v1.2.3-54-g00ecf