Page MenuHomePhabricator
Feed Advanced Search

Jan 11 2019

d.maznekov updated the question details for Q646: (stable) Promote 2019 Week 1 - don't work mail.
Jan 11 2019, 6:50 AM · Mail
d.maznekov asked Q646: (stable) Promote 2019 Week 1 - don't work mail.
Jan 11 2019, 6:35 AM · Mail

Jan 5 2019

epriestley added a revision to T12404: Implement a first-party SMTP client: D19961: Update the SMTP (PHPMailer) adapter for the new mail API; remove "encoding" and "mailer".
Jan 5 2019, 3:25 PM · Infrastructure, Mail
epriestley added a revision to T5969: Add timeouts to service calls to external mailers (was: PhabricatorMetaMTAWorker may hang indefinitely if "sendmail" hangs indefinitely): D19960: Upgrade Sendgrid to the modern mailer API; removes "api-user" option.
Jan 5 2019, 2:32 PM · Mail
epriestley added a revision to T5969: Add timeouts to service calls to external mailers (was: PhabricatorMetaMTAWorker may hang indefinitely if "sendmail" hangs indefinitely): D19959: Update Mailgun adapter for the new mail adapter API.
Jan 5 2019, 12:59 PM · Mail

Jan 4 2019

epriestley added a comment to T13234: Application email addresses may shadow user email addresses.

When Phabricator receives the mail, it doesn't know which "To" or "Cc" actually caused delivery

Jan 4 2019, 10:45 PM · Mail, Security
epriestley triaged T13234: Application email addresses may shadow user email addresses as Low priority.
Jan 4 2019, 10:39 PM · Mail, Security
epriestley added a revision to T5969: Add timeouts to service calls to external mailers (was: PhabricatorMetaMTAWorker may hang indefinitely if "sendmail" hangs indefinitely): D19956: Update Postmark adapter for multiple mail media.
Jan 4 2019, 8:09 PM · Mail
epriestley added a revision to T9141: Rename "MetaMTA" to "Mail": D19954: Rename "MetaMTA" mail attachments and add more mail message objects.
Jan 4 2019, 1:34 PM · Restricted Project, Mail

Jan 3 2019

epriestley claimed T13066: Refine inbound mail error behaviors, distinguishing between actual mail sender and acting-as sender.
Jan 3 2019, 7:52 PM · Mail
epriestley added a revision to T7477: Handle inbound email with phabricator address in the CC: D19952: Allow multiple mail receivers to react to an individual email.
Jan 3 2019, 7:47 PM · llvm, Restricted Project, Restricted Project, Mail
epriestley added a revision to T13066: Refine inbound mail error behaviors, distinguishing between actual mail sender and acting-as sender: D19952: Allow multiple mail receivers to react to an individual email.
Jan 3 2019, 7:47 PM · Mail
epriestley claimed T7477: Handle inbound email with phabricator address in the CC.
Jan 3 2019, 6:49 PM · llvm, Restricted Project, Restricted Project, Mail
epriestley added a revision to T7477: Handle inbound email with phabricator address in the CC: D19950: Consolidate some application email receiver code in preparation for API changes.
Jan 3 2019, 5:56 PM · llvm, Restricted Project, Restricted Project, Mail
epriestley added a revision to T7477: Handle inbound email with phabricator address in the CC: D19949: Remove "metamta.insecure-auth-with-reply-to" Config option.
Jan 3 2019, 4:45 PM · llvm, Restricted Project, Restricted Project, Mail
epriestley added a revision to T7477: Handle inbound email with phabricator address in the CC: D19948: Extract some email address utility code from the receiver stack.
Jan 3 2019, 4:06 PM · llvm, Restricted Project, Restricted Project, Mail
epriestley added a revision to T7477: Handle inbound email with phabricator address in the CC: D19947: Drop empty inbound mail at the beginning of the receive workflow, not inside object handlers.
Jan 3 2019, 2:28 PM · llvm, Restricted Project, Restricted Project, Mail
klimek added a comment to T4926: When emails are sent to new people (or lists) they should contain the change description and patch.

For the record: I'd like us to get away from maintaining our own phab fork, and I agree that means probably migrating to a different system at some point.

Jan 3 2019, 11:18 AM · llvm, Mail, Differential

Jan 2 2019

epriestley moved T11782: Inconsistent mail notification subject ([Updated] vs [Merged]) depending on from which task a duplicate task was merged from Backlog to Soon? on the Mail board.
Jan 2 2019, 10:39 PM · Mail, Maniphest, Bug Report
chandlerc added a comment to T4926: When emails are sent to new people (or lists) they should contain the change description and patch.

FWIW, I think describing these as "non mainstream" and the entire description of these communities sounds to me ... somewhere between biased and pejorative. Makes it hard for me to keep engaging in the discussion.

Jan 2 2019, 9:17 PM · llvm, Mail, Differential
epriestley added a comment to T4926: When emails are sent to new people (or lists) they should contain the change description and patch.

Maybe worth a mention -- GHC is another somewhat-similar project and moving to GitLab (e.g., https://mail.haskell.org/pipermail/ghc-devs/2018-October/016425.html). It sounds like GitLab is currently better able to spend resources to support free users with relatively non-mainstream use cases than we are.

Jan 2 2019, 6:38 PM · llvm, Mail, Differential
emaste added a comment to T4926: When emails are sent to new people (or lists) they should contain the change description and patch.

For the record, yes, FreeBSD is quite similar and what @chandlerc wrote is very much applicable to our community as well.

Jan 2 2019, 6:37 PM · llvm, Mail, Differential
epriestley added a comment to T4926: When emails are sent to new people (or lists) they should contain the change description and patch.

I think a lot of this behavior may be fairly unique to the LLVM project, since we've never really received this general class of feedback from other installs (except, perhaps, FreeBSD, which I think is somewhat culturally similar).

Jan 2 2019, 6:23 PM · llvm, Mail, Differential
chandlerc added a comment to T4926: When emails are sent to new people (or lists) they should contain the change description and patch.

For the record, while the rationale can be flippantly described as "I don't like clicking things", I'd like to give it a serious description.

Jan 2 2019, 5:44 PM · llvm, Mail, Differential
epriestley moved T8636: Update "Configuring Inbound Email" to describe application email configuration in greater detail from v3 to Soon? on the Mail board.
Jan 2 2019, 5:23 PM · Documentation, Mail
epriestley renamed T5969: Add timeouts to service calls to external mailers (was: PhabricatorMetaMTAWorker may hang indefinitely if "sendmail" hangs indefinitely) from PhabricatorMetaMTAWorker may hang indefinitely if "sendmail" hangs indefinitely to Add timeouts to service calls to external mailers (was: PhabricatorMetaMTAWorker may hang indefinitely if "sendmail" hangs indefinitely).
Jan 2 2019, 5:14 PM · Mail
epriestley moved T5969: Add timeouts to service calls to external mailers (was: PhabricatorMetaMTAWorker may hang indefinitely if "sendmail" hangs indefinitely) from v3 to Soon? on the Mail board.
Jan 2 2019, 5:14 PM · Mail
epriestley added a comment to T13082: Sometimes, mail about attaching revisions to tasks doesn't render the revision title.

This is the most upsetting bug in the software.

Jan 2 2019, 5:13 PM · Transactions, Mail
epriestley moved T13082: Sometimes, mail about attaching revisions to tasks doesn't render the revision title from Backlog to Soon? on the Mail board.
Jan 2 2019, 5:12 PM · Transactions, Mail
epriestley closed T9012: Nag users who reply-all as Wontfix.

I'm generally very hesitant to "nag" users, and this conflicts with T7477 (which wants email to a group of users, including an application email address, to work). Of the two, I think T7477 is significantly more valuable. And, some day, many years from now, I'd theoretically like to make metamta.one-mail-per-recipient the only supported mode, which fixes this anyway.

Jan 2 2019, 5:11 PM · llvm, Mail, Restricted Project
epriestley closed T1109: Allow inbound e-mail without any agent configuration via IMAP as Wontfix.

This is a big chunk of work with no current customer interest. It could be implemented as a third-party extension today; otherwise I think it's unlikely to move forward without customer interest.

Jan 2 2019, 5:08 PM · Mail
epriestley closed T2806: Document fetchmail/procmail as inbound mail options as Wontfix.

These are so niche that I think we probably don't want to formally support them in the upstream.

Jan 2 2019, 5:07 PM · Mail, Documentation
epriestley moved T6162: Add an "Email and unread notification" option in user's Email Preferences from Backlog to Future on the Mail board.
Jan 2 2019, 5:05 PM · Notifications, llvm, Restricted Project, Mail
epriestley closed T4926: When emails are sent to new people (or lists) they should contain the change description and patch as Wontfix.

I don't plan to change this without a better understanding of what problem we're solving. New users can click the link in the email to review the discussion, summary, and patch.

Jan 2 2019, 5:00 PM · llvm, Mail, Differential
epriestley removed a project from T13233: Consider supporting GPG/PGP: Must Encrypt Mail.

(Must Encrypt Mail is a magic meta-tag for testing the feature, not an organizational tag.)

Jan 2 2019, 4:20 PM · Diffusion, Mail
epriestley updated the task description for T13233: Consider supporting GPG/PGP.
Jan 2 2019, 4:16 PM · Diffusion, Mail
epriestley added a comment to T13233: Consider supporting GPG/PGP.

See T8962, somewhat. When receiving inbound mail, we could decrypt PGP messages and verify PGP signatures.

Jan 2 2019, 4:14 PM · Diffusion, Mail
epriestley closed T8962: GPG signed emails confuse phabricator as Wontfix.

It sounds like behavior here is basically correct/expected (content is treated as content; attachments are treated as attachments) and the original report was a top-posting vs quoting issue perhaps (see also T5181), just not ideal (preferable would be validating the signature, or at least discarding these attachments).

Jan 2 2019, 4:10 PM · Differential, Mail
epriestley added a comment to T13233: Consider supporting GPG/PGP.

The existence of alternate ... code signing standards would motivate

Jan 2 2019, 4:07 PM · Diffusion, Mail
epriestley triaged T13233: Consider supporting GPG/PGP as Low priority.
Jan 2 2019, 3:55 PM · Diffusion, Mail
epriestley closed T13204: First mail sent by Differential doesn't have a Message-ID header set properly as Wontfix.

It seems like any client which has this behavior must always break when a user is added to an existing thread (via, for example, "Reply All" + add them as a "Cc"). I think we're either missing something significant, or those clients (possibly just KMail?) aren't handling this situation correctly and this is a bug which should be fixed in those clients rather than Phabricator.

Jan 2 2019, 3:39 PM · Mail, Differential
epriestley closed T12240: Best of Both Worlds: one email to everyone with privacy controls and good threading as Wontfix.

I'm not confident there's really a path forward here even for the subset of mailers where we have full control of the SMTP envelope and can generate Message-IDs, which is only a small subset of mailers today. Even if it does work, it makes the "Reply All" problem worse.

Jan 2 2019, 3:33 PM · Restricted Project, Mail, Feature Request
epriestley closed T1837: Implement correct send-as-user behavior. as Wontfix.

Not clear that "Sender" is worth pursuing, even if it does give us more flexibility around configuring the "From" header in some cases.

Jan 2 2019, 3:30 PM · llvm, Restricted Project, Mail
epriestley closed T1735: Improve implementation of "Re:" prefix by dropping it from messages which are unambiguously first-in-thread as Invalid.

This is years old and I have no idea how to reproduce it. (I use Mail with "Organize by Conversation" as my primary mail client, and have for about a decade.)

Jan 2 2019, 3:27 PM · llvm, Restricted Project, Mail
epriestley moved T6670: Macro images don't show up on html emails if your instance requires logins from Backlog to Soon? on the Mail board.
Jan 2 2019, 3:22 PM · Macros, Mail
epriestley moved T9896: make images from comments visible in email from Backlog to Soon? on the Mail board.
Jan 2 2019, 3:21 PM · Restricted Project, Remarkup, Mail
epriestley moved T12910: Allow SES to be used for inbound mail from Backlog to Future on the Mail board.
Jan 2 2019, 3:20 PM · Mail
epriestley moved T12921: Link to referenced object in transaction emails from Backlog to Soon? on the Mail board.
Jan 2 2019, 3:19 PM · Mail
epriestley moved T7477: Handle inbound email with phabricator address in the CC from Future to Soon? on the Mail board.
Jan 2 2019, 3:19 PM · llvm, Restricted Project, Restricted Project, Mail
epriestley moved T13066: Refine inbound mail error behaviors, distinguishing between actual mail sender and acting-as sender from Future to Soon? on the Mail board.
Jan 2 2019, 3:19 PM · Mail
epriestley renamed T13066: Refine inbound mail error behaviors, distinguishing between actual mail sender and acting-as sender from Refine inbound mail error behaviors to Refine inbound mail error behaviors, distinguishing between actual mail sender and acting-as sender.
Jan 2 2019, 3:15 PM · Mail
epriestley moved T13066: Refine inbound mail error behaviors, distinguishing between actual mail sender and acting-as sender from Far Future to Future on the Mail board.
Jan 2 2019, 3:15 PM · Mail
epriestley added a comment to T13066: Refine inbound mail error behaviors, distinguishing between actual mail sender and acting-as sender.

This logic should probably look more like this [differentiating between explicit senders and receive-as-users].

Jan 2 2019, 3:15 PM · Mail

Dec 20 2018

epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D19922: Remove the onboard "mailKey" from Pholio Mocks.
Dec 20 2018, 7:56 PM · Mail

Nov 9 2018

JoyceBabu added a comment to T11314: Observe Mode + GitHub has weird mail interactions due to pull request refs.

I would like to add another use case where this would be beneficial.

Nov 9 2018, 7:23 AM · Diffusion (v3), Restricted Project, Mail, Haskell.org, Bug Report

Oct 17 2018

epriestley added a comment to T13115: Handle mail bounces inside Phabricator.

To pull the list down (this takes a while since we can only grab 100 results at a time and the API takes a while to respond):

Oct 17 2018, 8:47 PM · Mail
epriestley added a comment to T13115: Handle mail bounces inside Phabricator.

See PHI926, where a GSuite-backed install has seen multiple "550 No Such User" bounces to the same address even though the address exists. It's currently unclear if this is an issue in Mailgun, GSuite, the series of tubes between the two, or elsewhere.

Oct 17 2018, 8:24 PM · Mail

Sep 22 2018

bcooksley added a comment to T13204: First mail sent by Differential doesn't have a Message-ID header set properly.

In my case (using GMail) it appears to thread properly as well however apparently some clients (KMail) rely on these headers more heavily than others.

Sep 22 2018, 12:07 AM · Mail, Differential

Sep 21 2018

epriestley added a comment to T13204: First mail sent by Differential doesn't have a Message-ID header set properly.

In order for threading to work in email clients, the Message-ID header needs to be set on the initial email, with following emails having In-Reply-To / References set to that same Message-ID.

Sep 21 2018, 11:25 PM · Mail, Differential
epriestley added a comment to T13204: First mail sent by Differential doesn't have a Message-ID header set properly.

I can't reproduce this: mail threads properly in my client.

Sep 21 2018, 11:23 PM · Mail, Differential
bcooksley created T13204: First mail sent by Differential doesn't have a Message-ID header set properly.
Sep 21 2018, 10:56 PM · Mail, Differential

Sep 14 2018

epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D19670: Remove "mailKey" from "PhabricatorRepositoryCommit".
Sep 14 2018, 1:27 PM · Mail

Sep 10 2018

GoogleLegacy added a comment to T7804: Implement an "Acting User" Herald condition.
Sep 10 2018, 9:06 AM · Restricted Project, Herald, Mail

Aug 29 2018

epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D19620: Remove on-object mailkeys from Phriction.
Aug 29 2018, 3:29 PM · Mail

Jul 20 2018

amckinley added a revision to T13065: Move storage for `mailKey` to the Mail application: Restricted Differential Revision.
Jul 20 2018, 7:00 PM · Mail

Jun 15 2018

Krinkle added a comment to T13069: Make mail stamps routable on the server, include transaction information, and replace mail tags.

@epriestley T7468: No way to disable web notifications for token awards was marked as duplicate of T10448, which in turn was resolved with a pointer to this task.

Jun 15 2018, 1:22 AM · Mail

Jun 12 2018

epriestley closed T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances as Resolved.

D19485 fixed one small remaining bug; I deployed that to admin.

Jun 12 2018, 8:25 PM · Mail, Phacility, Abuse
epriestley added a revision to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances: Restricted Differential Revision.
Jun 12 2018, 2:12 PM · Mail, Phacility, Abuse

Jun 8 2018

epriestley lowered the priority of T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances from Normal to Low.
Jun 8 2018, 1:54 PM · Mail, Phacility, Abuse
epriestley added a comment to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances.
  • I cherry-picked to stable and deployed to admin.
  • I launched a test instance, invited 32 users, and saw only 20 invites actually go out.
  • After accepting two invites, I saw more invites go out.
  • I cancelled some invites, for good measure.
Jun 8 2018, 1:51 PM · Mail, Phacility, Abuse
epriestley added a revision to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances: Restricted Differential Revision.
Jun 8 2018, 1:34 PM · Mail, Phacility, Abuse

Jun 7 2018

epriestley added a revision to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances: Restricted Differential Revision.
Jun 7 2018, 3:30 PM · Mail, Phacility, Abuse
epriestley added a revision to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances: Restricted Differential Revision.
Jun 7 2018, 1:49 PM · Mail, Phacility, Abuse

Jun 6 2018

epriestley added a revision to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances: Restricted Differential Revision.
Jun 6 2018, 2:04 PM · Mail, Phacility, Abuse
epriestley added a revision to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances: Restricted Differential Revision.
Jun 6 2018, 12:36 PM · Mail, Phacility, Abuse
epriestley added a revision to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances: Restricted Differential Revision.
Jun 6 2018, 12:27 PM · Mail, Phacility, Abuse
epriestley added a revision to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances: Restricted Differential Revision.
Jun 6 2018, 12:23 PM · Mail, Phacility, Abuse
epriestley added a revision to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances: Restricted Differential Revision.
Jun 6 2018, 12:19 PM · Mail, Phacility, Abuse

Jun 5 2018

epriestley added a comment to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances.

I think my plan here is basically:

Jun 5 2018, 9:51 PM · Mail, Phacility, Abuse

Jun 4 2018

epriestley added a comment to T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances.

The "Pending Invites" counter didn't seem to work correctly for this instance.

Jun 4 2018, 10:45 PM · Mail, Phacility, Abuse
epriestley triaged T13150: Limit the allowed number of open invites for Phacility instances, particularly test instances as Normal priority.
Jun 4 2018, 10:31 PM · Mail, Phacility, Abuse

May 4 2018

epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D19427: Remove explicit "mailKey" from Owners packages.
May 4 2018, 8:44 PM · Mail
epriestley updated the task description for T13115: Handle mail bounces inside Phabricator.
May 4 2018, 6:42 PM · Mail

Apr 23 2018

epriestley updated the task description for T13065: Move storage for `mailKey` to the Mail application.
Apr 23 2018, 9:23 PM · Mail
epriestley added a comment to T13065: Move storage for `mailKey` to the Mail application.

Ah, yeah. No idea why I typed "blind" instead.

Apr 23 2018, 9:23 PM · Mail
amckinley added a comment to T13065: Move storage for `mailKey` to the Mail application.

There should be a very small number of readers (2-3?) of this field, so it should be possible to blind them safely like this:

Apr 23 2018, 9:16 PM · Mail
epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D19399: Manage object mailKeys automatically in Mail instead of storing them on objects.
Apr 23 2018, 9:01 PM · Mail

Apr 2 2018

epriestley triaged T13115: Handle mail bounces inside Phabricator as Low priority.
Apr 2 2018, 10:53 PM · Mail

Mar 14 2018

epriestley added a revision to T13069: Make mail stamps routable on the server, include transaction information, and replace mail tags: D19227: Improve documentation around using Mail stamps with Gmail.
Mar 14 2018, 8:07 PM · Mail

Feb 25 2018

epriestley added a project to T13053: Plans: Mail Tags and Failover: Plans.
Feb 25 2018, 3:28 PM · Plans, Must Encrypt Mail, Mail

Feb 22 2018

epriestley closed T13087: Outbound mail should get X-Phabricator-Mail-ID and X-Phabricator-Unique headers as Resolved by committing rPffcfc0465226: Add some delivery diagnostic headers to outbound mail.
Feb 22 2018, 8:47 PM · Mail
epriestley added a revision to T13087: Outbound mail should get X-Phabricator-Mail-ID and X-Phabricator-Unique headers: D19124: Add some delivery diagnostic headers to outbound mail.
Feb 22 2018, 6:35 PM · Mail

Feb 20 2018

epriestley triaged T13087: Outbound mail should get X-Phabricator-Mail-ID and X-Phabricator-Unique headers as Low priority.
Feb 20 2018, 9:50 PM · Mail

Feb 15 2018

epriestley added a comment to T13069: Make mail stamps routable on the server, include transaction information, and replace mail tags.

No. Mail stamps are not intended to answer "Why am I receiving this mail?" in the general case (which lives in T9412 now, I think).

Feb 15 2018, 4:13 PM · Mail
nemobis added a comment to T13069: Make mail stamps routable on the server, include transaction information, and replace mail tags.

Will stamps be applied for all email notifications and reasons for the recipient to receive them, so as to fix T6297: Maniphest email notification doesn't say why I'm receiving it?

Feb 15 2018, 3:38 PM · Mail
epriestley triaged T13082: Sometimes, mail about attaching revisions to tasks doesn't render the revision title as Low priority.
Feb 15 2018, 2:02 PM · Transactions, Mail

Feb 10 2018

epriestley closed T12677: Support multiple mail delivery services for automatic failover as Resolved.

This hasn't blown up in 24 hours and is about to promote, so anything else can be handled in followups.

Feb 10 2018, 12:40 AM · Phacility, Mail

Feb 9 2018

epriestley added a comment to T13053: Plans: Mail Tags and Failover.

(Testing "secure" flag in webhooks. 🐑)

Feb 9 2018, 11:09 PM · Plans, Must Encrypt Mail, Mail
epriestley added a revision to T12677: Support multiple mail delivery services for automatic failover: D19052: Document the STMP port option.
Feb 9 2018, 10:42 PM · Phacility, Mail
epriestley added a comment to T12677: Support multiple mail delivery services for automatic failover.

Seems OK:

Feb 9 2018, 2:17 AM · Phacility, Mail