swarm/api: close tar writer in GetDirectoryTar to flush and clean (#17339)

3bcb501c · Janoš Guljaš · Balint Gabor · d3e4c2dc · 3bcb501c
Commit 3bcb501c authored Aug 09, 2018 by Janoš Guljaš Committed by Balint Gabor Aug 09, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

api.go swarm/api/api.go +4 -0

No files found.
--- a/swarm/api/api.go
+++ b/swarm/api/api.go
@@ -525,6 +525,10 @@ func (a *API) GetDirectoryTar(ctx context.Context, uri *URI) (io.ReadCloser, err

 			return nil
 		})
+		// close tar writer before closing pipew
+		// to flush remaining data to pipew
+		// regardless of error value
+		tw.Close()
 		if err != nil {
 			apiGetTarFail.Inc(1)
 			pipew.CloseWithError(err)