Hotfix (#722)

* Fixed: stackTrace:null in output when options.IncludeExceptionStackTraceInErrors = false

* Fixed: crash when using obfuscated IDs

Author: Bart Koelman

src/JsonApiDotNetCore/Models/JsonApiDocuments/ErrorMeta.cs

@@ -15,8 +15,15 @@ public sealed class ErrorMeta
 
         public void IncludeExceptionStackTrace(Exception exception)
         {
-            Data["StackTrace"] = exception?.Demystify().ToString()
-                .Split(new[] {"\n"}, int.MaxValue, StringSplitOptions.RemoveEmptyEntries);
+            if (exception == null)
+            {
+                Data.Remove("StackTrace");
+            }
+            else
+            {
+                Data["StackTrace"] = exception.Demystify().ToString()
+                    .Split(new[] { "\n" }, int.MaxValue, StringSplitOptions.RemoveEmptyEntries);
+            }
         }
     }
 }

src/JsonApiDotNetCore/Services/DefaultResourceService.cs

@@ -213,7 +213,7 @@ public virtual async Task<TResource> UpdateAsync(TId id, TResource requestEntity
             }
 
             _repository.FlushFromCache(databaseEntity);
-            TResource afterEntity = await _repository.Get(databaseEntity.Id).FirstOrDefaultAsync();
+            TResource afterEntity = await _repository.Get(id).FirstOrDefaultAsync();
             _resourceChangeTracker.SetFinallyStoredAttributeValues(afterEntity);
 
             bool hasImplicitChanges = _resourceChangeTracker.HasImplicitChanges();