Edgewall Software
Modify

Opened 12 years ago

Closed 12 years ago

#5126 closed enhancement (fixed)

trac should have an option to obscure all email addresses

Reported by: anonymous Owned by: Christian Boos
Priority: normal Milestone: 0.11
Component: ticket system Version: devel
Severity: minor Keywords: email obfuscation CC field
Cc: Branch:
Release Notes:
API Changes:

Description

Currently if you add an email address to a ticket's cc, it will be publicly displayed on the webpage. It would be nice if there were a way to obscure this to prevent the addresses from being harvested by spammers. I've attached a prototype to demonstrate the functionality I'd like, but it's not well integrated with the rest of the system.

Thanks!

Attachments (3)

email.patch (1.3 KB ) - added by anonymous 12 years ago.
1459.diff (1.3 KB ) - added by Christian Boos 12 years ago.
Updated the CC: rendering patch that eblot posted on #1459, which adds e-mail obfuscation (#153 style)
5126.diff (2.2 KB ) - added by Emmanuel Blot 12 years ago.
Combination of both patches

Download all attachments as: .zip

Change History (14)

by anonymous, 12 years ago

Attachment: email.patch added

comment:1 by Christian Boos, 12 years ago

Milestone: 0.11
Owner: changed from Jonas Borgström to Christian Boos
Severity: normalminor

Thanks, that was one of the missing pieces for #153.

I still hope that we would provide flexible renderers for ticket fields, but in the meantime, this should be a start.

(to be done after the WorkFlow merge)

by Christian Boos, 12 years ago

Attachment: 1459.diff added

Updated the CC: rendering patch that eblot posted on #1459, which adds e-mail obfuscation (#153 style)

comment:2 by Christian Boos, 12 years ago

Keywords: email obfuscation CC field added
Owner: changed from Christian Boos to Emmanuel Blot
Version: devel

by Emmanuel Blot, 12 years ago

Attachment: 5126.diff added

Combination of both patches

comment:3 by Emmanuel Blot, 12 years ago

I've combined both patches in one file, but I should say that I'm a bit skeptical about the cc address obfuscation:

We can hide the domain in the CC field of the ticket property box, but we cannot do the same in the CC: text field: email addresses appear with the FQDN, and we cannot mask the domain here.

comment:4 by Christian Boos, 12 years ago

Wait, why don't you use format_author like shown in attachment:1459.diff? (that function takes into account the EMAIL_VIEW permission).

Then, for the CC: text field, we could go one little step further and only show it in full for TICKET_ADMINs and have a checkbox: [ ] Add me to CC or [ ] Remove me from CC depending if the current e-mail address if found or not in the field. No datamodel change for that, so this is not really the plan for #1459 but more a workaround for being able to correctly hide the e-mails.

comment:5 by Emmanuel Blot, 12 years ago

I guess we both are not refering to the same patch: none of the patches described here contain format_author.

Maybe attachment:ticket:153:privacy-r4476.diff is the right candidate?

comment:6 by Christian Boos, 12 years ago

That patch on #153 (or similar) is applied since a long time; no I really meant attachment:1459.diff, my slightly modified version of your patch attached here.

comment:7 by Christian Boos, 12 years ago

Owner: changed from Emmanuel Blot to Christian Boos

I'm refactoring a bit the ticket system so that the fields would be rendered in the template in a way similar to browser properties. So this feature will be better done inside the web_ui not in the template.

comment:8 by Christian Boos, 12 years ago

Status: newassigned

Done in r5485 (FieldRefactoring branch)

comment:9 by ThurnerRupert, 12 years ago

see also #5378 for ajax support.

comment:10 by Christian Boos, 12 years ago

Milestone: 0.11.10.11

Nearly there in trunk with r5812 and r5813.

Actually, r5812 was about the display of CC: changes in the HTML Change History and in the RSS view of the ticket, not yet when showing changes between arbitrary versions of the Ticket (i.e. the ticket diff view).

comment:11 by Christian Boos, 12 years ago

Resolution: fixed
Status: assignedclosed

Finalized in r5814.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christian Boos.
The resolution will be deleted.
to The owner will be changed from Christian Boos 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.