tandem/camper
tandem
/
camper
2
0
Fork 0
Code Issues 7 Pull Requests Packages Projects Releases Wiki Activity

Return HTTP 200 instead of HTTP 204 for payment notifications 

Redsys are a bunch of … something: they **only** recognize HTTP 200
as success; HTTP 204 apparently is an error for they, and don’t get
me started in not understanding what to do with an HTP 301.

Let’s just give in, and relax.
This commit is contained in:
jordi fita mas 2024-02-28 13:42:12 +01:00
parent 62d15970bf
commit 4e2df3f3c3
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
pkg/payment/public.go
View File

@ -251,7 +251,7 @@ func handleNotification(w http.ResponseWriter, r *http.Request, user *auth.User,
_ = sendEmail(r.Context(), conn, payment, company, user.Locale) /* shrug */ _ = sendEmail(r.Context(), conn, payment, company, user.Locale) /* shrug */
default: default:
} }
w.WriteHeader(http.StatusNoContent) w.WriteHeader(http.StatusOK)
default: default:
httplib.MethodNotAllowed(w, r, http.MethodPost) httplib.MethodNotAllowed(w, r, http.MethodPost)
} }