Page MenuHomePhabricator

MailProject
ActivePublic

Watchers

  • This project does not have any watchers.
  • View All

Details

Description

A piece of software of standard value and little note.

Recent Activity

Sep 28 2021

epriestley added a comment to T12404: Implement a first-party SMTP client.

each is usually easy to replace and I'm happy to accept a change to replace it if someone wants to reproduce/test it. I believe this (totally ridiculous) construction:

Sep 28 2021, 2:40 PM · Infrastructure, Mail
avivey added a comment to T12404: Implement a first-party SMTP client.

From https://discourse.phabricator-community.org/t/sending-emails-causes-an-exception/4966, it appears that both class.smtp.php and class.phpmailer-lite.php have calls to each which is removed in PHP 8.

Sep 28 2021, 7:55 AM · Infrastructure, Mail

Mar 16 2021

epriestley added a comment to T13065: Move storage for `mailKey` to the Mail application.

After recent changes, only DifferentialRevision and ManiphestTask still have onboard mail keys.

Mar 16 2021, 10:27 PM · Mail
epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D21634: Modernize "mailKey" on Fund initiatives.
Mar 16 2021, 10:22 PM · Mail
epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D21633: Modernize "mailKey" for Calendar Event.
Mar 16 2021, 10:19 PM · Mail
epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D21632: Modernize "mailKey" for Almanac Networks.
Mar 16 2021, 10:11 PM · Mail
epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D21631: Modernize "mailKey" on Almanac Namespaces.
Mar 16 2021, 10:08 PM · Mail
epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D21630: Modernize "mailKey" for Almanac Bindings.
Mar 16 2021, 10:05 PM · Mail
epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D21629: Modernize "mailKey" on AlamnacService.
Mar 16 2021, 10:01 PM · Mail
epriestley removed a revision from T13065: Move storage for `mailKey` to the Mail application: D21626: Make minor Almanac device modernization updates.
Mar 16 2021, 9:23 PM · Mail
epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D21626: Make minor Almanac device modernization updates.
Mar 16 2021, 6:18 PM · Mail
epriestley added a revision to T13065: Move storage for `mailKey` to the Mail application: D21625: Migrate Almanac Device "mailKey" to modern storage.
Mar 16 2021, 6:11 PM · Mail

Feb 25 2021

epriestley added a comment to T13068: Refinements for "Mute Notifications".

When users are @mentioned on an object, render their name in a disabled style if they've been muted.

Feb 25 2021, 7:26 PM · Notifications, Mail
epriestley closed T13570: Amazon is retiring SMTP V3 Signatures as Resolved.

This presumably worked. T12404 survives it, but that task captures everything remaining here.

Feb 25 2021, 7:19 PM · Mail
epriestley removed a project from T12921: Link to referenced object in transaction emails: Feature Request.
Feb 25 2021, 7:12 PM · Mail

Jan 14 2021

aleb added a comment to T12404: Implement a first-party SMTP client.

Recently smtp-relay.gmail.com stopped accepting email from our Phabricator instance because it turns out Phabricator was sending HELO localhost.localdomain instead of HELO smtp-relay.gmail.com when doing the SMTP connection.

Jan 14 2021, 6:45 PM · Infrastructure, Mail

Sep 15 2020

epriestley closed T13235: Replace "SimpleEmailService" with a subclass of "PhutilAWSFuture" that surfaces errors more clearly, a subtask of T13570: Amazon is retiring SMTP V3 Signatures, as Resolved.
Sep 15 2020, 8:03 PM · Mail
epriestley closed T13235: Replace "SimpleEmailService" with a subclass of "PhutilAWSFuture" that surfaces errors more clearly as Resolved by committing rP3dfa89dd5d27: Update SES API to use AWSv4 signatures.
Sep 15 2020, 8:03 PM · Mail
epriestley added a comment to T13235: Replace "SimpleEmailService" with a subclass of "PhutilAWSFuture" that surfaces errors more clearly.

I've marked D21461 as resolving this, since the new errors surface the particulars:

Sep 15 2020, 4:19 PM · Mail
epriestley added a revision to T13235: Replace "SimpleEmailService" with a subclass of "PhutilAWSFuture" that surfaces errors more clearly: D21461: Update SES API to use AWSv4 signatures.
Sep 15 2020, 4:15 PM · Mail
epriestley added a revision to T13570: Amazon is retiring SMTP V3 Signatures: D21461: Update SES API to use AWSv4 signatures.
Sep 15 2020, 4:15 PM · Mail
epriestley added a subtask for T13570: Amazon is retiring SMTP V3 Signatures: T13235: Replace "SimpleEmailService" with a subclass of "PhutilAWSFuture" that surfaces errors more clearly.
Sep 15 2020, 4:08 PM · Mail
epriestley added a parent task for T13235: Replace "SimpleEmailService" with a subclass of "PhutilAWSFuture" that surfaces errors more clearly: T13570: Amazon is retiring SMTP V3 Signatures.
Sep 15 2020, 4:08 PM · Mail

Aug 13 2020

epriestley claimed T13570: Amazon is retiring SMTP V3 Signatures.

I think every other AWS interaction already uses v4, since D14978 (January 2016).

Aug 13 2020, 4:00 PM · Mail
aeiser updated the task description for T13570: Amazon is retiring SMTP V3 Signatures.
Aug 13 2020, 1:55 PM · Mail
aeiser created T13570: Amazon is retiring SMTP V3 Signatures.
Aug 13 2020, 1:54 PM · Mail

Oct 26 2019

epriestley added a comment to T13233: Consider supporting GPG/PGP.

I am running through GPG for the first time:

Oct 26 2019, 6:44 PM · Diffusion, Mail

Sep 8 2019

epriestley updated the task description for T12910: Allow SES to be used for inbound mail.
Sep 8 2019, 2:58 PM · Mail

Aug 12 2019

epriestley triaged T13375: Treat "'to'" address failures as permanent in Mailgun adapter as Low priority.
Aug 12 2019, 3:14 PM · Mail

Jun 29 2019

epriestley added a comment to T13233: Consider supporting GPG/PGP.

This is sort of generically concerning to me:

Jun 29 2019, 8:47 PM · Diffusion, Mail

Jun 19 2019

epriestley closed T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility) as Resolved by committing rPcaaa1394ef15: Don't count "Cc: x@y.com" as a legitimate recipient if the user who has "x@y..
Jun 19 2019, 7:51 PM · Mail, Phacility
epriestley added a revision to T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility): D20593: Don't count "Cc: x@y.com" as a legitimate recipient if the user who has "x@y.com" attached to their account has not verified the address.
Jun 19 2019, 6:15 PM · Mail, Phacility

Jun 18 2019

epriestley renamed T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility) from Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility to Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility).
Jun 18 2019, 5:22 PM · Mail, Phacility
epriestley added a comment to T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility).

Aha! This user very cleverly added noreply@admin.phacility.com to their user account before they were disabled.

Jun 18 2019, 5:22 PM · Mail, Phacility
epriestley created T13317: Unverified/nonsense account emails can trigger inbound CCs (was: Inbound support pact email sometimes CCs disabled user `@maxhodges` in Phacility).
Jun 18 2019, 5:15 PM · Mail, Phacility

Apr 15 2019

epriestley moved T13233: Consider supporting GPG/PGP from Backlog to Far Future on the Diffusion board.
Apr 15 2019, 3:59 PM · Diffusion, Mail

Apr 14 2019

epriestley closed T11314: Observe Mode + GitHub has weird mail interactions due to pull request refs as Resolved.

See D20419 and T13277.

Apr 14 2019, 6:46 PM · Diffusion (v3), Restricted Project, Mail, Haskell.org, Bug Report
epriestley added a revision to T11314: Observe Mode + GitHub has weird mail interactions due to pull request refs: D20419: Do not publish/notify about commits which are not reachable from any "Autoclose" ref.
Apr 14 2019, 6:39 PM · Diffusion (v3), Restricted Project, Mail, Haskell.org, Bug Report

Mar 16 2019

epriestley closed T13265: Allow the SMTP mailer to be configured as a "Does Not Support Message-ID Header" mailer as Resolved by committing rPb469a5134ddd: Allow "SMTP" and "Sendmail" mailers to have "Message-ID" behavior configured in….
Mar 16 2019, 7:36 PM · Mail

Mar 14 2019

epriestley added a comment to T13265: Allow the SMTP mailer to be configured as a "Does Not Support Message-ID Header" mailer.

I also vaguely think some mailer API historically rejected mail with a "Message-ID" header with an actual error ("You can't set this header, we're going to set it for you.") rather than just ignoring/replacing it (maybe SendGrid?), but I might be misremembering or that behavior may have changed. We could still always set the header and drop it in the Adapter nowadays, this whole thing is just a nest of hornets and kicking it has pretty limited upside.

Mar 14 2019, 3:10 PM · Mail
brhodes added a comment to T13265: Allow the SMTP mailer to be configured as a "Does Not Support Message-ID Header" mailer.

I was imagining keeping the Message-ID, while having the first email also include the same value for In-Reply-To — I completely agree that removing the Message-ID would not work out. If the first email always included both fields, thus referencing itself as a reply to itself, I suppose there could be MUAs that would go into infinite loops. So, maybe too risky.

Mar 14 2019, 3:07 PM · Mail
epriestley added a comment to T13265: Allow the SMTP mailer to be configured as a "Does Not Support Message-ID Header" mailer.

I can imagine that cases like this may exist:

Mar 14 2019, 2:58 PM · Mail
brhodes added a comment to T13265: Allow the SMTP mailer to be configured as a "Does Not Support Message-ID Header" mailer.

The only effect I can imagine of always setting the In-Reply-To would be to encourage threading where it's not currently happening, so removing the behavior and deleting all these little methods would be very attractive to me if it were my project. :)

Mar 14 2019, 2:46 PM · Mail
epriestley added a comment to T13265: Allow the SMTP mailer to be configured as a "Does Not Support Message-ID Header" mailer.

There's probably some argument here for removing this behavior and assuming we can never generate a "Message-ID", but I'm hesitant to make changes like that since client threading is largely unknowable black magic which we can not test, and the current approach seems to generally produce acceptable results without an overwhelming amount of collateral complexity.

Mar 14 2019, 2:20 PM · Mail
epriestley added a revision to T13265: Allow the SMTP mailer to be configured as a "Does Not Support Message-ID Header" mailer: D20285: Allow "SMTP" and "Sendmail" mailers to have "Message-ID" behavior configured in "cluster.mailers".
Mar 14 2019, 1:53 PM · Mail
epriestley renamed T13265: Allow the SMTP mailer to be configured as a "Does Not Support Message-ID Header" mailer from Restore In-Reply-To field to first email about a Differential revision to Allow the SMTP mailer to be configured as a "Does Not Support Message-ID Header" mailer.
Mar 14 2019, 1:03 PM · Mail

Feb 25 2019

epriestley added a comment to T13068: Refinements for "Mute Notifications".

A very mild point in favor of this feature is that we haven't seen any negative feedback or confusion, although I think you might be the first user to say they actually like/use it.

Feb 25 2019, 2:31 PM · Notifications, Mail
20after4 added a comment to T13068: Refinements for "Mute Notifications".

FWIW I appreciate and use the mute feature (though only occasionally)

Feb 25 2019, 2:16 PM · Notifications, Mail

Feb 15 2019

epriestley removed a project from T5969: Add timeouts to service calls to external mailers (was: PhabricatorMetaMTAWorker may hang indefinitely if "sendmail" hangs indefinitely): Daemons.
Feb 15 2019, 2:01 AM · Mail

Jan 29 2019

epriestley added a revision to T12921: Link to referenced object in transaction emails: D20057: Allow modular transactions to override transaction title and body text in mail.
Jan 29 2019, 8:09 PM · Mail