Skip to:
Content

BuddyPress.org

Opened 14 years ago

Closed 10 years ago

#1944 closed enhancement (wontfix)

image-link within e-mail notification ?

Reported by: erich73's profile erich73 Owned by:
Milestone: Priority: trivial
Severity: normal Version:
Component: Core Keywords:
Cc:

Description

not sure if this is intended or a bug:

a user replied to one of my post with the posting of an image:

http://testbp.org/members/erich73/activity/66127/

At the e-mail notification which I did receive, the link to the image was included within the e-mail. Not sure if the link to an image should be included within the mail-notification ?

the following is the e-mail text:

*

luccame replied to one of your updates:

"<img src="http://28.media.tumblr.com/tumblr_kxqjxyFV2T1qzpwi0o1_500.png" />"

To view your original update and all comments, log in and visit: http://testbp.org/activity/p/66127/


To disable these notifications please log in and go to: http://testbp.org/members/erich73/settings/notifications/

*

Change History (3)

#1 @apeatling
14 years ago

  • Milestone changed from 1.2 to 1.3

Emails are sent as plain text, so this will be the case. Bumping this as we should consider this for 1.3.

#2 @boonebgorges
13 years ago

  • Component set to Core
  • Milestone changed from 1.3 to Future Release
  • Type changed from defect to enhancement

If we start stripping tags from email notifications, we are bound to run into all sorts of problems where we actually affect and distort the content of the message. IMO, more sophisticated filters (that is, more sophisticated than strip_tags() ) are probably the domain of plugins.

#3 @boonebgorges
10 years ago

  • Milestone Future Release deleted
  • Resolution set to wontfix
  • Severity set to normal
  • Status changed from new to closed

BP doesn't natively offer any support in the interface for adding HTML content, so I think it's outside our purview to remove/sanitize that content in emails. Marking wontfix.

Note: See TracTickets for help on using tickets.