Fix: Fix client IP extraction by handling multiple IPs correctly

Separated IPs by comma without space in `Split` function. Updated to use the last IP after trimming spaces, ensuring proper client IP extraction. Signed-off-by: Christian Roessner <c@roessner.co>
croessner · Sep 17, 2024 · 56fe033 · 56fe033
1 parent 146eb65
commit 56fe033
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/server/util/util.go b/server/util/util.go
@@ -451,9 +451,9 @@ func ProcessXForwardedFor(ctx *gin.Context, clientIP, clientPort *string) {
 
 		*clientIP = fwdAddress
 
-		multipleIPs := strings.Split(fwdAddress, ", ")
+		multipleIPs := strings.Split(fwdAddress, ",")
 		if len(multipleIPs) > 1 {
-			*clientIP = multipleIPs[0]
+			*clientIP = strings.TrimSpace(multipleIPs[len(multipleIPs)-1])
 		}
 
 		*clientPort = global.NotAvailable