remove rogue console.logs

lukman-paystack · Mar 16, 2021 · Apr 4, 2021 · Apr 4, 2021 · Apr 4, 2021 · Apr 13, 2021
commit 3af0523cfff00b87db8961df32fa92f7508ffd7c
diff --git a/postmanToAPI.js b/postmanToAPI.js
@@ -20,15 +20,12 @@ collection.item.forEach((APIObject) => {
                     subcommand_name = _subObject.name.toLowerCase();
                     subcommand_name = subcommand_name.replace(/ /g, '');
                     if (isAPIObject(_subObject)) {
-                        //add to API collection
                         let formattedAPI = createAPIObject(_subObject, command_name);
                         parsedAPIs[command_name].push(formattedAPI);
-                        // console.log(Object.keys(_subObject))
                     } else {
                         // console.log('Naah', _subObject)
                     }
                 })
-                // console.log('sub-'+subcommand_name, subObject.item.length);
             } else {
                 //add to API collection
                 let formattedAPI = createAPIObject(subObject, command_name);

diff --git a/src/commands/webhook.js b/src/commands/webhook.js
@@ -32,10 +32,12 @@ class WebhookCommand extends Command {
       if (!flags.forward) {
         this.error('To listen to webhook events locally, you have to specify a local route to forward events to using the forward flag e.g --forward localhost:3000/webhook')
       }
-      let urlObject = helpers.parseURL(flags.forward)
-
+      let urlObject = helpers.parseURL(flags.forward);
+      if(urlObject.hostname !== 'localhost' && urlObject.hostname !== '127.0.0.1'){
+        return helpers.errorLog(`Invalid host provided "${urlObject.hostname}" - You can only forward events to localhost`);
+      }
       if (!urlObject.port) {
-        urlObject.port = 8080
+        urlObject.port = 80
       }
       if (!urlObject.search || urlObject.search === '?') {
         urlObject.search = ''
@@ -51,7 +53,6 @@ class WebhookCommand extends Command {
       if (flags.domain === 'live') {
         domain = 'live'
       }
-      console.log('tunnel', ngrokURL)
       let originalWebhookUrl = db.read('selected_integration.' + domain + '_webhook_endpoint')
       helpers.infoLog(`Forwarding webhook events to ${flags.forward}`)
 

diff --git a/src/lib/helpers.js b/src/lib/helpers.js
@@ -21,14 +21,7 @@ const promiseWrapper = promise => (
 )
 
 function jsonLog(json) {
-  // let keys = Object.keys(json)
   console.log(json)
-  // keys.forEach(k => {
-  //   if (isJson(json[k]) || json[k] == null) {
-  //     return
-  //   }
-  //   infoLog(`${k} - - - -- - -- - - - - - -  - - - -  - ${json[k]}`)
-  // })
 }
 function successLog(error) {
   const eLog = chalk.green(error)
@@ -43,7 +36,7 @@ function isJson(val) {
 }
 
 function infoLog(error) {
-  const eLog = chalk.blue(error)
+  const eLog = chalk.blueBright(error)
   console.log(eLog)
 }
 function parseURL(uri) {
@@ -205,13 +198,13 @@ function webhookInspector(ngrokApi, tunnel) {
       requestBody = JSON.parse(requestBody)
       let responseCode = lastRequest.response.status_code.toString()
       let webhookDescription = getWebhookMessage(requestBody)
-      let responseMessage = responseCode + ' ' + lastRequest.response.status + ' - ' + requestBody.event + ' - ' + webhookDescription
+      let responseMessage =  lastRequest.response.status + ' - ' + requestBody.event + ' - ' + webhookDescription
       if (responseCode.slice(0, 1) !== '2') {
         responseMessage = chalk.red(responseMessage)
       }
       console.log(responseMessage)
       console.log(base64ToString(lastRequest.response.raw))
-      console.log('----------------------------------------------')
+      console.log('- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -')
     })
   }, 5000)
 }

diff --git a/src/lib/paystack.js b/src/lib/paystack.js
@@ -101,6 +101,10 @@ function setWebhook(url, token, integration, domain = 'test') {
     axios.put('https://api.paystack.co/integration/webhooks', data, { headers }).then(resp => {
       resolve(resp.data.message)
     }).catch(error => {
+      if(error.response){
+        reject(error.response.data.message);
+        return
+      }
       console.log(error.response.data)
       reject(error)
     })
@@ -238,7 +242,6 @@ function signIn(email, password) {
       method: 'POST',
       data: { email, password }
     }).then(response => {
-      console.log(response);
       resolve(response.data.data)
     }).catch(error => {
       if (error.response) {
@@ -263,7 +266,6 @@ function verifyToken(totp, token) {
       },
       data: { totp },
     }).then(response => {
-      console.log(response);
       resolve(response.data.data)
     }).catch(error => {
       helpers.errorLog(error.response.data.message || 'Unable to sign in, please try again in a few minutes')