mstar/goutils
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix header write order

Browse source 
Headers need to be written before status code
This commit is contained in:
Melody Becker 2025-04-09 13:11:12 +02:00
parent 2bc73d2262
commit b1bdc945b7
Signed by: mstar
SSH key fingerprint: SHA256:9VAo09aaVNTWKzPW7Hq2LW+ox9OdwmTSHRoD4mlz1yI
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
http/httpErr.go
View file

@ -32,8 +32,8 @@ func ProblemDetails(
details *string, details *string,
extras map[string]any, extras map[string]any,
) { ) {
w.WriteHeader(statusCode)
w.Header().Add("Content-Type", "application/problem+json") w.Header().Add("Content-Type", "application/problem+json")
w.WriteHeader(statusCode)
data := map[string]any{ data := map[string]any{
"type": errorType, "type": errorType,
"status": statusCode, "status": statusCode,
@ -58,8 +58,8 @@ func ProblemDetails(
// Write a simple problem details response. // Write a simple problem details response.
// It only provides the status code, as defined in RFC 9457, section 4.2.1 // It only provides the status code, as defined in RFC 9457, section 4.2.1
func ProblemDetailsStatusOnly(w http.ResponseWriter, statusCode int) { func ProblemDetailsStatusOnly(w http.ResponseWriter, statusCode int) {
w.WriteHeader(statusCode)
w.Header().Add("Content-Type", "application/problem+json") w.Header().Add("Content-Type", "application/problem+json")
w.WriteHeader(statusCode)
data := map[string]any{ data := map[string]any{
"type": "about:blank", "type": "about:blank",
"title": http.StatusText(statusCode), "title": http.StatusText(statusCode),