Bug #13274
closedMessy links in CAC email notifications
0%
Description
Hi Ray,
We're seeing an issue where emails forwarded with embedded links to CAC groups result in emails where the URLS appear in full. This can be problematic when those URLs are long, as is the case for the GC system, which adds Proofpoint spam prevention info to each link. I've attached a few screenshots and original emails to this message:
1. Original email (screenshot + raw content)
2. Commons email notification (screenshot + raw content)
Is there any way to improve this situation that you can think of? The issue is that the Commons email notifications become almost unreadable with the long links.
Files
Updated by Boone Gorges about 4 years ago
I'd like Ray to chime in as well, but I'd say that there's no real way around this. What appears to be happening is that the Forward feature in the mail client (whether the web client or some mail application on a device) is getting its content from the plaintext version of the email. And ProofPoint injects its wrappers directly into the text of the email, which means that if you're forwarding from a GC address, the long URLs are actually in the email content. The whole thing - ProofPoint URL injection and forwarding - happens well after the Commons has sent the email.
One thing worth mentioning is that this probably only surfaces in cases where the mail client uses plaintext content for forwarding. This may be configurable on a per-client basis, and it is not the default behavior for some clients, so it's likely that this is not a super widespread issue.
Updated by Matt Gold about 4 years ago
Hi Boone -- following our discussion of this during the dev call, I'm attaching a screenshot of what I see when I click "forward" in my email client. As you'll see, the links are still embedded -- it looks exactly like the original email I received in my inbox.
I am using Gmail through the web interface on the Chrome browser
Updated by Boone Gorges about 4 years ago
Thanks, Matt. This info helps to narrow it down a bit. It sounds like maybe this is something that RBE can handle more gracefully.
Updated by Raymond Hoh about 4 years ago
- Target version set to 1.17.8
Got distracted with other things and missed this one. Apologies!
RBE parses the plain-text content of the email before posting it to bbPress. Combine that with how ProofPoint changes the content of the email and you get the messy output in the forums and emails. Switching over to HTML email parsing would involve refactoring parts of RBE, which would take longer.
However, it is possible to filter the output on the frontend and also in group email blasts. I have a fix on production, which changes the ProofPoint content from the following:
Build: October – December 2020 Launch: 4 December 2020 at 1:00-2:00pm (EST) – Register here <https://urldefense.proofpoint.com/v2/url?u=https-3A__columbiauniversity.zoom.us_webinar_register_WN-5FEWyy-2DI-2DYTaqdXhYxs0TXGQ&d=DwMF-g&c=mRWFL96tuqj9V0Jjj4h40ddo0XsmttALwKjAEOCyUjY&r=PEBMbbahNHugBpfSYHnAddgsTYO89UN3skuC4DsKVLk&m=0Neg8F-YHmONFyb6N2hW2GyAc6zPBPHyJQAmizfDwlU&s=xC1LnwfBOFfEDJYfmSf0glO3OI4s1pYftVAAtbrN3sI&e=>
To:
Build: October – December 2020 Launch: 4 December 2020 at 1:00-2:00pm (EST) – Register here (columbiauniversity.zoom.us)
(columbiauniversity.zoom.us) is a hyperlink to the Zoom webinar. The sample above can be seen at https://commons.gc.cuny.edu/?p=102566.
As you can see, I've changed the long ProofPoint URL to only the domain of the actual link instead of the full URL (columbiauniversity.zoom.us/webinar/register/XXX). However, let me know if we would prefer the full URL to be displayed, or if we want to add something to the link to denote that it is a link. For example, (LINK - columbiauniversity.zoom.us).
Updated by Matt Gold about 4 years ago
that's really awesome, Ray! Thank you! As long as it is clear it is a URL, I would say that we don't need to include "LINK"
Updated by Raymond Hoh about 4 years ago
- Status changed from Assigned to Resolved
Thanks for the feedback, Matt.
I've committed the change, which is already live on production: https://github.com/cuny-academic-commons/cac/commit/19f58d6124fa4b52db329270982ca2e9f4ebba74