From 410267b0df438944488d9b9348981f6adbbdad65 Mon Sep 17 00:00:00 2001
From: Pascal Precht <pascal.precht@gmail.com>
Date: Wed, 22 Jan 2020 13:52:43 +0100
Subject: [PATCH] Fix Embark logger usage

The MythX plugin passes a single method of embark's logger to other APIs which breaks their
usage as a logger instance context is required inside each logger method.

This commit changes the plugin to pass around the entire logger instance instead of just
its log method.
---
 mythx.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/mythx.js b/mythx.js
index a21a311..ec61a3a 100644
--- a/mythx.js
+++ b/mythx.js
@@ -90,7 +90,7 @@ async function getStatus(uuid, embark) {
     
     try {
         const results = await armletClient.getIssues(uuid);
-        return ghettoReport(embark.logger.info, results);
+        return ghettoReport(embark.logger, results);
     } catch (err) {
         embark.logger.warn(err);
         return 1;
@@ -172,13 +172,13 @@ function ghettoReport(logger, results) {
     });
     
     if (issuesCount === 0) {
-        logger('No issues found');
+        logger.info('No issues found');
         return 0;
     }
     for (const group of results) {
-        logger(group.sourceList.join(', ').underline);
+        logger.info(group.sourceList.join(', ').underline);
         for (const issue of group.issues) {
-            logger(yaml.safeDump(issue, {'skipInvalid': true}));
+            logger.info(yaml.safeDump(issue, {'skipInvalid': true}));
         }
     }
     return 1;