Merge pull request #420 from senacor/feature/remove_check_for_empty_r…

…esult Remove check for empty result
senacor · Sep 5, 2024 · 52b1840 · 52b1840
2 parents 6a29787 + 933cb68
commit 52b1840
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 8 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,8 @@
 # CHANGELOG
 
+## 3.0.3 (05.09.2024)
+- Remove check for empty result
+
 ## 3.0.2 (22.08.2024)
 - Disable Application Insights internal debug logging
 

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "@senacor/azure-function-middleware",
-    "version": "3.0.2",
+    "version": "3.0.3",
     "description": "Middleware for azure functions to handle authentication, authorization, error handling and logging",
     "main": "dist/index.js",
     "types": "dist/index.d.ts",

diff --git a/src/middleware.ts b/src/middleware.ts
@@ -21,7 +21,7 @@ const middlewareCore =
         handler: T,
         postExecution: (PostExecutionFunction<T> | false)[],
     ) =>
-    async (request: Parameters<T>, context: InvocationContext): Promise<ReturnType<T> | Error> => {
+    async (request: Parameters<T>, context: InvocationContext): Promise<ReturnType<T> | undefined | Error> => {
         let handlerResult: MiddlewareResult<ReturnType<T>> = { $failed: false, $result: undefined };
 
         if (beforeExecution) {
@@ -65,10 +65,6 @@ const middlewareCore =
             return handlerResult.$error;
         }
 
-        if (handlerResult.$result === undefined) {
-            return new Error('Illegal-State: Result of the handler was empty.');
-        }
-
         return handlerResult.$result;
     };