IntenseWebs/boringproxy
3
0
Fork 0
mirror of https://github.com/boringproxy/boringproxy.git synced 2025-02-25 18:55:29 -06:00
Code Issues Packages Projects Releases Wiki Activity

Fix auto TLS cert bug for Client TLS tunnels

Browse Source 
It wasn't grabbing certs automatically.
This commit is contained in:
Anders Pitman 2022-02-17 13:58:20 -07:00
parent 392a1ec8d7
commit 08e1beba6e
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
client.go
View File

@ -310,14 +310,6 @@ func (c *Client) BoreTunnel(ctx context.Context, tunnel Tunnel) error {
// boringproxy server does. // boringproxy server does.
go httpServer.Serve(tlsListener) go httpServer.Serve(tlsListener)
// TODO: There's still quite a bit of duplication with what the server does. Could we
// encapsulate it into a type?
err = c.certConfig.ManageSync(ctx, []string{tunnel.Domain})
if err != nil {
log.Println("CertMagic error at startup")
log.Println(err)
}
} else { } else {
if tunnel.TlsTermination == "client-tls" { if tunnel.TlsTermination == "client-tls" {
@ -349,6 +341,14 @@ func (c *Client) BoreTunnel(ctx context.Context, tunnel Tunnel) error {
}() }()
} }
// TODO: There's still quite a bit of duplication with what the server does. Could we
// encapsulate it into a type?
err = c.certConfig.ManageSync(ctx, []string{tunnel.Domain})
if err != nil {
log.Println("CertMagic error at startup")
log.Println(err)
}
<-ctx.Done() <-ctx.Done()
return nil return nil