dashboard/app: bad inputs are client errors

google · Oct 1, 2024 · ea2b66a · ea2b66a
1 parent 5729ee3
commit ea2b66a
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/dashboard/app/graphs.go b/dashboard/app/graphs.go
@@ -216,11 +216,12 @@ func handleHeatmap(c context.Context, w http.ResponseWriter, r *http.Request, f
 		periodType = coveragedb.DayPeriod
 	}
 	if periodType != coveragedb.DayPeriod && periodType != coveragedb.MonthPeriod {
-		return fmt.Errorf("only day and month are allowed, but received %s instead", periodType)
+		return fmt.Errorf("only day and month are allowed, but received %s instead, %w",
+			periodType, ErrClientBadRequest)
 	}
 	pOps, err := coveragedb.PeriodOps(periodType)
 	if err != nil {
-		return err
+		return fmt.Errorf("%s: %w", err.Error(), ErrClientBadRequest)
 	}
 	periods := coveragedb.GenNPeriodsTill(12, civil.DateOf(time.Now()), pOps)
 	var style template.CSS