haraka-plugin-spamassassin 1.0.1 → 1.0.3

package/CHANGELOG.md

@@ -4,6 +4,17 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/).
 
 ### Unreleased
 
+### [1.0.3] - 2025-02-06
+
+- results: tidying up duplicate data
+- results.hits: deleted, alias for score
+- results.status: deleted, alias for flag
+- results.flag: change from Yes/No to boolean
+
+### [1.0.2] - 2025-01-26
+
+- prettier: move config into package.json
+
 ### [1.0.1] - 2025-01-09
 
 - dep(eslint): update to v9
@@ -12,4 +23,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/).
 
 - repackaged from haraka/Haraka
 
-[1.0.1]: https://github.com/haraka/haraka-plugin-spamassassin/releases/tag/v1.0.1
+[1.0.1]: https://github.com/haraka/haraka-plugin-spamassassin/releases/tag/1.0.1
+[1.0.2]: https://github.com/haraka/haraka-plugin-spamassassin/releases/tag/v1.0.2
+[1.0.0]: https://github.com/haraka/haraka-plugin-spamassassin/releases/tag/1.0.0
+[1.0.3]: https://github.com/haraka/haraka-plugin-spamassassin/releases/tag/v1.0.3

package/README.md

@@ -178,7 +178,6 @@ Other headers options you might find interesting or useful are:
     add_header all DCC _DCCB_: _DCCR_
     add_header all Tests _TESTS_
 
-## USAGE
 
 <!-- leave these buried at the bottom of the document -->
 

package/index.js

@@ -82,11 +82,9 @@ exports.hook_data_post = function (next, connection) {
           /Spam: (True|False) ; (-?\d+\.\d) \/ (-?\d+\.\d)/,
         )
         if (matches) {
-          spamd_response.flag = matches[1]
+          spamd_response.flag = matches[1] === 'True'
           spamd_response.score = matches[2]
-          spamd_response.hits = matches[2] // backwards compat
           spamd_response.reqd = matches[3]
-          spamd_response.flag = spamd_response.flag === 'True' ? 'Yes' : 'No'
         }
       } else {
         state = 'headers'
@@ -131,7 +129,6 @@ exports.hook_data_post = function (next, connection) {
     txn.notes.spamassassin = spamd_response
     connection.results.add(this, {
       time: (Date.now() - start) / 1000,
-      hits: spamd_response.hits,
       flag: spamd_response.flag,
     })
 
@@ -197,7 +194,7 @@ exports.munge_subject = function (conn, score) {
 }
 
 exports.do_header_updates = function (conn, spamd_response) {
-  if (spamd_response.flag === 'Yes') {
+  if (spamd_response.flag) {
     // X-Spam-Flag is added by SpamAssassin
     conn.transaction.remove_header('precedence')
     conn.transaction.add_header('Precedence', 'junk')
@@ -342,7 +339,7 @@ exports.log_results = function (conn, spamd_response) {
     : cfg.reject_threshold
 
   const human_text =
-    `status=${spamd_response.flag}` +
+    `status=${spamd_response.flag ? 'Yes' : 'No'}` +
     `, score=${spamd_response.score}` +
     `, required=${spamd_response.reqd}` +
     `, reject=${reject_threshold}` +
@@ -350,7 +347,6 @@ exports.log_results = function (conn, spamd_response) {
 
   conn.transaction.results.add(this, {
     human: human_text,
-    status: spamd_response.flag,
     score: parseFloat(spamd_response.score),
     required: parseFloat(spamd_response.reqd),
     reject: reject_threshold,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "haraka-plugin-spamassassin",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "description": "Haraka plugin that scans messages with SpamAssassin",
   "main": "index.js",
   "files": [
@@ -33,12 +33,16 @@
   },
   "homepage": "https://github.com/haraka/haraka-plugin-spamassassin#readme",
   "dependencies": {
-    "haraka-net-utils": "^1.7.1",
-    "haraka-utils": "^1.1.3"
+    "haraka-net-utils": "^1.7.2",
+    "haraka-utils": "^1.1.4"
   },
   "devDependencies": {
-    "@haraka/eslint-config": "2.0.2",
-    "address-rfc2821": "^2.1.2",
-    "haraka-test-fixtures": "1.3.8"
+    "@haraka/eslint-config": "^2.0.2",
+    "address-rfc2821": "^2.1.3",
+    "haraka-test-fixtures": "1.3.9"
+  },
+  "prettier": {
+    "singleQuote": true,
+    "semi": false
   }
 }