Add self-signed option to allow configure with private gitlab instances

index.js

@@ -8,13 +8,21 @@ const { configure } = require('./lib/commands/configure');
 const { verify } = require('./lib/commands/verify');
 const { getAllAssigned } = require('./lib/commands/getAllAssigned');
 const { getAllSubmitted } = require('./lib/commands/getAllSubmitted');
-
+const { disableCertificateVerification } = require('./lib/options/disableCertificateVerification');
 const { readConfig } = require('./lib/utils/readConfig');
 
 program
   .name('mergify')
   .version(pack.version);
 
+const options = [
+  {
+    trigger: '-s --self-signed',
+    description: 'disables the verification of certificates for commands',
+    fn: disableCertificateVerification
+  }
+];
+
 const commands = [
   {
     trigger: 'assigned',
@@ -52,6 +60,10 @@ const run = async() => {
     await verify();
   }
 
+  options.forEach(({trigger, description, fn}) => {
+    program.option(trigger, description, (...args) => fn(config, ...args));
+  });
+
   commands.forEach(({ trigger, description, fn }) => {
     program
       .command(trigger)

lib/commands/configure/index.js

@@ -8,7 +8,7 @@ const chalk = require('chalk');
 
 const configure = async() => {
   if(await checkConfigExists()){
-    logger.log(chalk.red.bold('⚠️ Mergify is already configured. Configuring again will override the existing file.'));
+    logger.warn('Mergify is already configured. Configuring again will override the existing file.');
   }
 
   try {

lib/commands/verify/index.js

@@ -22,8 +22,8 @@ const verify = async({ userId }) => {
     return process.exit(0);
   } catch (error) {
     spinner.stop();
-    logger.log('\n🙀  Oh no, could not complete verify. Please review your config');
-    logger.log(error);
+    logger.err('Oh no, could not complete verify. Please review your config');
+    logger.err(error);
     process.exit(1);
   }
 };

lib/options/disableCertificateVerification/index.js

@@ -0,0 +1,11 @@
+const {logger}  = require('../../utils/logger');
+
+const disableCertificateVerification = async() => {
+    logger.warn('Disabling certificate verification. This is unsafe and should only be used as last resort.');
+    process.env.NODE_TLS_REJECT_UNAUTHORIZED = "0";
+    return;
+};
+
+module.exports = {
+    disableCertificateVerification
+};

lib/utils/logger/index.js

@@ -1,3 +1,22 @@
+const chalk = require('chalk');
+
+const getLogger = function(transport){
+  return {
+    log : (...args) =>{
+      transport.log(...args);
+    },
+
+    warn : (...args) =>{
+      transport.log('⚠️', chalk.yellow.bold(...args));
+    },
+
+    err : (...args) =>{
+      transport.log('🙀', chalk.red.bold(...args));
+    }
+  }
+}
+
 module.exports = {
-  logger: console
-};
+  logger: getLogger(console),
+  getLogger: getLogger
+}

lib/utils/logger/index.spec.js

@@ -0,0 +1,42 @@
+const {getLogger} = require('./index');
+
+describe('utils/logger', () => {
+    let mockConsole;
+    let logger;
+
+    beforeAll(() => {
+        mockConsole = {
+            content: '',
+
+            log: function(...args) {
+                this.content = [...args];
+            },
+        }
+        logger = getLogger(mockConsole);
+    });
+
+    test('Testing normal log message', () => {
+
+        logger.log('test', 'plop');
+
+        expect(mockConsole.content).toHaveLength(2);
+        expect(mockConsole.content).toEqual(['test', 'plop']);
+    });
+
+    test('Testing warning log message', () => {
+
+        logger.warn('test', 'plop');
+
+        expect(mockConsole.content).toHaveLength(2);
+        expect(mockConsole.content[0]).toEqual('⚠️');
+    });
+
+    test('Testing error log message', () => {
+
+        logger.err('test', 'plop');
+
+        expect(mockConsole.content).toHaveLength(2);
+        expect(mockConsole.content[0]).toEqual('🙀');
+    });
+}); 
+