Skip to content

Commit 57a3624

Browse files
committed
cmd/gomote: fix gomote dialing to the coordinator after recent cert changes
No more self-signed cert on https://farmer.golang.org, so don't do the custom TLS dialing anymore. Just use the standard tls.Dial. Updates golang/go#16442 Change-Id: I2e29cbde3294aaaa74c0e82150ffe985f3639209 Reviewed-on: https://go-review.googlesource.com/39750 Reviewed-by: Keith Randall <[email protected]>
1 parent da737d3 commit 57a3624

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

build.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,11 @@ func (ci CoordinatorInstance) TLSHostPort() (string, error) {
4040
}
4141

4242
func (ci CoordinatorInstance) TLSDialer() func(network, addr string) (net.Conn, error) {
43+
if ci == "prod" {
44+
// TODO(bradfitz): once the staging coordinator has a
45+
// DNS name and LetsEncrypt, delete this whole method?
46+
return nil // uses default http.Transport.DialTLS dialer
47+
}
4348
caPool := x509.NewCertPool()
4449
tlsConf := &tls.Config{
4550
ServerName: "go", // fixed name; see build.go

0 commit comments

Comments
 (0)