chore: simplify error handling in template push

ammario · ammario · commit f4510ada55c5 · 2023-02-06T20:20:56.000Z
diff --git a/cli/templatepush.go b/cli/templatepush.go
@@ -35,14 +35,10 @@ func (pf *templateUploadFlags) stdin() bool {
 
 func (pf *templateUploadFlags) upload(cmd *cobra.Command, client *codersdk.Client) (*codersdk.UploadResponse, error) {
 	var (
-		content   io.Reader
-		pipeErrCh = make(chan error, 1)
+		content io.Reader
 	)
 	if pf.stdin() {
 		content = cmd.InOrStdin()
-		// No piping if reading from stdin.
-		pipeErrCh <- nil
-		close(pipeErrCh)
 	} else {
 		prettyDir := prettyDirectoryPath(pf.directory)
 		_, err := cliui.Prompt(cmd, cliui.PromptOptions{
@@ -56,11 +52,11 @@ func (pf *templateUploadFlags) upload(cmd *cobra.Command, client *codersdk.Clien
 
 		pipeReader, pipeWriter := io.Pipe()
 		go func() {
-			defer pipeWriter.Close()
-			defer close(pipeErrCh)
 			bufWr := bufio.NewWriter(pipeWriter)
 			defer bufWr.Flush()
-			pipeErrCh <- provisionersdk.Tar(bufWr, pf.directory, provisionersdk.TemplateArchiveLimit)
+
+			err := provisionersdk.Tar(bufWr, pf.directory, provisionersdk.TemplateArchiveLimit)
+			_ = pipeWriter.CloseWithError(err)
 		}()
 		defer pipeReader.Close()
 		content = pipeReader
@@ -76,9 +72,6 @@ func (pf *templateUploadFlags) upload(cmd *cobra.Command, client *codersdk.Clien
 	if err != nil {
 		return nil, xerrors.Errorf("upload: %w", err)
 	}
-	if err = <-pipeErrCh; err != nil {
-		return nil, xerrors.Errorf("pipe: %w", err)
-	}
 	return &resp, nil
 }
 
