chore: replace credentials.json for a environmental variable

Author: UlisesGascon

update/add-route.js

@@ -4,15 +4,15 @@ const qs = require('querystring')
 
 const url = 'https://api.mailgun.net/v3/routes'
 
-function addRoute (domain, creds, description, expression, actions, callback) {
+function addRoute (domain, description, expression, actions, callback) {
   const params = {
     description,
     expression,
     action: actions
   }
   const data = qs.stringify(params)
   const options = {
-    auth: `api:${creds['api-key']}`,
+    auth: `api:${process.env.MAILGUN_API_KEY}`,
     headers: {
       'content-type': 'application/x-www-form-urlencoded',
       'content-length': Buffer.byteLength(data)

update/delete-route.js

@@ -2,8 +2,8 @@ const jsonist = require('jsonist')
 
 const url = 'https://api.mailgun.net/v3/routes'
 
-function deleteRoute (domain, creds, id, callback) {
-  const options = { auth: `api:${creds['api-key']}` }
+function deleteRoute (domain, id, callback) {
+  const options = { auth: `api:${process.env.MAILGUN_API_KEY}` }
   jsonist.delete(`${url}/${id}`, options, callback)
 }
 

update/list-routes.js

@@ -2,8 +2,8 @@ const jsonist = require('jsonist')
 
 const url = 'https://api.mailgun.net/v3/routes'
 
-function listRoutes (domain, creds, callback) {
-  const options = { auth: `api:${creds['api-key']}` }
+function listRoutes (domain, callback) {
+  const options = { auth: `api:${cprocess.env.MAILGUN_API_KEY}` }
   jsonist.get(url, options, callback)
 }
 

update/update-aliases.js

@@ -33,7 +33,7 @@ function diff (domain, a1, a2) {
   })
 }
 
-function updateAliases (domain, creds, aliases, dryRun, callback) {
+function updateAliases (domain, aliases, dryRun, callback) {
   function adjustRoutes (routes) {
     const current = routes.items
 
@@ -56,19 +56,19 @@ function updateAliases (domain, creds, aliases, dryRun, callback) {
       const to = Array.isArray(alias.to) ? alias.to : [alias.to]
       console.log(`Adding ${alias.from} -> ${to.join(', ')}...`)
       if (!dryRun) {
-        addRoute(domain, creds, alias.from, alias.expression, alias.actions, done)
+        addRoute(domain, alias.from, alias.expression, alias.actions, done)
       }
     })
 
     toRemove.forEach(function (alias) {
       console.log(`Deleting ${alias.expression}...`)
       if (!dryRun) {
-        deleteRoute(domain, creds, alias.id, done)
+        deleteRoute(domain, alias.id, done)
       }
     })
   }
 
-  listRoutes(domain, creds, function (err, routes) {
+  listRoutes(domain, function (err, routes) {
     if (err) { throw err }
     adjustRoutes(routes)
   })

update/update.js

@@ -2,7 +2,6 @@
 
 const fs   = require('fs')
     , path = require('path')
-
     , updateAliases = require('./update-aliases')
 
 require('http').globalAgent.maxSockets = 20
@@ -15,28 +14,15 @@ if (process.argv.length < 3) {
 const dryRun    = process.argv.includes('--dry-run')
     , domain    = process.argv.filter((a) => a !== '--dry-run')[2].replace(/\/$/, '')
     , dir       = path.join(__dirname, '..', domain)
-    , credsFile = path.join(dir, 'credentials.json')
-
-if (!fs.statSync(dir).isDirectory()) {
-  console.error(`Usage: update <domain> ("domain" must be a directory above ${__dirname}`)
-  return process.exit(1)
-}
-
-if (!fs.existsSync(credsFile)) {
-  console.error(`Error: ${dir} does not have a credentials.json file`)
-  return process.exit(1)
-}
-
-const creds = require(credsFile)
 
-if (typeof creds['api-key'] != 'string') {
-  console.error(`Error: ${credsFile} does not have an "api-key" property`)
+if (!process.env.MAILGUN_API_KEY) {
+  console.error(`Error: MAILGUN_API_KEY environment variable is not set`)
   return process.exit(1)
 }
 
 const aliases = require(path.join(dir, 'aliases.json'))
 
-updateAliases(domain, creds, aliases, dryRun, function (err) {
+updateAliases(domain, aliases, dryRun, function (err) {
   if (err)
     throw err
 })