Added await in async field initialization. Fixes #1701

PiranhaCMS · Oct 19, 2021 · f5f7c46 · f5f7c46
1 parent 4692832
commit f5f7c46
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/.gitignore b/.gitignore
@@ -17,6 +17,8 @@ examples/CoreWeb/assets/lib
 # Runtime test files
 [Uu]ploads
 *.db
+*.db-shm
+*.db-wal
 
 templates/**/wwwroot/uploads
 templates/**/*.db

diff --git a/core/Piranha/Runtime/AppMethod.cs b/core/Piranha/Runtime/AppMethod.cs
@@ -39,7 +39,7 @@ public sealed class AppMethod
         /// <param name="instance">The object instance</param>
         /// <param name="scope">The current service scope</param>
         /// <returns>An async taks</returns>
-        public Task InvokeAsync(object instance, IServiceScope scope)
+        public async Task InvokeAsync(object instance, IServiceScope scope)
         {
             var param = new List<object>();
             foreach (var type in ParameterTypes)
@@ -49,13 +49,12 @@ public Task InvokeAsync(object instance, IServiceScope scope)
 
             if (IsAsync)
             {
-                return (Task)Method.Invoke(instance, param.ToArray());
+                await ((Task)Method.Invoke(instance, param.ToArray()))
+                    .ConfigureAwait(false);
             }
             else
             {
-                return Task.Run(() => {
-                    Method.Invoke(instance, param.ToArray());
-                });
+                Method.Invoke(instance, param.ToArray());
             }
         }
     }