fix: return 400 error if aggregation query is invalid

vernonjohnson · vernonjohnson · commit ad0fc3268625 · 2025-03-20T14:48:50.000-05:00
diff --git a/internal/handlers/transactions_handlers.go b/internal/handlers/transactions_handlers.go
@@ -158,7 +158,7 @@ func handleTransactionsRequest(c *gin.Context) {
 		if err != nil {
 			log.Error().Err(err).Msg("Error querying aggregates")
 			// TODO: might want to choose BadRequestError if it's due to not-allowed functions
-			api.InternalErrorHandler(c)
+			api.BadRequestErrorHandler(c, err)
 			return
 		}
 		queryResult.Aggregations = aggregatesResult.Aggregates

Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,7 @@ func handleTransactionsRequest(c *gin.Context) {`
`158`	`158`	`if err != nil {`
`159`	`159`	`log.Error().Err(err).Msg("Error querying aggregates")`
`160`	`160`	`// TODO: might want to choose BadRequestError if it's due to not-allowed functions`
`161`		`- api.InternalErrorHandler(c)`
	`161`	`+ api.BadRequestErrorHandler(c, err)`
`162`	`162`	`return`
`163`	`163`	`}`
`164`	`164`	`queryResult.Aggregations = aggregatesResult.Aggregates`