From 88e9f0190041facdc35d4b02c382e338a343270c Mon Sep 17 00:00:00 2001 From: Mattias Grenfeldt Date: Wed, 9 Sep 2020 13:13:23 +0200 Subject: [PATCH] Refactored big if-else chain into switch statement --- safehttp/plugins/cors/cors.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/safehttp/plugins/cors/cors.go b/safehttp/plugins/cors/cors.go index 446fe788..40c5dca3 100644 --- a/safehttp/plugins/cors/cors.go +++ b/safehttp/plugins/cors/cors.go @@ -120,12 +120,12 @@ func (it *Interceptor) Before(w *safehttp.ResponseWriter, r *safehttp.IncomingRe allowCredentials := h.Claim("Access-Control-Allow-Credentials") var status safehttp.StatusCode - m := r.Method() - if m == safehttp.MethodOptions { + switch r.Method() { + case safehttp.MethodOptions: status = it.preflight(w, r) - } else if m == safehttp.MethodHead { + case safehttp.MethodHead: status = safehttp.StatusMethodNotAllowed - } else { + default: status = it.request(w, r) }