feat(#59): disable discord error logs in dev mode

barthofu · Sep 12, 2022 · 594b6f7 · 594b6f7
1 parent adbe29d
commit 594b6f7
Show file tree

Hide file tree

Showing 6 changed files with 39 additions and 33 deletions.
diff --git a/src/commands/General/ping.ts b/src/commands/General/ping.ts
@@ -23,7 +23,7 @@ export default class PingCommand {
 			member: msg.inGuild() ? `${interaction.member},` : "",
 			time: msg.createdTimestamp - interaction.createdTimestamp,
 			heartbeat: client.ws.ping ? `The heartbeat ping is ${Math.round(client.ws.ping)}ms.` : ""
-		});
+		})
 
         await msg.edit(content)
 	}

diff --git a/src/events/interactionCreate.ts b/src/events/interactionCreate.ts
@@ -26,8 +26,9 @@ export default class InteractionCreateEvent {
         [interaction]: ArgsOf<'interactionCreate'>, 
         client: Client
     ) {
+
         // defer the reply
-        if(interaction instanceof CommandInteraction) await interaction.deferReply()
+        if (interaction instanceof CommandInteraction) await interaction.deferReply()
 
         // insert user in db if not exists
         await syncUser(interaction.user)

diff --git a/src/services/ErrorHandler.ts b/src/services/ErrorHandler.ts
@@ -29,7 +29,7 @@ export class ErrorHandler {
         process.on('unhandledRejection', (error: Error | any, promise: Promise<any>) => {
 
             // if instance of BaseError, call `handle` method
-            if(error instanceof BaseError) return error.handle()
+            if (error instanceof BaseError) return error.handle()
 
             // log the error
             this.logger.logError(error, "unhandledRejection")

diff --git a/src/services/Logger.ts b/src/services/Logger.ts
@@ -82,7 +82,8 @@ export class Logger {
 
 
     async discordChannel(channelId: string, message: string | MessageOptions = '', level?: typeof this.levels[number]) {
-        if(!this.client.token) return
+
+        if (!this.client.token) return
 
         const channel = await this.client.channels.fetch(channelId)
 
@@ -232,13 +233,13 @@ export class Logger {
         let embedTitle = ''
         let chalkedMessage = `(${chalk.bold.white('ERROR')})`
 
-        if(trace && trace[0]) {
+        if (trace && trace[0]) {
             message += ` ${type === 'Exception' ? 'Exception' : 'Unhandled rejection'} : ${error.message}\n${trace.map((frame: StackFrame) => `\t> ${frame.file}:${frame.lineNumber}`).join('\n')}`
             embedMessage += `\`\`\`\n${trace.map((frame: StackFrame) => `\> ${frame.file}:${frame.lineNumber}`).join('\n')}\n\`\`\``
             embedTitle += `***${type === 'Exception' ? 'Exception' : 'Unhandled rejection'}* : ${error.message}**`
             chalkedMessage += ` ${chalk.dim.italic.gray(type === 'Exception' ? 'Exception' : 'Unhandled rejection')} : ${error.message}\n${chalk.dim.italic(trace.map((frame: StackFrame) => `\t> ${frame.file}:${frame.lineNumber}`).join('\n'))}`
         } else {
-            if(type === 'Exception') {
+            if (type === 'Exception') {
                 message += `An exception as occured in a unknow file\n\t> ${error.message}`
                 embedMessage += `An exception as occured in a unknow file\n${error.message}`
             } else {
@@ -247,15 +248,15 @@ export class Logger {
             }
         }
 
-        if(embedMessage.length >= 4096) {        
+        if (embedMessage.length >= 4096) {        
             const paste = await this.pastebin.createPaste(embedTitle + "\n" + embedMessage)
             console.log(paste?.getLink())
             embedMessage = `[Pastebin of the error](https://rentry.co/${paste?.getLink()})`
         }
 
         if (logsConfig.error.console) this.console('error', chalkedMessage)
         if (logsConfig.error.file) this.file('error', message)
-        if (logsConfig.error.channel) this.discordChannel(logsConfig.error.channel, {
+        if (logsConfig.error.channel && process.env['NODE_ENV'] === 'production') this.discordChannel(logsConfig.error.channel, {
             embeds: [{
                 title: (embedTitle.length >= 256 ? (embedTitle.substring(0, 252) + "...") : embedTitle),
                 description: embedMessage,

diff --git a/src/services/Pastebin.ts b/src/services/Pastebin.ts
@@ -1,62 +1,66 @@
-import { Paste, RentryClient } from "rentry-pastebin";
-import { singleton } from "tsyringe";
-import dayjs from "dayjs";
+import { Paste, RentryClient } from "rentry-pastebin"
+import { singleton } from "tsyringe"
+import dayjs from "dayjs"
 
-import { Pastebin as PastebinEntity } from "@entities";
-import { Database } from "@services";
+import { Pastebin as PastebinEntity } from "@entities"
+import { Database } from "@services"
 import { Schedule } from '@decorators'
 
 @singleton()
 export class Pastebin {
-    private client: RentryClient = new RentryClient();
+    private client: RentryClient = new RentryClient()
 
     constructor(
         private db: Database,
     ) {
-       this.client.createToken();
+       this.client.createToken()
     }
 
     private async waitForToken(): Promise<void> {
+
         while(!this.client.getToken()) {
             await new Promise(resolve => setTimeout(resolve, 100))
         }
     }
 
     async createPaste(content: string, lifetime?: number): Promise<Paste | undefined> {
-        await this.waitForToken();
 
-        const paste = await this.client.createPaste({ content });
+        await this.waitForToken()
+
+        const paste = await this.client.createPaste({ content })
 
-        let pasteEntity = new PastebinEntity();
-        pasteEntity.id = paste.url;
-        pasteEntity.editCode = paste.editCode;
-        if(lifetime) pasteEntity.lifetime = Math.floor(lifetime)
+        let pasteEntity = new PastebinEntity()
+        pasteEntity.id = paste.url
+        pasteEntity.editCode = paste.editCode
+        if (lifetime) pasteEntity.lifetime = Math.floor(lifetime)
 
-        await this.db.getRepo(PastebinEntity).persistAndFlush(pasteEntity);
+        await this.db.getRepo(PastebinEntity).persistAndFlush(pasteEntity)
 
-        return paste.paste;
+        return paste.paste
     }
 
     async deletePaste(id: string): Promise<void> {
-        await this.waitForToken();
+
+        await this.waitForToken()
 
-        const paste = await this.db.getRepo(PastebinEntity).findOne({ id });
+        const paste = await this.db.getRepo(PastebinEntity).findOne({ id })
 
-        if(!paste) return;
+        if (!paste) return
 
-        await this.client.deletePaste(id, paste.editCode);
-        await this.db.getRepo(PastebinEntity).remove(paste);
+        await this.client.deletePaste(id, paste.editCode)
+        await this.db.getRepo(PastebinEntity).remove(paste)
     }
 
     @Schedule('*/30 * * * *')
     private async autoDelete(): Promise<void> {
-        const pastes = await this.db.getRepo(PastebinEntity).find({ lifetime: { $gt: 0 } });
+
+        const pastes = await this.db.getRepo(PastebinEntity).find({ lifetime: { $gt: 0 } })
 
         for(const paste of pastes) {
-            const diff = dayjs().diff(dayjs(paste.createdAt), 'day');
+            const diff = dayjs().diff(dayjs(paste.createdAt), 'day')
 
-            if(diff >= paste.lifetime) {
-                await this.client.deletePaste(paste.id, paste.editCode);
+            if (diff >= paste.lifetime) {
+                await this.client.deletePaste(paste.id, paste.editCode)
             }
         }
     }

diff --git a/src/utils/functions/error.ts b/src/utils/functions/error.ts
@@ -1,4 +1,4 @@
-import { parse } from "stacktrace-parser";
+import { parse } from "stacktrace-parser"
 
 export const getCallerFile = (depth: number = 0) => {