Forum OpenACS Q&A: Possible bug with acs-mail-lite?

Collapse
1: Possible bug with acs-mail-lite?
Posted by Nima Mazloumi on 04/10/08 10:41 AM
Hi,

I tried to use

set extra_headers [ns_set new]
acs_mail_lite::send -to_addr $to \
-from_addr $from \
-subject $subject \
-body $body \
-mime_type "text/html" \
-extraheaders $extra_headers

to send out an email. But randomly I get the error:

can't read "state(header)": no such variable

This comes from the mime package from tcllib.

The email is sent out but without subject, body and with undisclosed-recipients.

I am using latest code in HEAD.

Collapse
2: Re: Possible bug with acs-mail-lite? (response to 1)
Posted by Emmanuelle Raffenne on 04/10/08 11:12 AM
Hi Nima,

Which version of Tcllib are you using?

Collapse
3: Re: Possible bug with acs-mail-lite? (response to 2)
Posted by Nima Mazloumi on 04/10/08 11:22 AM
I use 1.10
Collapse
4: Re: Possible bug with acs-mail-lite? (response to 1)
Posted by Emmanuelle Raffenne on 04/10/08 11:35 AM
Nima,

It looks like you pass an empty set to extraheaders. This arg is optional, if there's no extra headers, don't set it. On the other hand, extraheaders should be a list of pairs.

Collapse
5: Re: Possible bug with acs-mail-lite? (response to 4)
Posted by Nima Mazloumi on 04/10/08 12:28 PM
This error occurs even without the extra headers.
Collapse
6: Re: Possible bug with acs-mail-lite? (response to 5)
Posted by Emmanuelle Raffenne on 04/10/08 12:37 PM
Nima,

OK. I'm going to install tcllib 1.10 then. I don't have that problem with tcllib 1.8 and IIRC I didn't either when I tested with 1.10 but I may have missed something.

Collapse
7: Re: Possible bug with acs-mail-lite? (response to 1)
Posted by Emmanuelle Raffenne on 04/12/08 01:37 PM
Hi Nima and all,

I've committed a fix at oacs-5-4. Stefan and I tested with both tcllib 1.8 and 1.10 and it appears to work correctly now (no more random errors) but I'd appreciate if you test too and confirm the fix.

Since it was a random error, it's hard to say what caused it. Anyway, the fix consists of reordering in a more logical way the steps to build the message and add the headers. It looks like tcllib is very picky with that.