kong icon indicating copy to clipboard operation
kong copied to clipboard

fix(http-log): add port information to the host header

Open dingjiayi opened this issue 1 year ago • 3 comments

Summary

Add port to host header when non-default port is used.The log server receives requests with a host header that does not include the port number.

Checklist

  • [x] The Pull Request has tests
  • [x] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
  • [ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix #13067

dingjiayi avatar May 28 '24 11:05 dingjiayi

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar May 28 '24 11:05 CLAassistant

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


dingjiayi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar May 28 '24 11:05 CLAassistant

Hi @ADD-SP . Could you please take a look at the PR ?

dingjiayi avatar May 29 '24 12:05 dingjiayi

CLA assistant check Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

dingjiayi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it.

Seems the CLA is failing as you used a different email for the commit, could you please check it to make sure the CLA is signed?

ADD-SP avatar May 30 '24 08:05 ADD-SP

CLA assistant check All committers have signed the CLA.

@ADD-SP Apologies for the confusion. I misunderstood the statement "All committers have signed the CLA," which led me to believe that I had already signed it. To ensure a clean PR process, I have closed the previous PR and created a new one. Kong/kong#13116

dingjiayi avatar May 30 '24 11:05 dingjiayi