Skip to content

Commit

Permalink
Address feedback for PR #2308 - 3
Browse files Browse the repository at this point in the history
  • Loading branch information
Che Lin committed Oct 23, 2024
1 parent d39c355 commit 9f2190b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions cmd/options.go
Original file line number Diff line number Diff line change
Expand Up @@ -98,8 +98,8 @@ func WithLazyRefresh() Option {
}
}

// WithConnRefuseNotify configures the Proxy to start a goroutine and run the
// given notify callback function in the event of a connection refuse.
// WithConnRefuseNotify configures the Proxy to call the provided function when
// a connection is refused. The notification function is run in a goroutine.
func WithConnRefuseNotify(n func()) Option {
return func(c *Command) {
c.connRefuseNotify = n
Expand Down
2 changes: 1 addition & 1 deletion internal/healthcheck/healthcheck_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ func newProxyWithParams(t *testing.T, maxConns uint64, dialer cloudsql.Dialer, i
Instances: instances,
MaxConnections: maxConns,
}
p, err := proxy.NewClient(context.Background(), dialer, logger, c)
p, err := proxy.NewClient(context.Background(), dialer, logger, c, nil)
if err != nil {
t.Fatalf("proxy.NewClient: %v", err)
}
Expand Down

0 comments on commit 9f2190b

Please sign in to comment.