Feat: supporting a custom logger.

richardgirges · Mar 1, 2024 · 7e186bd · 7e186bd
1 parent 98028e9
commit 7e186bd
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/lib/index.js b/lib/index.js
@@ -8,6 +8,7 @@ const busboy = require('busboy'); // eslint-disable-line no-unused-vars
 
 const DEFAULT_OPTIONS = {
   debug: false,
+  logger: undefined,
   uploadTimeout: 60000,
   fileHandler: false,
   uriDecodeFileNames: false,

diff --git a/lib/utilities.js b/lib/utilities.js
@@ -22,7 +22,7 @@ let tempCounter = 0;
 const debugLog = (options, msg) => {
   const opts = options || {};
   if (!opts.debug) return false;
-  console.log(`Express-file-upload: ${msg}`); // eslint-disable-line
+  (opts.logger || console).log(`Express-file-upload: ${msg}`);
   return true;
 };
 

diff --git a/test/utilities.spec.js b/test/utilities.spec.js
@@ -50,6 +50,16 @@ describe('utilities: Test of the utilities functions', function() {
       assert.equal(debugLog({debug: true}, testMessage), true);
     });
 
+    it('supports a custom logger', () => {
+      const calls = [];
+      const logger = {
+        log: (...args) => calls.push(args)
+      };
+      debugLog({debug: true, logger}, testMessage);
+      assert.equal(calls.length, 1);
+      assert.deepEqual(calls[0], [`Express-file-upload: ${testMessage}`]);
+    });
+
   });
   //isFunc tests
   describe('Test isFunc function', () => {