fix: #620

Author: maurei

src/JsonApiDotNetCore/Configuration/JsonApiOptions.cs

@@ -32,12 +32,12 @@ public class JsonApiOptions : IJsonApiOptions
         /// <summary>
         /// Whether or not stack traces should be serialized in Error objects
         /// </summary>
-        public static bool DisableErrorStackTraces { get; set; }
+        public static bool DisableErrorStackTraces { get; set; } = true;
 
         /// <summary>
         /// Whether or not source URLs should be serialized in Error objects
         /// </summary>
-        public static bool DisableErrorSource { get; set; }
+        public static bool DisableErrorSource { get; set; } = true;
 
         /// <summary>
         /// Whether or not ResourceHooks are enabled. 

src/JsonApiDotNetCore/Controllers/BaseJsonApiController.cs

@@ -141,7 +141,7 @@ public virtual async Task<IActionResult> PostAsync([FromBody] T entity)
                 return Forbidden();
 
             if (_jsonApiOptions.ValidateModelState && !ModelState.IsValid)
-                return UnprocessableEntity(ModelState.ConvertToErrorCollection<T>(GetAssociatedResource()));
+                return UnprocessableEntity(ModelState.ConvertToErrorCollection<T>());
 
             entity = await _create.CreateAsync(entity);
 
@@ -155,7 +155,7 @@ public virtual async Task<IActionResult> PatchAsync(TId id, [FromBody] T entity)
                 return UnprocessableEntity();
 
             if (_jsonApiOptions.ValidateModelState && !ModelState.IsValid)
-                return UnprocessableEntity(ModelState.ConvertToErrorCollection<T>(GetAssociatedResource()));
+                return UnprocessableEntity(ModelState.ConvertToErrorCollection<T>());
 
             var updatedEntity = await _update.UpdateAsync(id, entity);
 
@@ -181,12 +181,12 @@ public virtual async Task<IActionResult> DeleteAsync(TId id)
             return NoContent();
         }
 
-        internal Type GetAssociatedResource()
-        {
-            return GetType().GetMethod(nameof(GetAssociatedResource), BindingFlags.Instance | BindingFlags.NonPublic)
-                            .DeclaringType
-                            .GetGenericArguments()[0];
-        }
+        //internal Type GetAssociatedResource()
+        //{
+        //    return GetType().GetMethod(nameof(GetAssociatedResource), BindingFlags.Instance | BindingFlags.NonPublic)
+        //                    .DeclaringType
+        //                    .GetGenericArguments()[0];
+        //}
     }
     public class BaseJsonApiController<T>
     : BaseJsonApiController<T, int>

src/JsonApiDotNetCore/Extensions/IApplicationBuilderExtensions.cs

@@ -6,6 +6,7 @@
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Hosting;
 using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.Hosting;
 using Microsoft.Extensions.Logging;
 
 namespace JsonApiDotNetCore.Extensions
@@ -20,7 +21,6 @@ public static class IApplicationBuilderExtensions
         /// <returns></returns>
         public static void UseJsonApi(this IApplicationBuilder app)
         {
-            DisableDetailedErrorsIfProduction(app);
             LogResourceGraphValidations(app);
             using (var scope = app.ApplicationServices.CreateScope())
             {
@@ -38,14 +38,10 @@ public static void UseJsonApi(this IApplicationBuilder app)
             app.UseEndpoints(endpoints => endpoints.MapControllers());
         }
 
-        private static void DisableDetailedErrorsIfProduction(IApplicationBuilder app)
+        public static void EnableDetailedErrors(this IApplicationBuilder app)
         {
-            var webHostEnvironment = (IWebHostEnvironment) app.ApplicationServices.GetService(typeof(IWebHostEnvironment));
-            if (webHostEnvironment.EnvironmentName == "Production")
-            {
-                JsonApiOptions.DisableErrorStackTraces = true;
-                JsonApiOptions.DisableErrorSource = true;
-            }
+            JsonApiOptions.DisableErrorStackTraces = false;
+            JsonApiOptions.DisableErrorSource = false;
         }
 
         private static void LogResourceGraphValidations(IApplicationBuilder app)

src/JsonApiDotNetCore/Extensions/ModelStateExtensions.cs

@@ -9,7 +9,7 @@ namespace JsonApiDotNetCore.Extensions
 {
     public static class ModelStateExtensions
     {
-        public static ErrorCollection ConvertToErrorCollection<T>(this ModelStateDictionary modelState, Type resourceType)
+        public static ErrorCollection ConvertToErrorCollection<T>(this ModelStateDictionary modelState) where T : class, IIdentifiable
         {
             ErrorCollection collection = new ErrorCollection();
             foreach (var entry in modelState)
@@ -19,7 +19,7 @@ public static ErrorCollection ConvertToErrorCollection<T>(this ModelStateDiction
                     continue;
                 }
 
-                var targetedProperty = resourceType.GetProperty(entry.Key);
+                var targetedProperty = typeof(T).GetProperty(entry.Key);
                 var attrName = targetedProperty.GetCustomAttribute<AttrAttribute>().PublicAttributeName;
 
                 foreach (var modelError in entry.Value.Errors)

src/JsonApiDotNetCore/Formatters/JsonApiWriter.cs

@@ -3,6 +3,7 @@
 using System.Threading.Tasks;
 using JsonApiDotNetCore.Internal;
 using JsonApiDotNetCore.Serialization.Server;
+using Microsoft.AspNetCore.Mvc;
 using Microsoft.AspNetCore.Mvc.Formatters;
 using Microsoft.Extensions.Logging;
 using Newtonsoft.Json;
@@ -32,33 +33,39 @@ public async Task WriteAsync(OutputFormatterWriteContext context)
                 throw new ArgumentNullException(nameof(context));
 
             var response = context.HttpContext.Response;
-            using (var writer = context.WriterFactory(response.Body, Encoding.UTF8))
+            using var writer = context.WriterFactory(response.Body, Encoding.UTF8);
+            string responseContent;
+
+            if (_serializer == null)
+            {
+                responseContent = JsonConvert.SerializeObject(context.Object);
+            }
+            else
             {
                 response.ContentType = Constants.ContentType;
-                string responseContent;
-                if (_serializer == null)
+                try
                 {
-                    responseContent = JsonConvert.SerializeObject(context.Object);
-                }
-                else
-                {
-                    try
+                    if (context.Object is ProblemDetails pd)
                     {
-                        responseContent = _serializer.Serialize(context.Object);
-                    }
-                    catch (Exception e)
-                    {
-                        _logger?.LogError(new EventId(), e, "An error ocurred while formatting the response");
                         var errors = new ErrorCollection();
-                        errors.Add(new Error(400, e.Message, ErrorMeta.FromException(e)));
+                        errors.Add(new Error(pd.Status.Value, pd.Title, pd.Detail));
                         responseContent = _serializer.Serialize(errors);
-                        response.StatusCode = 400;
+                    } else
+                    {
+                        responseContent = _serializer.Serialize(context.Object);
                     }
                 }
-
-                await writer.WriteAsync(responseContent);
-                await writer.FlushAsync();
+                catch (Exception e)
+                {
+                    _logger?.LogError(new EventId(), e, "An error ocurred while formatting the response");
+                    var errors = new ErrorCollection();
+                    errors.Add(new Error(500, e.Message, ErrorMeta.FromException(e)));
+                    responseContent = _serializer.Serialize(errors);
+                    response.StatusCode = 500;
+                }
             }
+            await writer.WriteAsync(responseContent);
+            await writer.FlushAsync();
         }
     }
 }