From bc1ac6ca5bb0c205372c6fae9c3779dcc4325546 Mon Sep 17 00:00:00 2001
From: Sonja Krause-Harder <sonja.krause-harder@elastic.co>
Date: Mon, 11 May 2020 15:24:08 +0200
Subject: [PATCH] Revert "Use logger in package install handler."

This reverts commit d31c8905be513449a11e01468c6459102df0c28b.
---
 .../ingest_manager/server/routes/epm/handlers.ts       | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/x-pack/plugins/ingest_manager/server/routes/epm/handlers.ts b/x-pack/plugins/ingest_manager/server/routes/epm/handlers.ts
index 6855f56bf7706b..fd3a9c520b90a4 100644
--- a/x-pack/plugins/ingest_manager/server/routes/epm/handlers.ts
+++ b/x-pack/plugins/ingest_manager/server/routes/epm/handlers.ts
@@ -27,7 +27,6 @@ import {
   installPackage,
   removeInstallation,
 } from '../../services/epm/packages';
-import { appContextService } from '../../services';
 
 export const getCategoriesHandler: RequestHandler = async (context, request, response) => {
   try {
@@ -122,12 +121,10 @@ export const getInfoHandler: RequestHandler<TypeOf<typeof GetInfoRequestSchema.p
 export const installPackageHandler: RequestHandler<TypeOf<
   typeof InstallPackageRequestSchema.params
 >> = async (context, request, response) => {
-  const logger = appContextService.getLogger();
-  const { pkgkey } = request.params;
-  const savedObjectsClient = context.core.savedObjects.client;
-  const callCluster = context.core.elasticsearch.adminClient.callAsCurrentUser;
-  if (logger) logger.info(`Attempting installation of package ${pkgkey}.`);
   try {
+    const { pkgkey } = request.params;
+    const savedObjectsClient = context.core.savedObjects.client;
+    const callCluster = context.core.elasticsearch.adminClient.callAsCurrentUser;
     const res = await installPackage({
       savedObjectsClient,
       pkgkey,
@@ -139,7 +136,6 @@ export const installPackageHandler: RequestHandler<TypeOf<
     };
     return response.ok({ body });
   } catch (e) {
-    if (logger) logger.error(`Error during installation of package ${pkgkey}: ${e}`);
     if (e.isBoom) {
       return response.customError({
         statusCode: e.output.statusCode,