Skip to content

Commit d7a3fe8

Browse files
committed
chore: re-arrange the error message build
1 parent cc05ecd commit d7a3fe8

1 file changed

Lines changed: 8 additions & 3 deletions

File tree

VirtualFinland.UserAPI/src/VirtualFinland.UsersAPI/Middleware/ErrorHandlerMiddleware.cs

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ public async Task Invoke(HttpContext context)
3737

3838
ErrorResponseDetails errorResponseDetails = new()
3939
{
40-
Type = "InternalServerError",
41-
Message = error.Message
40+
Type = "",
41+
Message = ""
4242
};
4343

4444
switch (error)
@@ -47,20 +47,25 @@ public async Task Invoke(HttpContext context)
4747
// custom application error
4848
response.StatusCode = (int)HttpStatusCode.Unauthorized;
4949
errorResponseDetails.Type = "Unauthorized";
50+
errorResponseDetails.Message = error.Message ?? "Not authorized";
5051
break;
5152
case NotFoundException:
5253
// not found error
5354
response.StatusCode = (int)HttpStatusCode.NotFound;
5455
errorResponseDetails.Type = "NotFound";
56+
errorResponseDetails.Message = error.Message ?? "Not found";
5557
break;
56-
case BadRequestException e:
58+
case BadRequestException:
5759
// bad request error
5860
response.StatusCode = (int)HttpStatusCode.BadRequest;
5961
errorResponseDetails.Type = "BadRequest";
62+
errorResponseDetails.Message = error.Message ?? "Bad request";
6063
break;
6164
default:
6265
// unhandled error
6366
response.StatusCode = (int)HttpStatusCode.InternalServerError;
67+
errorResponseDetails.Type = "InternalServerError";
68+
errorResponseDetails.Message = error.Message ?? "Internal Server Error";
6469
break;
6570
}
6671

0 commit comments

Comments
 (0)