Fix multiple ip issues on proxy servers

[lohanidamodar]

On a proxy server the header returns comma separated multiple IPs, where the first one is always the user's original IP.

[eldadfux]

Does that even make sense? Why do they return multiple IPs? Which proxy server did that?

[TorstenDittmann]

I would trim() the to be returned value.

From what I have read, there can be spaces 👍🏻

[TorstenDittmann]

Does that even make sense? Why do they return multiple IPs? Which proxy server did that?

https://serverfault.com/a/997805

[lohanidamodar]

Does that even make sense? Why do they return multiple IPs? Which proxy server did that?

I talked briefly in meeting, In Digitalocean Apps Platform, you can check it https://appwrite-lite-t22pl.ondigitalocean.app/

[eldadfux]

@lohanidamodar I would also split the statement to two part and validate the [0] key is actually their (just to be on the safe side) using isset or ?? operator. The fallback an be an empty string.

[eldadfux]

@lohanidamodar we should probably later add the same fix to utopia/framework request object

[lohanidamodar]

@lohanidamodar I would also split the statement to two part and validate the [0] key is actually their (just to be on the safe side) using isset or ?? operator. The fallback an be an empty string.

done

[lohanidamodar]

@lohanidamodar we should probably later add the same fix to utopia/framework request object

Will look into it

[eldadfux]

@lohanidamodar looks good, waiting for the tests to pass 👌🏻