Page MenuHomePhabricator
Create Task
Maniphest T9427

Error Processing Mail (Multiple Receivers) when replying to audit mail
Closed, ResolvedPublic
Actions

Description

Your email to Phabricator was not processed, because an error occurred while
trying to handle it:

Phabricator is not able to process this mail because more than one application
is willing to accept it, creating ambiguity. Mail needs to be accepted by
exactly one receiving application.

Accepting receivers: PhabricatorAuditMailReceiver,
PhabricatorCountdownMailReceiver.

-- Original Message Body -----------------------------------------------------

So meta, so test.

-- Original Message Headers --------------------------------------------------

to: C16521+32548+beb002cf14f0241d@phabricator.com
from: Chris Burroughs <chris.burroughs@gmail.com>
subject: Re: [Diffusion] [Commented On] rGITTEST97826afb7189: some testing

Revisions and Commits

rP Phabricator
D14163rP36d9908e6cd1 Move commits to the "COMMIT" mail prefix

Related Objects

Event Timeline

cburroughs created this task.Sep 16 2015, 12:49 PM
cburroughs updated the task description. (Show Details)
cburroughs added a subscriber: cburroughs.
epriestley added a subscriber: epriestley.Sep 16 2015, 1:08 PM

Countdown and Audit both use the C prefix after D13713, so C123 matches both "Countdown 123" and "Commit (with internal ID) 123".

I think the available fixes are:

  1. Switch Countdown to some other prefix, like COUNT (but this is inconsistent, since Countdown owns the monogram).
  2. Switch Diffusion to some other prefix, like COMMIT (but this will break replies to old mail, although they're already broken today).

I tend to favor (2) since (1) sticks us with legacy junk forever and I think the cost is not large.

See also T7477, although the leading solution there would result in your mail leaving a comment on both the countdown and the commit. This is better as a behavior, perhaps, but also clearly nonsense.

epriestley mentioned this in T7477: Handle inbound email with phabricator address in the CC.Sep 16 2015, 1:09 PM
chad added projects: Audit, Mail.Sep 16 2015, 1:47 PM
joshuaspence added a subscriber: joshuaspence.Sep 16 2015, 8:37 PM
anda added a subscriber: anda.Sep 17 2015, 12:36 PM

(2) would be good. Just as you note, replying to old commits is already broken anyway...

epriestley added a revision: D14163: Move commits to the "COMMIT" mail prefix.Sep 25 2015, 3:01 PM
epriestley claimed this task.Sep 25 2015, 3:01 PM
epriestley triaged this task as Normal priority.
epriestley closed this task as Resolved by committing rP36d9908e6cd1: Move commits to the "COMMIT" mail prefix.Sep 25 2015, 5:43 PM
epriestley added a commit: rP36d9908e6cd1: Move commits to the "COMMIT" mail prefix.
Phabricator · Built by Phacility