bugcvsreport - Bugs: bug #466, no error message helpful enough...

 
 
Show feedback again

You are not allowed to post comments on this tracker with your current authentification level.

bug #466: no error message helpful enough when call to mail fails

Submitted by:  Mathieu Roy <yeupou>
Submitted on:  Wed 30 Jun 2004 09:44:13 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Priority: 1 - LaterStatus: None
Privacy: PublicAssigned to: Vincent Caron <zerodeux>
Open/Closed: Open

Wed 30 Jun 2004 10:46:09 AM UTC, comment #2:

Indeed, using syslog for runtime errors would be best, since the first place a sysadmin will look to find a problem is definitely there -- with such warning, we would have found this error two days ago :)

If the Net::SMTP method does allow to send mail via the local exim, I find very questionable the choice to make that method default. In my opinion, defaut should definitely be the method that relies on the real SMTP server installed, that's what sysadmin are likely to expect from such program. But maybe there's a Debian perl-packaging specific rule about it, I do not know.

Mathieu Roy <yeupou>
Wed 30 Jun 2004 10:02:19 AM UTC, comment #1:

Sadly, cvsreport cannot report such an error back to the user (ie. client side), because the instance which sends the e-mail is a forked process which polls and gathers the different commits (one per folder, CVS design...) into one changeset.

One has to add the '--debug' flag in its CVSROOT/commitinfo, and read the /tmp/cvsreport.log on the server side to figure out what happens. The 'test' module of cvsreport on Gna! serves that purpose.

Cvsreport already does some preliminary checking in command_mail_check(). I could add something like 'command not found' and warn the user early enough. This would not fix the unsupported '-a' flag though.

Note that the Debian maintainer sponsored and defaulted to the Net::SMTP method. Understandably.

I guess I should syslog runtime errors, and logalert would do the rest :).

Vincent Caron <zerodeux>
Project AdministratorIn charge of this item.
Wed 30 Jun 2004 09:44:13 AM UTC, original submission:

When the call to `mail` fails, cvsreport should print a warning.

Mathieu Roy <yeupou>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by zerodeux (Posted a comment)
  • -unavailable- added by yeupou (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    No Changes Have Been Made to This Item
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup