https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7166
Kevin A. McGrail <kmcgr...@pccc.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|blocker |normal
Priority|P5 |P3
--- Comment #15 from Kevin A. McGrail <kmcgr...@pccc.com> ---
(In reply to Mark Martinec from comment #13)
> So a quick fix is to update the built-in list of top-level
> domains, at least removing the .tp and .gov.tp :
>
> removed domain .tp from hard-coded default in Conf.pm,
> removed debug output from t/uri_text.t
> Sending lib/Mail/SpamAssassin/Conf.pm
> Sending t/uri_text.t
> Committed revision 1672316.
Sorry, I knew this part but was leaving that in because it was showing an issue
that when we remove the old RegistrarBoundaries functionality and rely solely
on 20_aux_tlds (currently ONLY in their as a transition for 3rd party plugins),
ALL the tests in uri_text.t would presumably fail.
The good news is that your work really helps me confirm this is NOT a release
blocker.
The out of sync .tp issue brought to light a bug. Syncing the two sources of
TLDs so that the test passes, isn't a fix. I think we need to fix the test
implementation so that we copy 20_aux_tlds.cf.
Thanks a lot also for letting us duplicate this locally as that should make it
easier to resolve.
I've added this as a dependency for the release of 3.4.2 noted on bug 7170 and
lowered this ticket's priority and remove the blocker status for 3.4.1.
regards,
KAM
--
You are receiving this mail because:
You are the assignee for the bug.
