Release v1.0.3 (#2)

- config: add verify.enabled setting
- doc(CONTRIBUTORS): added

CHANGELOG.md

@@ -4,6 +4,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/).
 
 ### Unreleased
 
+### [1.0.3] - 2024-04-15
+
+- config: add `verify.enabled` setting
+
 ### [1.0.2] - 2024-04-10
 
 - fix: properly scope the logdebug injections
@@ -14,3 +18,4 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/).
 
 [1.0.0]: https://github.com/haraka/haraka-plugin-dkim/releases/tag/v1.0.0
 [1.0.2]: https://github.com/haraka/haraka-plugin-dkim/releases/tag/v1.0.2
+[1.0.3]: https://github.com/haraka/haraka-plugin-dkim/releases/tag/v1.0.3

CONTRIBUTORS.md

@@ -0,0 +1,8 @@
+# Contributors
+
+This handcrafted artisinal software is brought to you by:
+
+| <img height="80" src="https://avatars.githubusercontent.com/u/261635?v=4"><br><a href="https://github.com/msimerson">msimerson</a> (<a href="https://github.com/haraka/haraka-plugin-dkim/commits?author=msimerson">3</a>) |
+| :------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------: |
+
+<sub>this file is maintained by [.release](https://github.com/msimerson/.release)</sub>

config/dkim.ini

@@ -10,6 +10,8 @@ headers = From, Sender, Reply-To, Subject, Date, Message-ID, To, Cc, MIME-Versio
 
 
 [verify]
+enabled = true
+
 ; Recommended (but not default) values presented below.
 
 ; in secs

index.js

@@ -22,15 +22,20 @@ exports.register = function () {
     plugin.logdebug(str)
   }
 
-  this.register_hook('data_post', 'dkim_verify')
-  this.register_hook('queue_outbound', 'hook_pre_send_trans_email')
+  if (this.cfg.verify.enabled) {
+    this.register_hook('data_post', 'dkim_verify')
+  }
+
+  if (this.cfg.sign.enabled) {
+    this.register_hook('queue_outbound', 'hook_pre_send_trans_email')
+  }
 }
 
 exports.load_dkim_ini = function () {
   this.cfg = this.config.get(
     'dkim.ini',
     {
-      booleans: ['-sign.enabled'],
+      booleans: ['-sign.enabled', '+verify.enabled'],
     },
     () => {
       this.load_dkim_ini()
@@ -304,6 +309,8 @@ exports.get_sender_domain = function (connection) {
 }
 
 exports.dkim_verify = function (next, connection) {
+  if (!this.cfg.verify.enabled) return next()
+
   const txn = connection?.transaction
   if (!txn) return next()
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "haraka-plugin-dkim",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Haraka DKIM plugin",
   "main": "index.js",
   "files": [
@@ -16,8 +16,8 @@
     "prettier": "npx prettier . --check",
     "prettier:fix": "npx prettier . --write --log-level=warn",
     "test": "node --test",
-    "versions": "npx @msimerson/dependency-version-checker check",
-    "versions:fix": "npx @msimerson/dependency-version-checker update"
+    "versions": "npx dependency-version-checker check",
+    "versions:fix": "npx dependency-version-checker update"
   },
   "repository": {
     "type": "git",
@@ -36,7 +36,7 @@
   "homepage": "https://github.com/haraka/haraka-plugin-dkim#readme",
   "devDependencies": {
     "@haraka/eslint-config": "1.1.3",
-    "haraka-test-fixtures": "1.3.5"
+    "haraka-test-fixtures": "1.3.7"
   },
   "dependencies": {
     "address-rfc2821": "^2.1.2",

test/dkim_sign.js

@@ -14,7 +14,7 @@ beforeEach(() => {
   this.plugin = new fixtures.plugin('dkim')
 
   this.connection = fixtures.connection.createConnection()
-  this.connection.transaction = fixtures.transaction.createTransaction()
+  this.connection.init_transaction()
 })
 
 describe('get_sender_domain', () => {

test/dkim_signer.js

@@ -54,7 +54,7 @@ beforeEach(() => {
   this.plugin = new fixtures.plugin('dkim')
 
   this.connection = fixtures.connection.createConnection()
-  this.connection.transaction = fixtures.transaction.createTransaction()
+  this.connection.init_transaction()
   // this.connection.transaction.mail_from = {}
 })
 

test/index.js

@@ -36,7 +36,7 @@ describe('uses text fixtures', () => {
 
   it('sets up a transaction', () => {
     this.connection = fixtures.connection.createConnection({})
-    this.connection.transaction = fixtures.transaction.createTransaction({})
+    this.connection.init_transaction()
     assert.ok(this.connection.transaction.header)
   })
 })
@@ -51,6 +51,7 @@ const expectedCfg = {
       'From, Sender, Reply-To, Subject, Date, Message-ID, To, Cc, MIME-Version',
   },
   verify: {
+    enabled: true,
     timeout: 29,
   },
   headers_to_sign: [