fix(query): show processlit host is null in HTTPQuery

[TCeason]

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

Fix show processlit host is null in HTTPQuery.

In HTTP auth , init the client addr.

• Fixes show processlist host is NULL #15359

Tests

• Logic Test

Type of change

• Bug Fix (non-breaking change which fixes an issue)

---

This change is 

[TCeason]

cc @wubx

[BohuTANG]

In cloud, the client ip will be gateway?

[TCeason]

In cloud, the client ip will be gateway?

cc @flaneur2020 @ZhiHanZ Cloud you please answer this question?

BTW, we had a method get_client_ip it use the same way to parse req.uri.

databend/src/query/service/src/servers/http/middleware.rs

Line 116 in 0d07cc6

pub fn get_client_ip(req: &Request) -> Option<String> {

[flaneur2020]

oh, i think using databend/src/query/service/src/servers/http/middleware.rs get_client_ip might be better

[flaneur2020]

In cloud, the client ip will be gateway?

not only cloud, whenever you put a proxy up front of the query instances, you will get the ip of the proxy.

when you had a proxy, it'd better to utilize the get_client_ip() method to get the real client ip which proxy added into the headers like X-Real-IP.

[TCeason]

In cloud, the client ip will be gateway?

not only cloud, whenever you put a proxy up front of the query instances, you will get the ip of the proxy.

when you had a proxy, it'd better to utilize the get_client_ip() method to get the real client ip which proxy added into the headers like X-Real-IP.

So in cloud it will display as null? Because only a IP can not parse as socketaddr.

#15359 (comment)

Here the host is ip:port

[flaneur2020]

In cloud, the client ip will be gateway?

not only cloud, whenever you put a proxy up front of the query instances, you will get the ip of the proxy.
when you had a proxy, it'd better to utilize the get_client_ip() method to get the real client ip which proxy added into the headers like X-Real-IP.

So in cloud it will display as null? Because only a IP can not parse as socketaddr.

#15359 (comment)

Here the host is ip:port

IMHO it will be filled up with the proxy's ip address 🤔

[TCeason]

In cloud, the client ip will be gateway?

not only cloud, whenever you put a proxy up front of the query instances, you will get the ip of the proxy.
when you had a proxy, it'd better to utilize the get_client_ip() method to get the real client ip which proxy added into the headers like X-Real-IP.

So in cloud it will display as null? Because only a IP can not parse as socketaddr.
#15359 (comment)
Here the host is ip:port

IMHO it will be filled up with the proxy's ip address 🤔

Does the show procelist need to display ports?

Now the get client ip address will be obtained from the header. But this is a pure ip, and it's represented by a string.

In our current session ctx, client_ip is Option.

This appears to be CK-compatible behavior. But the port should be meaningless information.

So you want to set session _ctx.client_ip: Option and discard the port.