From 22201bf30a24dff7c4411999a81cc3e42ffa0942 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rui=20Tom=C3=A9?=
 <108268980+r-tome@users.noreply.github.com>
Date: Wed, 18 Jan 2023 09:56:08 +0000
Subject: [PATCH] [EC-619] Exceeding seat limit results in vague "Unhandled
 server error" message (#2558)

* [EC-619] Added 'AggregateException' type to ExceptionHandlerFilter to return error messages in response

* [EC-619] Updated ExceptionHandlerFilterAttribute to return multiple errors using ErrorResponseModel internal property
---
 .../Utilities/ExceptionHandlerFilterAttribute.cs    | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/src/Api/Utilities/ExceptionHandlerFilterAttribute.cs b/src/Api/Utilities/ExceptionHandlerFilterAttribute.cs
index 422bfa62d4..60506e46d0 100644
--- a/src/Api/Utilities/ExceptionHandlerFilterAttribute.cs
+++ b/src/Api/Utilities/ExceptionHandlerFilterAttribute.cs
@@ -92,6 +92,19 @@ public class ExceptionHandlerFilterAttribute : ExceptionFilterAttribute
             errorMessage = "Unauthorized.";
             context.HttpContext.Response.StatusCode = 401;
         }
+        else if (exception is AggregateException aggregateException)
+        {
+            context.HttpContext.Response.StatusCode = 400;
+            var errorValues = aggregateException.InnerExceptions.Select(ex => ex.Message);
+            if (_publicApi)
+            {
+                publicErrorModel = new ErrorResponseModel(errorMessage, errorValues);
+            }
+            else
+            {
+                internalErrorModel = new InternalApi.ErrorResponseModel(errorMessage, errorValues);
+            }
+        }
         else
         {
             var logger = context.HttpContext.RequestServices.GetRequiredService<ILogger<ExceptionHandlerFilterAttribute>>();