Compare View
Commits (2)
Changes
Showing 3 changed files Side-by-side Diff
coreapi/index.js
... | ... | @@ -5,7 +5,11 @@ const request = require('request'); |
5 | 5 | const logger = require('../logger'); |
6 | 6 | const coreUrl = require('../core-url'); |
7 | 7 | |
8 | -logger.verbose(`CORE URL: ${coreUrl}`); | |
8 | +function isLogDisabled() { | |
9 | + return global.KOMODO_SDK_NO_LOG_ON_COREAPI; | |
10 | +} | |
11 | + | |
12 | +isLogDisabled() || logger.verbose(`CORE URL: ${coreUrl}`); | |
9 | 13 | |
10 | 14 | function doRequest(params, cb) { |
11 | 15 | return new Promise((resolve) => { |
... | ... | @@ -15,13 +19,13 @@ function doRequest(params, cb) { |
15 | 19 | qs: params.qs || null, |
16 | 20 | }; |
17 | 21 | |
18 | - logger.verbose('Requesting to CORE', { | |
22 | + isLogDisabled() || logger.verbose('Requesting to CORE', { | |
19 | 23 | xid: params.xid, method: options.method, fullpath: options.url, qs: options.qs, |
20 | 24 | }); |
21 | 25 | |
22 | 26 | request(options, (err, res, body) => { |
23 | 27 | if (err) { |
24 | - logger.warn(`COREAPI: Error doing HTTP ${options.method} to CORE. ${err.toString()}`, { xid: params.xid }); | |
28 | + isLogDisabled() || logger.warn(`COREAPI: Error doing HTTP ${options.method} to CORE. ${err.toString()}`, { xid: params.xid }); | |
25 | 29 | |
26 | 30 | resolve([err]); |
27 | 31 | if (typeof cb === 'function') cb(err); |
... | ... | @@ -30,7 +34,7 @@ function doRequest(params, cb) { |
30 | 34 | |
31 | 35 | if (res.statusCode !== 200) { |
32 | 36 | const errStatusCode = new Error('COREAPI: CORE responded with non HTTP STATUS CODE 200'); |
33 | - logger.warn(`COREAPI: CORE returning HTTP STATUS CODE ${res.statusCode}, not 200`, { xid: params.xid, body }); | |
37 | + isLogDisabled() || logger.warn(`COREAPI: CORE returning HTTP STATUS CODE ${res.statusCode}, not 200`, { xid: params.xid, body }); | |
34 | 38 | |
35 | 39 | resolve([errStatusCode]); |
36 | 40 | if (typeof cb === 'function') cb(errStatusCode); |
... | ... | @@ -42,7 +46,7 @@ function doRequest(params, cb) { |
42 | 46 | bodyObject = JSON.parse(body); |
43 | 47 | } catch (e) { |
44 | 48 | const errNoJson = new Error('COREAPI: CORE responded with non JSON body'); |
45 | - logger.verbose([errNoJson]); | |
49 | + isLogDisabled() || logger.verbose([errNoJson]); | |
46 | 50 | |
47 | 51 | resolve([errNoJson, body]); |
48 | 52 | if (typeof cb === 'function') cb(errNoJson, body); |
package-lock.json