chore: error mgt for JSON problem

ncarlier · ncarlier · commit f46023ceb4b0 · 2024-08-04T05:58:48.000Z
diff --git a/pkg/utils/json-problem.go b/pkg/utils/json-problem.go
@@ -2,6 +2,7 @@ package utils
 
 import (
 	"encoding/json"
+	"fmt"
 	"net/http"
 )
 
@@ -24,5 +25,7 @@ func WriteJSONProblem(w http.ResponseWriter, problem JSONProblem) {
 	w.Header().Set("Content-Type", "application/problem+json; charset=utf-8")
 	w.Header().Set("X-Content-Type-Options", "nosniff")
 	w.WriteHeader(problem.Status)
-	json.NewEncoder(w).Encode(problem)
+	if err := json.NewEncoder(w).Encode(problem); err != nil {
+		fmt.Fprintf(w, `{"title": "%d", "detail": "%s"}`, http.StatusInternalServerError, err.Error())
+	}
 }

Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,7 @@ package utils`
`2`	`2`
`3`	`3`	`import (`
`4`	`4`	`"encoding/json"`
	`5`	`+ "fmt"`
`5`	`6`	`"net/http"`
`6`	`7`	`)`
`7`	`8`
`@@ -24,5 +25,7 @@ func WriteJSONProblem(w http.ResponseWriter, problem JSONProblem) {`
`24`	`25`	`w.Header().Set("Content-Type", "application/problem+json; charset=utf-8")`
`25`	`26`	`w.Header().Set("X-Content-Type-Options", "nosniff")`
`26`	`27`	`w.WriteHeader(problem.Status)`
`27`		`- json.NewEncoder(w).Encode(problem)`
	`28`	`+ if err := json.NewEncoder(w).Encode(problem); err != nil {`
	`29`	+ fmt.Fprintf(w, `{"title": "%d", "detail": "%s"}`, http.StatusInternalServerError, err.Error())
	`30`	`+ }`
`28`	`31`	`}`