[prev in list] [next in list] [prev in thread] [next in thread] 

List:       subversion-issues
Subject:    =?UTF-8?B?W0lzc3VlIDE4MDRdIE5ldyAtIG1haWxlci5weSBkb2Vzbid0IGNhdA==?=
From:       rzigweid () tigris ! org
Date:       2004-03-24 14:58:09
Message-ID: 20040324145809.24310.qmail () tigris ! org
[Download RAW message or body]

http://subversion.tigris.org/issues/show_bug.cgi?id=1804
                  Issue #:|1804
                  Summary:|mailer.py doesn't catch SMTPRecipientsRefused in
                          |finish()
                Component:|subversion
                  Version:|current
                 Platform:|All
                      URL:|
               OS/Version:|Windows 3.1
                   Status:|NEW
        Status whiteboard:|
                 Keywords:|
               Resolution:|
               Issue type:|DEFECT
                 Priority:|P3
             Subcomponent:|tools
              Assigned to:|issues@subversion
              Reported by:|rzigweid






------- Additional comments from rzigweid@tigris.org Wed Mar 24 06:58:09 -0800 2004 \
------- I haven't decided if this is a bug or a feature yet.  I can see some \
advantages to either method.  That  said, I'm adding it as an issue anyway because it \
makes generic project type handling problematic.

finish() of mailer.py does not attempt to catch the exception SMTPRecipientsRefused, \
which is raised if  the recipient of the destined mail does not exist.  This blocks \
any other mails that might be sent out as  well.  Given the commit has already \
happened, and the lack of error/message reporting of the hook  system in general, my \
solution was just to catch the exception and pass.  I'm not sure whether or not  I'm \
satisfied with that solution though, as a message should probably be sent to stderr.

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@subversion.tigris.org
For additional commands, e-mail: issues-help@subversion.tigris.org


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic