mirror of
https://we.phorge.it/source/phorge.git
synced 2024-12-04 20:52:43 +01:00
72e33c9e5a
Summary: Amazon SES does not allow us to set a Message-ID header, which means that threads are incorrect in Mail.app (and presumably other applications which respect In-Reply-To and References) because the initial email does not have anything which attaches it to the rest of the thread. To fix this, never rely on Message-ID if the mailer doesn't support Message-ID. (In the Amazon SES case, Amazon generates its own Message-ID which we can't know ahead of time). I additionally used all the Lisk isolation from the other tests to make this testable and wrote tests for it. I also moved the idea of a thread ID lower in the stack and out of DifferentialMail, which should not be responsible for implementation details. NOTE: If you push this, it will cause a one-time break of threading for everyone using Outlook since I've changed the seed for generating Thread-Index. I feel like this is okay to avoid introducing more complexity here. Test Plan: Created and then updated a revision, messages delivered over Amazon SES threaded correctly in Mail.app. Verified headers. Unit tests. Reviewed By: rm Reviewers: aran, tuomaspelkonen, jungejason, rm Commenters: aran CC: aran, rm, epriestley Differential Revision: 195
90 lines
2 KiB
PHP
90 lines
2 KiB
PHP
<?php
|
|
|
|
/*
|
|
* Copyright 2011 Facebook, Inc.
|
|
*
|
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
|
* you may not use this file except in compliance with the License.
|
|
* You may obtain a copy of the License at
|
|
*
|
|
* http://www.apache.org/licenses/LICENSE-2.0
|
|
*
|
|
* Unless required by applicable law or agreed to in writing, software
|
|
* distributed under the License is distributed on an "AS IS" BASIS,
|
|
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
* See the License for the specific language governing permissions and
|
|
* limitations under the License.
|
|
*/
|
|
|
|
/**
|
|
* Mail adapter that doesn't actually send any email, for writing unit tests
|
|
* against.
|
|
*/
|
|
class PhabricatorMailImplementationTestAdapter
|
|
extends PhabricatorMailImplementationAdapter {
|
|
|
|
private $guts = array();
|
|
private $config;
|
|
|
|
public function __construct(array $config) {
|
|
$this->config = $config;
|
|
}
|
|
|
|
public function setFrom($email) {
|
|
$this->guts['from'] = $email;
|
|
return $this;
|
|
}
|
|
|
|
public function addReplyTo($email) {
|
|
$this->guts['reply-to'] = $email;
|
|
return $this;
|
|
}
|
|
|
|
public function addTos(array $emails) {
|
|
foreach ($emails as $email) {
|
|
$this->guts['tos'][] = $email;
|
|
}
|
|
return $this;
|
|
}
|
|
|
|
public function addCCs(array $emails) {
|
|
foreach ($emails as $email) {
|
|
$this->guts['ccs'][] = $email;
|
|
}
|
|
return $this;
|
|
}
|
|
|
|
public function addHeader($header_name, $header_value) {
|
|
$this->guts['headers'][] = array($header_name, $header_value);
|
|
return $this;
|
|
}
|
|
|
|
public function setBody($body) {
|
|
$this->guts['body'] = $body;
|
|
return $this;
|
|
}
|
|
|
|
public function setSubject($subject) {
|
|
$this->guts['subject'] = $subject;
|
|
return $this;
|
|
}
|
|
|
|
public function setIsHTML($is_html) {
|
|
$this->guts['is-html'] = $is_html;
|
|
return $this;
|
|
}
|
|
|
|
public function supportsMessageIDHeader() {
|
|
return $this->config['supportsMessageIDHeader'];
|
|
}
|
|
|
|
public function send() {
|
|
$this->guts['did-send'] = true;
|
|
return true;
|
|
}
|
|
|
|
public function getGuts() {
|
|
return $this->guts;
|
|
}
|
|
|
|
}
|