Making WordPress.org

Opened 5 weeks ago

Last modified 5 weeks ago

#7770 new defect (bug)

Email addresses entered on trac are not truncated in slack bot.

Reported by: peterwilsoncc's profile peterwilsoncc Owned by:
Milestone: Priority: normal
Component: Trac Keywords:
Cc:

Description (last modified by peterwilsoncc)

If a user enters an email address on trac, the email address is truncated for display when another user views the ticket. For example peterwilsoncc[at]git[dot]wordpress[dot]org becomes peterwilsoncc@...

However the truncation is not reflected when the tickets are cross posted to slack channels such as #core-firehose. To demonstrate: peterwilsoncc@… will display the full email address in #meta-firehose.

My understanding of how the slack bot works suggests these will also be sent in their raw form to to users subscribed to the relevant trac's firehose notifications.

Edit: Sure enough, the email address displayed in the Slack notification

Change History (2)

#1 @peterwilsoncc
5 weeks ago

  • Description modified (diff)

#2 @dd32
5 weeks ago

I'm pondering if we actually need the email truncation?

The Slackbot notifications use the same content as the Email notifications (because that's where it gets it's data from).

The public email archives for the mailing lists has it's own email truncation: https://lists.wordpress.org/pipermail/wp-meta/2024-September/060048.html

Since the Slackbot notifications aren't publicly archived, we can probably leave it as-is.

I'd be more inclined to add the EMAIL_VIEW permission for all Authenticated users on Trac (Leaving it truncated for logged out users) such that logged in users can see the email addresses..

Note: See TracTickets for help on using tickets.