Merge pull request #35 from filecoin-project/fix/no-latest-on-export-failure

fix: do not update latest when export fails

Author: travisperson

cmd/filecoin-chain-archiver/cmds/create.go

@@ -323,6 +323,10 @@ var cmdCreate = &cli.Command{
 		if flagDiscard {
 			logger.Infow("discarding output")
 			io.Copy(ioutil.Discard, r)
+
+			if err := <-errCh; err != nil {
+				return err
+			}
 		} else {
 			host := u.Hostname()
 			port := u.Port()
@@ -365,6 +369,10 @@ var cmdCreate = &cli.Command{
 				"expiration_rule_id", info.ExpirationRuleID,
 			)
 
+			if err := <-errCh; err != nil {
+				return err
+			}
+
 			latestLocation, err := url.QueryUnescape(info.Location)
 			if err != nil {
 				logger.Errorw("failed to decode location url", "location", info.Location, "err", err)
@@ -401,10 +409,6 @@ var cmdCreate = &cli.Command{
 			)
 		}
 
-		if err := <-errCh; err != nil {
-			return err
-		}
-
 		logger.Infow("finished", "digiest", fmt.Sprintf("%x", h.Sum(nil)))
 
 		return nil