Move dispatch call out of HTTP try-catch

We use try...catch for HTTP control flow logic. We don't want to treat
runtime errors the same as HTTP errors though. Move `dispatch` call
out of that try catch.

x-pack/plugins/security_solution/public/resolver/store/middleware/index.ts

@@ -43,24 +43,26 @@ export const resolverMiddlewareFactory: MiddlewareFactory = (context) => {
         action.type === 'appDetectedMissingEventData'
       ) {
         const entityIdToFetchFor = action.payload;
-        let result: ResolverRelatedEvents;
+        let result: ResolverRelatedEvents | undefined;
         try {
           result = await context.services.http.get(
             `/api/endpoint/resolver/${entityIdToFetchFor}/events`,
             {
               query: { events: 100 },
             }
           );
+        } catch {
+          api.dispatch({
+            type: 'serverFailedToReturnRelatedEventData',
+            payload: action.payload,
+          });
+        }
 
+        if (result) {
           api.dispatch({
             type: 'serverReturnedRelatedEventData',
             payload: result,
           });
-        } catch (e) {
-          api.dispatch({
-            type: 'serverFailedToReturnRelatedEventData',
-            payload: action.payload,
-          });
         }
       }
     };