Fix logger

logger/error.go

@@ -13,7 +13,7 @@ type ValidationError struct {
 	Details validator.ValidationErrors
 }
 
-var errorMap = map[string]int{}
+var errorMap = map[error]int{}
 
 var (
 	matchFirstCap = regexp.MustCompile("(.)([A-Z][a-z]+)")
@@ -22,11 +22,11 @@ var (
 
 func RegisterErrors(m map[error]int) {
 	for k, v := range m {
-		errorMap[k.Error()] = v
+		errorMap[k] = v
 	}
 }
 
-func FailValidation(root, details error) ValidationError {
+func FailValidation(root, details error) *ValidationError {
 	var v ValidationError
 
 	v.Root = root
@@ -35,10 +35,10 @@ func FailValidation(root, details error) ValidationError {
 		v.Details = e
 	}
 
-	return v
+	return &v
 }
 
-func (e ValidationError) Error() string {
+func (e *ValidationError) Error() string {
 	var sb strings.Builder
 
 	if e.Root != nil {

logger/logger.go

@@ -58,24 +58,24 @@ func (l Logger) LogResponseMessage(w http.ResponseWriter, r *http.Request, s int
 func (l Logger) LogServiceError(w http.ResponseWriter, r *http.Request, e error) {
 	var (
 		u = unwrap(e)
-		m = e.Error()
+		m = u.Error()
 		s = http.StatusInternalServerError
 		f []apiField
 	)
 
-	if v, ok := errorMap[e.Error()]; ok {
+	if v, ok := errorMap[u]; ok {
 		s = v
 	}
 
 	switch g := u.(type) {
-	case ValidationError:
+	case *ValidationError:
 		e = g
 
 		if g.Root != nil {
 			n := unwrap(g.Root)
 			m = n.Error()
 
-			if v, ok := errorMap[n.Error()]; ok {
+			if v, ok := errorMap[n]; ok {
 				s = v
 			}
 		}

logger/logger_test.go

@@ -110,7 +110,7 @@ func TestLogServiceError(t *testing.T) {
 			name:       "validation error",
 			severity:   logging.Error,
 			statusCode: http.StatusInternalServerError,
-			err:        blueLogging.ValidationError{Root: errors.New("root error")},
+			err:        &blueLogging.ValidationError{Root: errors.New("root error")},
 			msg:        "root error",
 		},
 	}