Hi,
I have use your webform module.
when I install SMTP Authentication Support and PHPMailer. http://drupal.org/project/smtp

when I submit the form, next appear "invalid address ................"

I suppose "invalid address" phrase must in the SMTP Authentication Support and PHPMailer.

when I uninstall the SMTP Authentication Support and PHPMailer ,the webform is working smooth,no error.

is it a bug ?

bug in the smtp or webform?

hope to have your answer.

tks

CommentFileSizeAuthor
#6 smtp-703482.patch889 bytesjrglasgow

Comments

franz’s picture

Category: bug » support

Hi,
I'm no maintainer for webform, so I can't help with it.

How did you configure your SMTP settings?

Which version of PHPMailer are you using? Did you apply the patch?

Try enabling the debug option and report the printed information.

jerome@drupal.org’s picture

Hi,
I have the same issue.
Anybody find a workaround ?

Best Regards,

Jerome

franz’s picture

Status: Active » Postponed (maintainer needs more info)

Please post debug info.

gbilski’s picture

I too got the error invalid address with webform subissions.This error occurs with beta4, downgrading to beta3 fixes it. From the Drupal log: Cannot modify header information - headers already sent by (output started at /var/www/servers/caci/sites/all/modules/smtp/phpmailer/class.phpmailer.php:413) in /var/www/servers/caci/includes/common.inc on line 335.

franz’s picture

Please read the whole post before replying...

How did you configure your SMTP settings?
Which version of PHPMailer are you using? Did you apply the patch?
Try enabling the debug option and report the printed information.

jrglasgow’s picture

Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new889 bytes

I have found the cause of this problem.

Between Beta3 and Beta4 this bit of code was added

  //Hack to fix reply-to issue.
  $properfrom = variable_get('site_mail', '');
  if (!empty($properfrom)) {
    $headers['From'] = $properfrom;
  }
  if (!isset($headers['Reply-To']) || empty($headers['Reply-To'])) {
    $headers['Reply-To'] = $from;
  }

It takes the From address and uses it as the Reply-To address.
E-mail header fields allow the From address to be in the form: Name <name@example.com>
The Reply-To address can only be in the form name@example.com

This patch checks to see if the address is in an improper format and changes it accordingly.

Mark B’s picture

The patch failed to apply for me against 6.x-1.0-beta4:

---------------------------
TortoiseMerge
---------------------------
The patch seems outdated! The file line
    $headers['From'] = $properfrom;
and the patchline
        watchdog('smtp', 'There is no submitted from address.', array(), WATCHDOG_ERROR);
do not match!
---------------------------

but I made the changes manually and my webforms now send nicely.

franz’s picture

Version: 6.x-1.0-beta4 » 7.x-1.x-dev

Thanks for the work

Commited 6.x...

Must be commited on 7.x as well

franz’s picture

Version: 7.x-1.x-dev » 6.x-1.0-beta4
Status: Needs review » Fixed

Commited on 7.x

franz’s picture

Version: 6.x-1.0-beta4 » 7.x-1.x-dev

ops... version bump

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

aparna1234’s picture

hi...
Iam going to develop drupal website but sending emails through exim4(mta) is not done can I use phpmailer to send emails i.e registration and newsletters..? can anyone help me....?