Closed
Description
This issue is automatically created based on existing pull request: #28928: [vcl] don't explicitly hash the host header
Hashing req.http.host
/client.ip
is already handled by the built-in
vcl
so there's no need to repeat it explicitly.
It's also a bit confusing as req.url
is not explicitly handled, even
though it's a more important hash input than the host.
note: all versions have been changed for the sake of consistency
but both the 4.x and 5.x series have been EOL'd a (long) while ago and users
should be encouraged to upgraded as soon as possible.
Contribution checklist (*)
- Pull request has a meaningful description of its purpose
- All commits are accompanied by meaningful commit messages
- All new or changed code is covered with unit/integration tests (if applicable)
- All automated tests passed successfully (all builds are green)
Metadata
Metadata
Assignees
Labels
The issue has been fixed in 2.4-develop branchGate 3 Passed. Manual verification of the issue completed. Issue is confirmedGate 4. Acknowledged. Issue is added to backlog and ready for developmentNo current plan to fix. Fixing can be deferred as a logical part of more important work.The issue has been reproduced on latest 2.4-develop branchAffects aesthetics, professional look and feel, “quality” or “usability”.
Type
Projects
Status
Done