Skip to content

Commit

Permalink
Merge pull request #6 from blue-health/feature/add-more-packages
Browse files Browse the repository at this point in the history
Ensure our error map has hashable keys
  • Loading branch information
mpraski authored Sep 9, 2022
2 parents dcf9362 + 8642d77 commit bdef261
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions logger/error.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ type ValidationError struct {
Details validator.ValidationErrors
}

var errorMap = map[error]int{}
var errorMap = map[string]int{}

var (
matchFirstCap = regexp.MustCompile("(.)([A-Z][a-z]+)")
Expand All @@ -22,7 +22,7 @@ var (

func RegisterErrors(m map[error]int) {
for k, v := range m {
errorMap[k] = v
errorMap[k.Error()] = v
}
}

Expand Down
4 changes: 2 additions & 2 deletions logger/logger.go
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ func (l Logger) LogServiceError(w http.ResponseWriter, r *http.Request, e error)
f []apiField
)

if v, ok := errorMap[u]; ok {
if v, ok := errorMap[m]; ok {
s = v
}

Expand All @@ -76,7 +76,7 @@ func (l Logger) LogServiceError(w http.ResponseWriter, r *http.Request, e error)
n := unwrap(g.Root)
m = n.Error()

if v, ok := errorMap[n]; ok {
if v, ok := errorMap[m]; ok {
s = v
}
}
Expand Down

0 comments on commit bdef261

Please sign in to comment.