From 28534a1a53f04dc785f7b5c9dca82da2848bac94 Mon Sep 17 00:00:00 2001 From: Koala Yeung Date: Tue, 13 Oct 2020 12:19:15 +0800 Subject: [PATCH] session: Remove guard condition from map delete * delete() on a non-existed key in map does not cause any error. Remove the guarding conditions. --- session.go | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/session.go b/session.go index 9663235..563468e 100644 --- a/session.go +++ b/session.go @@ -160,19 +160,10 @@ func MapRemoteHost(inner SessionHandler) SessionHandler { // SCRIPT_NAME func FilterAuthReqParams(inner SessionHandler) SessionHandler { return func(client Client, req *Request) (*ResponsePipe, error) { - if _, ok := req.Params["CONTENT_LENGTH"]; ok { - delete(req.Params, "CONTENT_LENGTH") - } - if _, ok := req.Params["PATH_INFO"]; ok { - delete(req.Params, "PATH_INFO") - } - if _, ok := req.Params["PATH_TRANSLATED"]; ok { - delete(req.Params, "PATH_TRANSLATED") - } - if _, ok := req.Params["SCRIPT_NAME"]; ok { - delete(req.Params, "SCRIPT_NAME") - } - + delete(req.Params, "CONTENT_LENGTH") + delete(req.Params, "PATH_INFO") + delete(req.Params, "PATH_TRANSLATED") + delete(req.Params, "SCRIPT_NAME") return inner(client, req) } }