rationalize return logic in validateHeader()

This commit is contained in:
Notealot 2021-10-27 00:19:06 +08:00
parent 1f30270b0c
commit 513b59fa88

7
gin.go
View File

@ -416,9 +416,7 @@ func (engine *Engine) isTrustedProxy(ip net.IP) bool {
}
func (engine *Engine) validateHeader(header string) (clientIP string, valid bool) {
if header == "" {
return "", false
}
if header != "" {
items := strings.Split(header, ",")
for i := len(items) - 1; i >= 0; i-- {
ipStr := strings.TrimSpace(items[i])
@ -433,7 +431,8 @@ func (engine *Engine) validateHeader(header string) (clientIP string, valid bool
return ipStr, true
}
}
return
}
return "", false
}
// parseTrustedProxies parse Engine.trustedProxies to Engine.trustedCIDRs