better logging for access management issues

Altinn · Dec 19, 2024 · a962424 · a962424
1 parent c4b4ef2
commit a962424
Showing 1 changed file with 24 additions and 0 deletions.
diff --git a/src/Altinn.App.Core/Internal/AccessManagement/AccessManagementClient.cs b/src/Altinn.App.Core/Internal/AccessManagement/AccessManagementClient.cs
@@ -7,6 +7,8 @@
 using Altinn.App.Core.Internal.AccessManagement.Models;
 using Altinn.App.Core.Internal.App;
 using Altinn.Common.AccessTokenClient.Services;
+using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Mvc;
 using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Options;
 
@@ -68,6 +70,28 @@ public async Task<DelegationResponse> DelegateRights(DelegationRequest delegatio
             }
             else
             {
+                try
+                {
+                    var problemDetails = JsonSerializer.Deserialize<ProblemDetails>(httpContent);
+                    if (problemDetails is not null)
+                    {
+                        logger.LogError(
+                            "Got error status code for access management request. Status code: {StatusCode}. Problem details: {ProblemDetails}",
+                            httpResponseMessage.StatusCode,
+                            problemDetails
+                        );
+                        throw new AccessManagementRequestException(
+                            "Got error status code for access management request.",
+                            problemDetails,
+                            httpResponseMessage.StatusCode,
+                            httpContent
+                        );
+                    }
+                }
+                catch (JsonException)
+                {
+                    response = null;
+                }
                 throw new HttpRequestException("Got error status code for access management request.");
             }
             return response;