Edgewall Software
Modify

Opened 19 years ago

Closed 16 years ago

Last modified 15 years ago

#1359 closed enhancement (duplicate)

Add ticket change email headers to allow more intelligent filtering of ticket notification emails

Reported by: chris@… Owned by: Emmanuel Blot
Priority: normal Milestone:
Component: notification Version: 0.8.1
Severity: normal Keywords: email headers needinfo
Cc: chris@…, jan@… Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Basically, Trac sends (if configured properly) emails to the users each time a ticket is modified. In my case, I wanted to filter the tickets a bit, to identify tikets being closed or assigned to me.

Unfortunately, if your mail goes to an IMAP server without server filters, most email clients can't filter on the contents of the body, where the specific field changes are enumerated. Plus that would be, well, ugly.

This change adds additional X-Trac email headers each time a ticket is changed.

These headers are of the format:

X-Trac-Ticket-Change-<field>

with the new value of the field as the header value.

This allows for easy client side filtering, nd handles the case when multiple changes are made in one submission (you get multiple headers in the email, 1 per field)

Thanks for watching!

Attachments (1)

trac.patch (1.2 KB ) - added by chris@… 19 years ago.
Patch against the trunk to add additional email headers for ticket notifications

Download all attachments as: .zip

Change History (13)

by chris@…, 19 years ago

Attachment: trac.patch added

Patch against the trunk to add additional email headers for ticket notifications

comment:1 by anonymous, 19 years ago

Summary: Add ticket change email headers to allow more intelligent filtering of tiket notification emailsAdd ticket change email headers to allow more intelligent filtering of ticket notification emails

comment:2 by anonymous, 19 years ago

Cc: chris@… added

comment:3 by Christian Boos, 17 years ago

Component: generalnotification
Milestone: 0.12
Owner: changed from Jonas Borgström to Emmanuel Blot

Consider for 0.12.

comment:4 by anonymous, 17 years ago

Cc: jan@… added

comment:5 by Emmanuel Blot, 17 years ago

Keywords: needinfo added; notification ticket removed
Status: newassigned

I'm wondering if is worth overloading the SMTP headers with potentially many fields.

Couldn't a single header, X-Trac-Ticket-Changes: with a comma separated list of modified fields as value be enough?

comment:6 by chris@…, 17 years ago

A list *might* be enough, but I used this change extensively to do really intelligent filtering on a project where I got hundreds of notifications a day. It was far easier to setup filters for specific headers; parsing (especially with server side filtering agents like Sieve) might be much harder. Besides, aren't headers pretty well understood?

comment:7 by Emmanuel Blot, 16 years ago

See also #6516 which contains another patch

comment:8 by Christian Boos, 16 years ago

For filtering, doesn't #6516 makes more sense?

Proposing to close this one as duplicate of #6516 (superseded by, actually).

comment:11 by chris@…, 16 years ago

The original urge for writing this patch the way it is is exactly as cited above: "I used this change extensively to do really intelligent filtering on a project where I got hundreds of notifications a day. It was far easier to setup filters for specific headers; parsing (especially with server side filtering agents like Sieve) might be much harder."

Having lots of headers made it easy for vanilla filtering tools (like Sieve engines) to filter in some really complex ways; if it was a list it would have been far more tedious.

comment:12 by Christian Boos, 16 years ago

chris, in case you had some doubts, I was not saying that the meaning of the "text" of comment:9 was spam, rather than a spammer had the good idea to copy the text from (your) comment:6 as a whole before adding a spam link at the end of it, in order to circumvent the spam protection system…

comment:13 by Christian Boos, 16 years ago

Resolution: duplicate
Status: assignedclosed

See superseder #6516.

comment:14 by Christian Boos, 15 years ago

Milestone: 0.13

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Emmanuel Blot.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Emmanuel Blot to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.