>> Attached also is tld_lists.pl, a companion file that needs to be dropped >> in lib/Qpsmtpd/ which provides the list of first, second, and third >> level >> TLDs that we care about. It's derived from our URIBL datafeed as well >> as > > Would they be ok if this was > released as an independent CPAN module? Either way, can we > structure this as an API instead of just an include file?
I've already pretty much done the work to access tld_lists as an API, but I'm now doubting the use unless it is released to CPAN: it really makes the most sense to have Qpsmtpd::TLDLists.pm itself still pull in the contents of tld_lists.pl as I posted them, in a 'require', because those contents are created by an automated script. It seems silly to require the modification of said automated script if we want it to generate different logic. But when Qpsmtpd::TLDLists require's tld_lists.pl, then comes the question 'if Qpsmtpd::TLDLIsts is the only thing that accesses this data, and plugins/uribl is the only thing that uses Qpsmtpd::TLDLists, then why doesn't plugins/uribl just read the data itself? You guys can have it whichever way you want it but I say, don't be afraid to treat a .pl file as if it were basically a static, shipped config file in a format easily read by perl plugins :) -Jared
