Skip to content

Commit

Permalink
The Host part of the URL doesn't actually get used when we initiate c…
Browse files Browse the repository at this point in the history
…onnections to UNIX domain sockets. As of golang/go#60374 (in the latest Go releases at the time of this writing), we must set it to something that looks like a hostname or requests will fail.
  • Loading branch information
hc-github-team-secure-vault-core committed Aug 23, 2023
1 parent e4ce872 commit 77566cc
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion api/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -545,7 +545,7 @@ func (c *Config) ParseAddress(address string) (*url.URL, error) {
// be pointing to the protocol used in the application layer and not to
// the transport layer. Hence, setting the fields accordingly.
u.Scheme = "http"
u.Host = socket
u.Host = "localhost"
u.Path = ""
} else {
return nil, fmt.Errorf("attempting to specify unix:// address with non-transport transport")
Expand Down

0 comments on commit 77566cc

Please sign in to comment.