guqing's blog

What type of PR is this?

/kind improvement
/area core
/milestone 2.18.x

What this PR does / why we need it:

在 IpAddressUtils 中增加对 X-Forwarded-Proto 和 X-Forwarded-Host 请求头的支持

Which issue(s) this PR fixes:

Fixes #6255

Does this PR introduce a user-facing change?

None

从 developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Forwarded-Proto 和 developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Forwarded-Host 了解到，以上两个请求头并没有明确说会包含 IP 地址。我认为 Halo 这边不应该适配这两个请求头。

已经 hold 了，在验证用户的问题，应该确实不需要这个 PR halo-dev/halo#6255 (comment)

这个 canvas-confetti 应该已经没用了，可以去掉，它是在脚手架中放烟花用的,此建议不影响 PR 合并

/hold

那应该就是不是你说的这两个请求头哦，重新安装一下插件看一下请求头有什么是包含真实 IP 的
plugin-ip-1.0.0-SNAPSHOT.jar.zip

这个是雷池获取上游的的设置，比如腾讯云eo默认就是Tencent-Client-IP在这个地方就要设置Tencent-Client-IP

Hi @52lemon6 , 已经在 #6363 中添加了对这两个请求头的支持，但在合并之前，可以帮忙在你的 Halo 上安装这个插件
plugin-ip-1.0.0-SNAPSHOT.jar.zip
并访问 域名/echo-ip 来验证一下是否可以获取到正确的 IP 以验证这个改动的正确性吗

这个是雷池获取上游的的设置，比如腾讯云eo默认就是Tencent-Client-IP在这个地方就要设置Tencent-Client-IP
已在 #6363 中增加了对这两个请求头的支持，可以帮忙测试一下看你能否正确获取到真实 IP 吗

What type of PR is this?

/kind improvement
/area core
/milestone 2.18.x

What this PR does / why we need it:

在 IpAddressUtils 中增加对 X-Forwarded-Proto 和 X-Forwarded-Host 请求头的支持

Which issue(s) this PR fixes:

Fixes #6255

Does this PR introduce a user-facing change?

None

guqing/halo

✍ 一款优秀的开源博客发布应用。

希望增加X-Forwarded-Proto和X-Forwarded-Host，通过雷池代理的网站是通过这俩请求头传递上游IP的

Hi @52lemon6 , 看看能否通过配置的方式解决 https://waf-ce.chaitin.cn/docs/faq/other ，一般来说 X-Forwarded-For 和 X-Real-IP 才是最常用的