I found on Sourceforge that what was working with mailnotify with one user stopped when further ones were added. Here are my notes that lead me to conclude that the problem is probably due to the fact that the body of the message being passed to smtp->data($body) actually contains the addresses to which the message is addressed.

To debug mail on Source Forge:

rm ~/conceptmapping-group/twiki/data/ConceptMapping/.mailnotify
perl -d ~/conceptmapping-group/cgi-bin/twiki/mailnotify
b 156
c
s
s (to Step through until Net::SMTP loads.)
b 95
c
s
b 131
c
n
$smtp->debug(100)
n

Revealed:
Net::SMTP=GLOB(0x860c040)<<< 550 Syntax error in 'To' header: malformed address: conceptmapping@yahoogroups.com\n may not follow Martin.Cleaver@BCS.org.uk : failing address is: Martin.Cleaver@BCS.org.uk conceptmapping@yahoogroups.com

Not sure what's going on. Earlier I got:

141:		  $smtp->quit();
  DB<11> n
Net::SMTP=GLOB(0x860c040)>>> MAIL FROM:
Net::SMTP=GLOB(0x860c040)<<< 250  is syntactically corr
ect
TWiki::Net::_sendEmailByNetSMTP(Net.pm:133):
133:		  $smtp->to( @to, { SkipBad => 1 } );
  DB<11> n
Net::SMTP=GLOB(0x860c040)>>> RCPT TO:
Net::SMTP=GLOB(0x860c040)<<< 250  is syntactically corr
ect
Net::SMTP=GLOB(0x860c040)>>> RCPT TO:
Net::SMTP=GLOB(0x860c040)<<< 250  is syntactically
 correct
TWiki::Net::_sendEmailByNetSMTP(Net.pm:134):
134:		  $smtp->data( $data );
  DB<11> n
Net::SMTP=GLOB(0x860c040)>>> DATA
Net::SMTP=GLOB(0x860c040)<<< 354 Enter message, ending with "." on a line by itself
Net::SMTP=GLOB(0x860c040)>>> From: Martin.Cleaver@BCS.org.uk
Net::SMTP=GLOB(0x860c040)>>> To: Martin.Cleaver@BCS.org.uk conceptmapping@yahoogroups.com
Net::SMTP=GLOB(0x860c040)>>> Subject: TWiki.ConceptMapping - Automated notification of topic changes
Net::SMTP=GLOB(0x860c040)>>> MIME-Version: 1.0
Net::SMTP=GLOB(0x860c040)>>> Content-Type: multipart/mixed; boundary="------------2D594AE113AD25493C2C7246"

Which indicates to me that it did like the multiple Tos on the line in the body of the message. Very strange, seems that the email is being prepared to include headers and that the mail system does not allow this, perhaps it does not like the lack of commas between the multiple recipient names.

Test: remove one of the names so that only one person gets notfied.

That time it worked! It only had one recipient: Net::SMTP=GLOB(0x86a3084)>>> . Net::SMTP=GLOB(0x86a3084)<<< 250 OK id=14bv4Y-00011E-00

Perhaps should remove headers or use multiple sends, one for each user.

-- MartinCleaver - 10 Mar 2001

Does adding a comma

Martin.Cleaver@BCS.org.uk conceptmapping@yahoogroups.com

here help?

-- NicholasLee - 11 Mar 2001

I have seen some problems with the new SMTP code without Net::SMTP installed. Symptom is that new registrants do not get sent an email.

-- SteveRoe - 12 Mar 2001

Steve, could you expand the details on your installation.

-- NicholasLee - 12 Mar 2001

Nicholas: Yup. This worked:

Mailnotify:
59c59
<	  my $notifylist = &wiki::getEmailNotifyList($webName);
---
>	  my $notifylist = &wiki::getEmailNotifyList($webName); $notifylist =~ s/ /, /g;

Thanks!

Martin.

---

Which version of the code are you using? The current beta has this and this:

  my @notifylist = &TWiki::getEmailNotifyList($webName);
[...]
  my $notifylist = join ', ', @notifylist;

You could probably replace the regexp sub. with a join in your modification.

-- NicholasLee - 22 Mar 2001

A slightly bastardised copy of the December 2000 release ;^)

-- MartinCleaver - 22 Mar 2001

As an FYI and in case anyone is searching, I had this exact problem when I switched my mail transport from qmail to exim. The RFC for mail messages says commas are required. Qmail accepts spaces as a valid separator whereas Exim is more stringent.

I patched wiki.pm to do the same as above.

-- BruceWilson - 13 May 2001

TopicClassification:

BugResolved