Use winston for logging

README.md

@@ -8,15 +8,14 @@
 # Required API endpoint/authentication variables
 SONAR_URL=https://instance.sonar.software/api/graphql
 SONAR_TOKEN=
-RINGCENTRAL_CLIENT_ID=
-RINGCENTRAL_CLIENT_SECRET=
-RINGCENTRAL_USERNAME=
-RINGCENTRAL_EXTENSION=
-RINGCENTRAL_PASSWORD=
+RC_APP_KEY=
+RC_APP_SECRET=
+RC_LOGIN_USERNAME=
+RC_LOGIN_EXT=
+RC_LOGIN_PASSWORD=
 
-# Set to 'sandbox' to use RingCentral's sandbox API
-# Any other value will result in using RingCentral's production API
-RINGCENTRAL_SERVER=production
+# Set to any value to enable use of RingCentral's sandbox API
+RC_SANDBOX=
 
 # The database to use
 # valid options: pg, sqlite

src/index.ts

@@ -9,11 +9,11 @@ function checkEnv() {
   [
     "SONAR_URL",
     "SONAR_TOKEN",
-    "RINGCENTRAL_CLIENT_ID",
-    "RINGCENTRAL_CLIENT_SECRET",
-    "RINGCENTRAL_USERNAME",
-    "RINGCENTRAL_EXTENSION",
-    "RINGCENTRAL_PASSWORD",
+    "RC_APP_KEY",
+    "RC_APP_SECRET",
+    "RC_LOGIN_USERNAME",
+    "RC_LOGIN_EXT",
+    "RC_LOGIN_PASSWORD",
     "EXTENSION_TICKET_GROUPS",
   ].forEach((env) => {
     if (process.env[env] === undefined) {
@@ -59,25 +59,22 @@ async function initSonar() {
       }
     `
   );
-  logger.info(`Authenticated to Sonar as '${user.me.name}'`);
+  logger.info(`Authenticated to Sonar as '${user.me.name}'.`);
   return sonar;
 }
 
 async function initRingCentralSDK() {
   const sdk = new SDK({
-    server:
-      SDK.server[
-        process.env.RINGCENTRAL_SERVER === "sandbox" ? "sandbox" : "production"
-      ],
-    clientId: process.env.RINGCENTRAL_CLIENT_ID,
-    clientSecret: process.env.RINGCENTRAL_CLIENT_SECRET,
+    server: SDK.server[process.env.RC_SANDBOX ? "sandbox" : "production"],
+    clientId: process.env.RC_APP_KEY,
+    clientSecret: process.env.RC_APP_SECRET,
   });
 
   const login = () =>
     sdk.login({
-      username: process.env.RINGCENTRAL_USERNAME,
-      extension: process.env.RINGCENTRAL_EXTENSION,
-      password: process.env.RINGCENTRAL_PASSWORD,
+      username: process.env.RC_LOGIN_USERNAME,
+      extension: process.env.RC_LOGIN_EXT,
+      password: process.env.RC_LOGIN_PASSWORD,
     });
 
   if (DEBUG) {
@@ -91,11 +88,11 @@ async function initRingCentralSDK() {
   platform.on(platform.events.refreshError, async (err) => {
     logger.error("Refresh token error:", err);
     await login();
-    logger.info("RingCentral re-authentication successful");
+    logger.info("RingCentral re-authentication successful.");
   });
 
   await login();
-  logger.info("Authenticated to RingCentral");
+  logger.info("Authenticated to RingCentral.");
 
   return sdk;
 }
@@ -104,7 +101,7 @@ async function initDB() {
   const db = knex(knexConfig);
   if (!process.env.DB_SKIP_MIGRATIONS) {
     await db.migrate.latest();
-    logger.info("Database migrations run successfully");
+    logger.info("Database migrations run successfully.");
   }
   return db;
 }
@@ -117,13 +114,12 @@ async function main() {
     const rcsdk = await initRingCentralSDK();
     const db = await initDB();
 
-    logger.info("Starting");
-
+    logger.info("Starting ticketizer...");
     const intervals = ticketize(sonar, rcsdk, db, getTicketizeConfig());
 
     ["SIGINT", "SIGTERM", "SIGQUIT"].forEach((sig) => {
       process.on(sig, async () => {
-        logger.info(`Caught ${sig}, shutting down...`);
+        logger.info(`\nCaught ${sig}, shutting down...`);
         const results = await Promise.allSettled(
           intervals.map((interval) => interval.clear())
         );
@@ -134,7 +130,6 @@ async function main() {
             logger.error(result.reason);
           }
         });
-        await rcsdk.logout();
         process.exit(errors ? 1 : 0);
       });
     });

src/ticketize.ts

@@ -340,7 +340,7 @@ export function ticketize(
    * @param firstRun whether this is the first run
    */
   async function fetchAndStoreNewVoicemails(firstRun = false) {
-    logger.verbose("Checking for new voicemails");
+    logger.info("Checking for new voicemails");
     const extensions = await getValidRCExtensions();
     return Promise.all(
       extensions.map((extension) =>

src/util.ts

@@ -1,17 +1,10 @@
 import PhoneNumber from "awesome-phonenumber";
-import winston, { format } from "winston";
+import winston from "winston";
 
 export const DEBUG = !!process.env.DEBUG;
 export const logger = winston.createLogger({
   level: DEBUG ? "debug" : process.env.LOG_LEVEL ?? "info",
-  transports: [new winston.transports.Console()],
-  format: format.combine(
-    format.errors({ stack: true }),
-    format.printf(
-      ({ level, message, stack }) =>
-        `${level}: ${message}${stack ? "\n" + stack : ""}`
-    )
-  ),
+  transports: [new winston.transports.()],
 });
 
 export function getNationalNumber(input: string) {