Edgewall Software
Modify

Ticket #7499 (new enhancement)

Opened 4 years ago

Last modified 18 months ago

Notification mail header "Precedence" should be configurable

Reported by: i.bardarov@… Owned by:
Priority: low Milestone: next-major-0.1X
Component: notification Version: 0.11
Severity: trivial Keywords: notification
Cc: thijstriemstra
Release Notes:
API Changes:

Description (last modified by rblank) (diff)

I want to use Trac to notify a mailing list of ticket change.

Every time when Trac send email I got this error/success in the mail log:

delivery 1118: success: Precedence:_junk_-_message_ignored/did_0+0+1/

To bypass this we have to comment in notification.py

headers['Precedence'] = 'bulk'

It should be great if you can include this as a config option in feature version.

Thanks

Attachments

Change History

comment:1 Changed 4 years ago by anonymous

  • Keywords ui, javascript added
  • Owner changed from eblot to anonymous
  • Severity changed from trivial to normal
  • Status changed from new to assigned

comment:2 Changed 4 years ago by anonymous

  1. abc

1. def

  1. 233

comment:3 Changed 4 years ago by rblank

  • Description modified (diff)
  • Keywords notification added; ui, javascript removed
  • Milestone set to 0.13
  • Owner anonymous deleted
  • Severity changed from normal to trivial
  • Status changed from assigned to new

An improved notification system is scheduled for 0.13.

comment:4 Changed 3 years ago by eblot

#8609 flagged as duplicate

comment:5 follow-up: Changed 3 years ago by cboos

(#8609 has a patch)

comment:6 in reply to: ↑ 5 Changed 3 years ago by eblot

Replying to cboos:

(#8609 has a patch)

Note that disabling this header is a bad workaround that may lead to other issues, such as the ones for which these headers have been initially introduced. I'm not sure Trac should send 'limited' emails to circumvent issues in other software…

comment:7 follow-up: Changed 3 years ago by dandre@…

I have tested replacing

headers['Precedence'] = 'bulk'

with

headers['Precedence'] = 'list'

and the mail is correctly delivered. So instead of my previous patch in #8609 I suggest to set precedence header to list.

I suggest to change milestone from 0.13 to 0.11.6 too

comment:8 Changed 18 months ago by thijstriemstra

  • Cc thijstriemstra added
  • Type changed from defect to enhancement

+1 for making this configurable.

comment:9 in reply to: ↑ 7 Changed 18 months ago by thijstriemstra

Replying to dandre@…:

I have tested replacing

headers['Precedence'] = 'bulk'

with

headers['Precedence'] = 'list'

Patch for 0.12-stable:

  • trac/notification.py

     
    409409        headers['X-Trac-Version'] =  __version__ 
    410410        headers['X-Trac-Project'] =  projname 
    411411        headers['X-URL'] = self.env.project_url 
    412         headers['Precedence'] = 'bulk' 
     412        headers['Precedence'] = 'list' 
    413413        headers['Auto-Submitted'] = 'auto-generated' 
    414414        headers['Subject'] = self.subject 
    415415        headers['From'] = (self.from_name or projname, self.from_email) 
View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as new
as The resolution will be set. Next status will be 'closed'
to The owner will be changed from (none). Next status will be 'new'
The owner will be changed from (none) to anonymous. Next status will be 'assigned'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.