fix: call skip_hosts via 'this' instead of exports (#11)

- skip configuration was not being set correctly.
-  also makes the existing "txn, relaying, is_private" test pass

Author: gramakri

Changes.md

@@ -1,6 +1,7 @@
 
 ### Unreleased
 
+- Fix bug where skip configuration was ignored
 
 ### [1.2.0] - 2023-01-19
 

index.js

@@ -87,7 +87,7 @@ exports.helo_spf = async function (next, connection, helo) {
   const plugin = this;
 
   // bypass auth'ed or relay'ing hosts if told to
-  const skip_reason = exports.skip_hosts(connection);
+  const skip_reason = this.skip_hosts(connection);
   if (skip_reason) {
     connection.results.add(plugin, {skip: `helo(${skip_reason})`});
     return next();
@@ -147,7 +147,7 @@ exports.hook_mail = async function (next, connection, params) {
   if (!txn) return next();
 
   // bypass auth'ed or relay'ing hosts if told to
-  const skip_reason = exports.skip_hosts(connection);
+  const skip_reason = this.skip_hosts(connection);
   if (skip_reason) {
     txn.results.add(plugin, {skip: `host(${skip_reason})`});
     return next(CONT, `skipped because host(${skip_reason})`);