Add status index

Author: brandonkelly

CHANGELOG.md

@@ -9,6 +9,7 @@
 
 ### Changed
 - Webhooks now requires Craft 3.7 or later.
+- Improved the performance of clearing out completed webhook request logs. ([#32](https://github.com/craftcms/webhooks/issues/32))
 
 ### Fixed
 - Fixed an error that could occur when calling `craft\webhooks\Plugin::getRequestData()` if an invalid request ID was passed.

src/Plugin.php

@@ -87,7 +87,7 @@ class Plugin extends \craft\base\Plugin
     /**
      * @inheritdoc
      */
-    public $schemaVersion = '2.3.1';
+    public $schemaVersion = '2.4.0';
 
     /**
      * @var SendRequestJob[] The request jobs that should be queued up at the end of the web request.

src/migrations/Install.php

@@ -72,6 +72,7 @@ public function safeUp(): bool
         $this->createIndex(null, '{{%webhooks}}', ['groupId', 'name']);
         $this->createIndex(null, '{{%webhooks}}', ['name'], true);
         $this->createIndex(null, '{{%webhookrequests}}', ['debounceKey', 'status']);
+        $this->createIndex(null, '{{%webhookrequests}}', ['status']);
         $this->addForeignKey(null, '{{%webhooks}}', ['groupId'], '{{%webhookgroups}}', ['id'], 'SET NULL');
         $this->addForeignKey(null, '{{%webhookrequests}}', ['webhookId'], '{{%webhooks}}', ['id'], 'SET NULL');
 

src/migrations/m210830_202902_status_index.php

@@ -0,0 +1,29 @@
+<?php
+
+namespace craft\webhooks\migrations;
+
+use craft\db\Migration;
+
+/**
+ * m210830_202902_status_index migration.
+ */
+class m210830_202902_status_index extends Migration
+{
+    /**
+     * @inheritdoc
+     */
+    public function safeUp(): bool
+    {
+        $this->createIndex(null, '{{%webhookrequests}}', ['status']);
+        return true;
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function safeDown(): bool
+    {
+        echo "m210830_202902_status_index cannot be reverted.\n";
+        return false;
+    }
+}