luxolus
/
open-consul
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

Backport of NET-4897 - update comment to include the current issue url from the go team. into release/1.16.x (#18266) 

backport of commit 23f0cfc2d6955bcb599cca56b14dc76cc41d5320

Co-authored-by: John Murret <john.murret@hashicorp.com>
This commit is contained in:
hc-github-team-consul-core 2023-08-03 18:27:55 -04:00 committed by GitHub
parent 44ef42b4c2
commit 44460b5f95
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
api/api.go
View File

@ -1005,8 +1005,10 @@ func (r *request) toHTTP() (*http.Request, error) {
// this is required since go started validating req.host in 1.20.6 and 1.19.11.
// prior to that they would strip out the slashes for you. They removed that
// behavior and added more strict validation as part of a CVE.
// https://github.com/golang/go/issues/60374
// the hope is that
// This issue is being tracked by the Go team:
// https://github.com/golang/go/issues/61431
// If there is a resolution in this issue, we will remove this code.
// In the time being, this is the accepted workaround.
if strings.HasPrefix(r.url.Host, "/") {
r.url.Host = "localhost"
}