https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6485

[email protected] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |software+spamassas...@kd6lv
                   |                            |w.ampr.org

--- Comment #2 from [email protected] 2010-11-30 20:12:19 
UTC ---
Doesn't work for me.  The variables are not set - a null string is substituted.

SpamAssassin Version:  3.3.1 (2010-03-16)

Generating line:  "report ASN: _ASN_ (_ASNCIDR_) _RELAYCOUNTRY_"
Resulting  line:  "ASN:  () US **"
(Sample taken from the bug report e-mail I got for your followup.)
  US <= thor.apache.org.  (relaycountry string)
  ** <= localhost.        (as expected. RC works.  ASN doesn't.)

My report line is bracketed by "ifplugin Mail::SpamAssassin::Plugin::ASN" and
"endif" - so it would only inject into the report if the plugin module were in
fact loaded.  Full context:

ifplugin Mail::SpamAssassin::Plugin::ASN
report ASN: _ASN_ (_ASNCIDR_) _RELAYCOUNTRY_
asn_lookup origin.asn.spameatingmonkey.net _ASNCIDR_ _ASN_
asn_lookup origin.asn.cymru.com
asn_lookup ip2asn.sasm4.net
endif

...

It's in my pre file:
!cat /etc/mail/spamassassin/local.pre
tryplugin Mail::SpamAssassin::Plugin::ASN  <- Here it is.
tryplugin Mail::SpamAssassin::Plugin::AWL
tryplugin Mail::SpamAssassin::Plugin::DCC
...


Debugging log says it's there:
!spamassassin -D all --lint
Dec  1 00:51:58.694 [11558] dbg: logger: adding facilities: all
...
Dec  1 00:52:01.614 [11558] dbg: config: read file
/etc/mail/spamassassin/local.pre
...
Dec  1 00:52:02.030 [11558] dbg: plugin: loading
Mail::SpamAssassin::Plugin::SPF from @INC
Dec  1 00:52:02.076 [11558] dbg: plugin: loading
Mail::SpamAssassin::Plugin::ASN from @INC    <-- Here it is.
Dec  1 00:52:02.109 [11558] dbg: plugin: loading
Mail::SpamAssassin::Plugin::AWL from @INC
...
Dec  1 00:54:32.075 [11558] dbg: plugin:
Mail::SpamAssassin::Plugin::ASN=HASH(0xb0d9fd8) implements 'parsed_metadata',
priority 0  (Referenced/used too.)

DNS cache has these entries (1800 seconds later):
22.11.211.140.origin.asn.cymru.com. 13133 TXT
                              "3701 | 140.211.0.0/16 | US | arin | 1990-06-11"
22.11.211.140.ip2asn.sasm4.net. 2334 TXT "as3701"
22.11.211.140.asn.routeviews.org. 56589 TXT "3701" "140.211.0.0" "16"

The entry from spameatingmonkey.net timed out (300 seconds).  Therefore, the
module is loaded, fetched the data, but the variables were NOT set for
substitution into the report function.  That's the bug.

I am using the patch in 6484.  It does set the variables correctly per the
debugging statements.

-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to