Merge pull request #9753 from filecoin-project/gstuart/fix-devnet-deals

fix: cli: Check upper bound for deal duration
filecoin-project · Dec 12, 2022 · 33a176e · 33a176e
2 parents ab25809 + c2e4690
commit 33a176e
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/cli/client.go b/cli/client.go
@@ -627,7 +627,13 @@ uiLoop:
 
 			minDealDurationDays := uint64(build.MinDealDuration) / (builtin.SecondsInDay / build.BlockDelaySecs)
 			if days < int(minDealDurationDays) {
-				printErr(xerrors.Errorf("minimum duration is %d days", minDealDurationDays))
+				printErr(xerrors.Errorf("minimum duration is %d days, got %d", minDealDurationDays, days))
+				continue
+			}
+
+			maxDealDurationDays := uint64(build.MaxDealDuration) / (builtin.SecondsInDay / build.BlockDelaySecs)
+			if days > int(maxDealDurationDays) {
+				printErr(xerrors.Errorf("maximum duration is %d days, got %d", maxDealDurationDays, days))
 				continue
 			}