[ic] email-raw usertag broken for Net::SMTP FIX

Peter peter at pajamian.dhs.org
Sat Apr 30 21:57:16 EDT 2005


On 04/30/05 07:50, Mike Heins wrote:
> Quoting Peter (peter at pajamian.dhs.org):
> 
>>Well technically it's not broken because it does specify in the docs 
>>that it uses sendmail.  But if you use Net::SMTP for sending out emails 
>>then no emails will be sent via this tag.  I've modified the tag to work 
>>with Net::SMTP as well (feel free to tweak and change or whatever) and 
>>would like to submit this to be included in future versions of 
>>interchange.  Here's a patch to IC 5.2.
>>
>>Note that in theory you could eliminate all bbut the middle clause to 
>>the if/elsif/else statement, but i wanted to keep the original 
>>functionality of being able to send directly to sendmail rather than 
>>simulate it through the send_mail function.
> 
> 
> This seems like a useful idea. It just so happens that I have just
> enhanced the [email ...] tag to do attachments with MIME::Lite,
> and haven't yet committed it. It uses [email-raw ..] to send,
> so I can put both in at the same time.

One thing I failed to mention before is that my patch fails to account 
for the possibility of folded headers.  I didn't think too much of it 
because I don't anticipate anyone using the email-raw tag to send emails 
out with folded headers, but if you want to make it work with then an 
additional regexp to unfold them should fix it.  Something like the 
following before the split line should do it:

$headers =~ s/\r?\n([ \t])/$1/s;

Peter


More information about the interchange-users mailing list