Enable level config for console logger (#1324)

* enable level config for console logger

* move logger to shared dir

Author: waltjones

src/browser/core.js

@@ -1,7 +1,7 @@
 import Client from '../rollbar.js';
 import * as _ from '../utility.js';
 import API from '../api.js';
-import logger from './logger.js';
+import logger from '../logger.js';
 import * as globals from './globalSetup.js';
 
 import Transport from './transport.js';
@@ -17,6 +17,7 @@ import tracingDefaults from '../tracing/defaults.js';
 import ReplayManager from './replay/replayManager.js';
 
 function Rollbar(options, client) {
+  logger.init({ logLevel: options.logLevel || 'error' });
   this.options = _.handleOptions(defaultOptions, options, null, logger);
   this.options._configuredOptions = options;
   const Telemeter = this.components.telemeter;
@@ -121,6 +122,9 @@ Rollbar.global = function (options) {
 };
 
 Rollbar.prototype.configure = function (options, payloadData) {
+  if (options.logLevel) {
+    logger.init({ logLevel: options.logLevel });
+  }
   var oldOptions = this.options;
   var payload = {};
   if (payloadData) {

src/browser/logger.js

@@ -2,7 +2,7 @@ import { record as rrwebRecordFn } from '@rrweb/record';
 import { EventType } from '@rrweb/types';
 
 import hrtime from '../../tracing/hrtime.js';
-import logger from '../logger.js';
+import logger from '../../logger.js';
 
 export default class Recorder {
   _options;

src/browser/replay/recorder.js

@@ -1,5 +1,5 @@
 import id from '../../tracing/id.js';
-import logger from '../logger.js';
+import logger from '../../logger.js';
 
 /**
  * ReplayManager - Manages the mapping between error occurrences and their associated

src/browser/replay/replayManager.js

@@ -1,6 +1,6 @@
 import * as _ from '../utility.js';
 import errorParser from '../errorParser.js';
-import logger from './logger.js';
+import logger from '../logger.js';
 
 function handleDomException(item, options, callback) {
   if (item.err && errorParser.Stack(item.err).name === 'DOMException') {

src/browser/transforms.js

@@ -1,4 +1,4 @@
-import logger from '../logger.js';
+import logger from '../../logger.js';
 import * as _ from '../../utility.js';
 
 function makeFetchRequest({accessToken, url, method, payload, headers, callback, timeout}) {

src/browser/transport/fetch.js

@@ -1,7 +1,7 @@
 /*global XDomainRequest*/
 
 import * as _ from '../../utility.js';
-import logger from '../logger.js';
+import logger from '../../logger.js';
 
 function makeXhrRequest({
   accessToken,

src/browser/transport/xhr.js

@@ -0,0 +1,29 @@
+let log = () => {};
+
+const levels = {
+  debug: 0,
+  info: 1,
+  warn: 2,
+  error: 3,
+  disable: 4,
+};
+
+const logger = {
+  error: (...args) => log('error', args),
+  warn: (...args) => log('warn', args),
+  info: (...args) => log('info', args),
+  debug: (...args) => log('debug', args),
+  log: (...args) => log('info', args),
+  init: ({ logLevel }) => {
+    log = function(level, args) {
+      if (levels[level] < levels[logLevel]) return;
+
+      args.unshift('Rollbar:');
+
+      /* eslint-disable no-console */
+      console[level].apply(console, args);
+    }
+  }
+};
+
+export default logger;

src/logger.js

@@ -1,7 +1,7 @@
 import Client from '../rollbar.js';
 import * as _ from '../utility.js';
 import API from '../api.js';
-import logger from './logger.js';
+import logger from '../logger.js';
 import * as rnDefaults from './defaults.js';
 import { version, reportLevel } from '../defaults.js';
 
@@ -20,6 +20,7 @@ function Rollbar(options, client) {
     options = {};
     options.accessToken = accessToken;
   }
+  logger.init({ logLevel: options.logLevel || 'error' });
   this.options = _.handleOptions(Rollbar.defaultOptions, options, null, logger);
   this.options._configuredOptions = options;
   // This makes no sense in a long running app
@@ -74,6 +75,9 @@ Rollbar.global = function (options) {
 };
 
 Rollbar.prototype.configure = function (options, payloadData) {
+  if (options.logLevel) {
+    logger.init({ logLevel: options.logLevel });
+  }
   var oldOptions = this.options;
   var payload = {};
   if (payloadData) {