History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: CCNET-1046
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Ross Patterson
Reporter: Ross Patterson
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
CruiseControl .NET

Email publisher ignores IntegrationStatus.Exception

Created: 02/Jan/08 09:07 AM   Updated: Sunday 12:48 PM
Component/s: Publisher
Affects Version/s: 1.3
Fix Version/s: 1.4

Original Estimate: Unknown Remaining Estimate: Unknown Time Spent: Unknown
Issue Links:
Dependency
 
This issue is depended on by:
CCNET-1650 failed integration and exception duri... Minor Closed


 Description  « Hide
Steve Trefethen (strefethen@gmail.com) reported to the ccnet-devel list:

In EmailMessage.cs the method EmailMessage.Recipients does not explicitly handle IntegrationStatus.Exception. It looks like it will be handled in the if(BuildStateChanged()) case but not specifically as an Status type like Failed etc. I've created a sourcecontrol plugin that uses FTP to query for the existence of files and occasionally the FTP connection fails and throws an exception but I'm not getting email notifications in those cases. My email publisher is configured for "Failed" builds and I'm getting notifications successfully in that case but not for IntegrationStatus.Exception.

 All   Comments   Work Log   Change History      Sort Order:
Ross Patterson [02/Jan/08 10:03 AM]
Subversion revision 3585 changes the email publisher to treat IntegrationStatus.Exception as another case of "failure".