backport of commit 5d4c9567a9

api/proxy/option.go

@@ -133,6 +133,9 @@ func WithSessionTeardownTimeout(with time.Duration) Option {
 }
 
 // WithApiClient provides an optional Boundary API client
+// Experimental: It is unclear whether the current usage of this option is the
+// approach that we want to take in the long term. This may be removed at any
+// point going forward.
 func WithApiClient(with *api.Client) Option {
 	return func(o *Options) error {
 		o.withApiClient = with

api/proxy/proxy.go

@@ -264,6 +264,9 @@ func (p *ClientProxy) Start(opt ...Option) (retErr error) {
 					p.cancel()
 					return
 				}
+
+				// TODO: Determine if this is useful or if there is a better approach
+				// that we may use in the long term.
 				if p.apiClient != nil {
 					// If we can tell that the session for the connection we just
 					// closed is terminated, we can close the listener, otherwise