Commitf517e5f

authored

fix(enterprise/cli): add missing defer close of closer funcs (#15986)

1 parent23b1c2d commitf517e5fCopy full SHA for f517e5f

File tree

-4

lines changed

-4

lines changed

Lines changed: 5 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -34,15 +34,15 @@ import (`
`34`	`34`	`"github.com/coder/serpent"`
`35`	`35`	`)`
`36`	`36`
`37`		`-typeclosers []func()`
	`37`	`+typecloserFuncs []func()`
`38`	`38`
`39`		`-func (cclosers)Close() {`
	`39`	`+func (ccloserFuncs)Close() {`
`40`	`40`	`for_,closeF:=rangec {`
`41`	`41`	`closeF()`
`42`	`42`	`}`
`43`	`43`	`}`
`44`	`44`
`45`		`-func (c*closers)Add(ffunc()) {`
	`45`	`+func (c*closerFuncs)Add(ffunc()) {`
`46`	`46`	`c=append(c,f)`
`47`	`47`	`}`
`48`	`48`
`@@ -113,7 +113,8 @@ func (r RootCmd) proxyServer() serpent.Command {`
`113`	`113`	`serpent.RequireNArgs(0),`
`114`	`114`	`),`
`115`	`115`	`Handler:func(inv*serpent.Invocation)error {`
`116`		`-varclosersclosers`
	`116`	`+varcloserscloserFuncs`
	`117`	`+deferclosers.Close()`
`117`	`118`	`// Main command context for managing cancellation of running`
`118`	`119`	`// services.`
`119`	`120`	`ctx,topCancel:=context.WithCancel(inv.Context())`

Comments

(0)