Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I had to change the from header in action_send_email because some smtp servers choke on the "< .*>" syntax.
I simply changed
$from = "$site_name <" . variable_get('site_mail', ini_get('sendmail_from')) . '>';
to
$from = variable_get('site_mail', ini_get('sendmail_from'));
to get things to work properly.
Comment | File | Size | Author |
---|---|---|---|
#3 | actions.inc_bad_from.patch | 666 bytes | pearcec |
Comments
Comment #1
pearcec CreditAttribution: pearcec commentedThis is how it is done is user.module. I would change it in CVS just for consistency.
Comment #2
jvandyk CreditAttribution: jvandyk commentedFixed. Thanks.
Comment #3
pearcec CreditAttribution: pearcec commentedI had another issue with the from address. I don't think I need to open another issue. I already have the patch.
I was getting this as the from address:
"=?UTF-8?B?RnJvbTogaW5mb0BhYm91dC5pbmZvClJlcGx5LXRvOiBpbmZvQGFib3V0LmluZm8=?=.=?UTF-8?B?ClgtTWFpbGVyOiBEcnVwYWwKUmV0dXJuLXBhdGg6IGluZm9AYWJvdXQuaW5mbwo=?=.=?UTF-8?B?RXJyb3JzLXRvOiBpbmZvQGFib3V0LmluZm8=?=@localhost.localdomain"
When I compared it to the user.module, it simply used $from in the drupal_mail function call. Instead of having all the other header information. Seems drupal_mail puts all that header information in anyway.
Comment #4
Pomliane CreditAttribution: Pomliane commentedThis version of Actions is not supported anymore. The issue is closed for this reason.
Please upgrade to a supported version and feel free to reopen the issue on the new version if applicable.
This issue has been automagically closed by a script.