Handle multiple types here

Author: chrisronline

x-pack/plugins/alerting/server/lib/errors/es_error_parser.ts

@@ -47,12 +47,11 @@ const getEsCause = (obj: ErrorObject = {}, causes: string[] = []): string[] => {
 
 export const getEsErrorMessage = (error: Error) => {
   let message = error.message;
-  const apiError = (error as ElasticsearchError)?.meta?.body?.error;
+  const apiError =
+    (error as ElasticsearchError)?.error?.meta?.body?.error ??
+    (error as ElasticsearchError)?.meta?.body?.error;
   if (apiError) {
-    console.log({ apiError });
     message += `, caused by: "${getEsCause(apiError as ErrorObject)}"`;
-  } else {
-    console.log({ error });
   }
   return message;
 };

x-pack/plugins/alerting/server/lib/errors/types.ts

@@ -11,11 +11,15 @@ export interface ErrorThatHandlesItsOwnResponse extends Error {
   sendResponse(res: KibanaResponseFactory): IKibanaResponse;
 }
 
+interface ElasticsearchErrorMeta {
+  body?: {
+    error?: ApiError;
+  };
+}
+
 export interface ElasticsearchError extends Error {
-  // error?: ApiError;
-  meta?: {
-    body?: {
-      error?: ApiError;
-    };
+  error?: {
+    meta?: ElasticsearchErrorMeta;
   };
+  meta?: ElasticsearchErrorMeta;
 }