Fix QRMBot Command

Author: StrangeWill

Dockerfile

@@ -7,7 +7,7 @@ EXPOSE 5010
 ENV ASPNETCORE_URLS=http://+:5010
 
 RUN apt-get update && \
-    apt-get install wget unzip -y
+    apt-get install wget unzip curl perl perl-base perl-modules libclone-perl libdate-manip-perl libdatetime-format-strptime-perl libdatetime-perl libjson-perl libmath-bigint-perl libmath-round-perl libswitch-perl libtext-csv-perl liburi-perl -y
 # Creates a non-root user with an explicit UID and adds permission to access the /app folder
 # For more info, please refer to https://aka.ms/vscode-docker-dotnet-configure-containers
 RUN adduser -u 5678 --disabled-password --gecos "" appuser && chown -R appuser /app

Services/CommandService.cs

@@ -34,7 +34,7 @@ public CommandService(
         //  I choose the lazy solution, I choose... this.
         Commands.AddRange(Directory.EnumerateFiles("qrmbot/lib")
             .Select(f => new QrmBotCommand(
-                f.Replace(".pl", string.Empty),
+                f.Split("/").Last().Replace(".pl", string.Empty),
                 serviceProvider.GetRequiredService<Slack>(),
                 serviceProvider.GetRequiredService<IConfiguration>(),
                 serviceProvider.GetRequiredService<ILogger<QrmBotCommand>>()))
@@ -54,13 +54,20 @@ public CommandService(
     /// <returns></returns>
     public async Task HandleMessage(WebhookMessage message, string commandSuffix = "")
     {
-        // Remove leading forward-slash and any optional suffix
-        var messageCommand = message.Command.Substring(1);
-        messageCommand = messageCommand.Substring(0, messageCommand.Length - commandSuffix.Length);
+        try
+        {
+            // Remove leading forward-slash and any optional suffix
+            var messageCommand = message.Command.Substring(1);
+            messageCommand = messageCommand.Substring(0, messageCommand.Length - commandSuffix.Length);
 
-        var command = Commands.Find((c) => c.CommandText == messageCommand);
+            var command = Commands.Find((c) => c.CommandText == messageCommand);
 
-        await command.Execute(message);
+            await command.Execute(message);
+        }
+        catch (Exception exception)
+        {
+            Logger.LogError(exception, "Error running command: {}", message.Command);
+        }
     }
 
     /// <summary>
@@ -75,6 +82,14 @@ public async Task HandleInteractive(InteractiveMessage message)
     {
         var command = Commands.Find((c) => c.CommandText == message.CallbackId);
 
-        await command.ExecuteInteractive(message);
+        try
+        {
+            Logger.LogError("Test: {}", message.CallbackId);
+            await command.ExecuteInteractive(message);
+        }
+        catch (Exception exception)
+        {
+            Logger.LogError(exception, "Error running command: {}", message.CallbackId);
+        }
     }
 }

Services/QrmBotCommand.cs

@@ -21,10 +21,11 @@ public QrmBotCommand(string command, Slack slack, IConfiguration configuration,
 
     protected override async Task HandleMessage(WebhookMessage webhookMessage)
     {
+        var slackUser = await Slack.GetUser(webhookMessage.UserId);
         var perlStartInfo = new ProcessStartInfo(@"perl")
         {
             // MAKE SURE WE ESCAPE DOUBLE QUOTES DON'T RUIN MY DAY I SWEAR
-            Arguments = $"{Directory}qrmbot/lib/{CommandText} \"{webhookMessage.Text.Replace("\"", "\\\"")}\"",
+            Arguments = $"{Directory}lib/{CommandText} \"{webhookMessage.Text.Replace("\"", "\\\"")}\"",
             UseShellExecute = false,
             RedirectStandardOutput = true,
             RedirectStandardError = true,
@@ -43,6 +44,8 @@ await Slack.PostMessage(new ChatMessage
         {
             Channel = webhookMessage.ChannelId,
             Text = output,
+            Username = slackUser.Profile.DisplayName,
+            IconUrl = slackUser.Profile.ImageOriginal
         });
     }
 }