From efb18ef75ec2bb9c08ebef7f5c029d8e7e103a95 Mon Sep 17 00:00:00 2001
From: Vernon Johnson <vtj2105@columbia.edu>
Date: Thu, 20 Mar 2025 14:48:50 -0500
Subject: [PATCH] fix: return 400 error if aggregation query is invalid

---
 internal/handlers/transactions_handlers.go | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/internal/handlers/transactions_handlers.go b/internal/handlers/transactions_handlers.go
index a94f91c..982a055 100644
--- a/internal/handlers/transactions_handlers.go
+++ b/internal/handlers/transactions_handlers.go
@@ -157,8 +157,7 @@ func handleTransactionsRequest(c *gin.Context) {
 		aggregatesResult, err := mainStorage.GetAggregations("transactions", qf)
 		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