When Diffusion sends out an email about repository updates (for example, when a revision is closed), it contains several links to the server instance for more information.
Often for us (90% of the time estimated), there's a minor issue that occurs around the point of the link to the Phabricator server. This link contains a new line or whitespace character so the link is not clickable. Occasionally this space character is in the title instead of the link tag. This may be because there is a limit to the number of characters the mail client (Apple mail in this case) will use before it inserts a new line character.
Example email (scrubbed). The below example is an email that was copied and pasted and sensitive information was removed, however the issue is exact as shown. The recipient of this email would not be able to click on the link https://myserver.something.com/D12345 due to the end line.
This behavior does not happen all the time, only when there is a suffient amount of characters without a newline before the link. As mentioned above, sometimes the new line occurs after the link within the title and the mail client renders this as a space. In the example used previously, the link would look like https://myserver.so mething.com/D12345. It will still be clickable in the mail client though unlike the previous example.
A potential solution to the issue (just a guess) may be to just add an end line character at an appropriate spot before this link so it renders correctly.
Date: Mon, 26 Sep 2016 15:57:07 -0700 To: <scrubbed> From: <scrubbed> Subject: Re: [Differential] [Closed] D12345: <scrubbed> Message-id: <scrubbed> X-Priority: 3 X-Phabricator-Sent-This-Message: Yes X-Mail-Transport-Agent: MetaMTA X-Auto-Response-Suppress: All X-Phabricator-Mail-Tags: <differential-updated>, <differential-committed> X-Herald-Rules: <60> X-Phabricator-Projects: <scrubbed> X-Phabricator-To: <scrubbed> Precedence: bulk In-reply-to: <scrubbed> References: <scrubbed> Thread-index: <scrubbed> --Boundary_(ID_oj3/m6Rep4skVmJAwianMA) Content-type: text/html; CHARSET=US-ASCII Content-transfer-encoding: 7BIT <table><tr><td style="">This revision was automatically updated to reflect the committed changes.<br />Closed by commit <scrubbed></td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="<scrubbed>" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="<scrubbed>" rel="noreferrer">https://<scrubbed></a></div></div><br /><div><strong>REPOSITORY</strong><div><div>r<scrubbed></div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://myserver.someth ing.com/D12345" rel="noreferrer">https://myserver.something.com/D12345</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div><scrubbed><br /> [...] --Boundary_(ID_oj3/m6Rep4skVmJAwianMA)--
Version:
phabricator 34af66122e47f3194156847e36e583b5d67cfcda (Aug 15 2016)
arcanist de1231c62a4fa4935a33e73dc95bb352ce40a0a0 (May 31 2016)
phutil 5a9becb0920ec3983925f3617f76933702a925d5 (Jun 7 2016)