GetConsoleHTTPClient should not cache clientIP

[Timost]

NOTE

If this case is urgent, please subscribe to Subnet so that our 24/7 support team may help you faster.

Expected Behavior

Minio console forwards the correct clientIP to the minio-go client

Current Behavior

Minio console caches the first clientIP it sees and then reuses it along with the client even if the clientIP changed.

Possible Solution

Cache only the client (if possible) and set the clientIP every time in GetConsoleHTTPClient

Steps to Reproduce (for bugs)

See original issue (below)

Context

This a follow up to an issue/discussion on minio/minio: see minio/minio#17878 (comment)

Regression

No

Your Environment

Version used (minio --version): RELEASE.2023-08-04T17-40-21Z (didn't test with the latest version yet.
Server setup and configuration:
4 vm
1 podman pod with 1 minio container and 1 nginx container each
Operating System and version (uname -a): Host runs Ubuntu 22.04 with podman 4.6.0

[cesnietor]

issue might have been introduced by #2864 as mentioned in the other thread, we'll investigate.

[Timost]

Hi,
Would you consider it, if I opened a PR for that ?
Or is it too delicate for an outsider.

[cesnietor]

@Timost we are already working on it.
You're more than welcome to help us testing when we send a PR.
Also is an open source project so anyone can send contributions :D
Thanks.

[Timost]

Great news !
I'll be waiting for the PR then !

Thank you ! 🙏