This project does not seem to handle request data directly as such no vulnerable execution paths were found.
include
, or for example
via PHP's auto-loading mechanism.
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
2 | |||
3 | namespace Kinglozzer\SilverStripeMailgunner; |
||
4 | |||
5 | use Convert; |
||
6 | use Debug; |
||
7 | use Exception; |
||
8 | use Mailer as SilverstripeMailer; |
||
9 | use Mailgun\HttpClientConfigurator; |
||
10 | use Mailgun\Mailgun; |
||
11 | use Mailgun\Messages\BatchMessage; |
||
12 | use Mailgun\Messages\MessageBuilder; |
||
13 | use SS_Log; |
||
14 | use SapphireTest; |
||
15 | |||
16 | class Mailer extends SilverstripeMailer |
||
17 | { |
||
18 | /** |
||
19 | * @var string |
||
20 | * @config |
||
21 | */ |
||
22 | private static $api_domain = ''; |
||
23 | |||
24 | /** |
||
25 | * @var string |
||
26 | * @config |
||
27 | */ |
||
28 | private static $api_endpoint = ''; |
||
29 | |||
30 | /** |
||
31 | * @var string |
||
32 | * @config |
||
33 | */ |
||
34 | private static $api_key = ''; |
||
35 | |||
36 | /** |
||
37 | * @var boolean |
||
38 | * @config |
||
39 | */ |
||
40 | private static $debug = false; |
||
41 | |||
42 | /** |
||
43 | * An array of temporary file handles opened to store attachments |
||
44 | * @var array |
||
45 | */ |
||
46 | protected $tempFileHandles = []; |
||
47 | |||
48 | /** |
||
49 | * @var Mailgun |
||
50 | */ |
||
51 | protected $mailgunClient; |
||
52 | |||
53 | /** |
||
54 | * {@inheritdoc} |
||
55 | */ |
||
56 | 12 | public function __construct() |
|
57 | { |
||
58 | 12 | $config = $this->config(); |
|
59 | 12 | $configurator = new HttpClientConfigurator(); |
|
60 | 12 | $configurator->setApiKey($config->api_key); |
|
61 | 12 | $configurator->setDebug($config->debug); |
|
62 | |||
63 | 12 | if ($config->api_endpoint) { |
|
64 | $configurator->setEndpoint($config->api_endpoint); |
||
65 | } |
||
66 | |||
67 | 12 | $this->setMailgunClient(Mailgun::configure($configurator)); |
|
68 | 12 | } |
|
69 | |||
70 | /** |
||
71 | * @param Mailgun $client |
||
72 | * @return self |
||
73 | */ |
||
74 | 12 | public function setMailgunClient(Mailgun $client) |
|
75 | { |
||
76 | 12 | $this->mailgunClient = $client; |
|
77 | 12 | return $this; |
|
78 | } |
||
79 | |||
80 | /** |
||
81 | * @return Mailgun |
||
82 | */ |
||
83 | 1 | public function getMailgunClient() |
|
84 | { |
||
85 | 1 | return $this->mailgunClient; |
|
86 | } |
||
87 | |||
88 | /** |
||
89 | * {@inheritdoc} |
||
90 | */ |
||
91 | 1 | public function sendPlain($to, $from, $subject, $plainContent, $attachments = [], $headers = []) |
|
92 | { |
||
93 | 1 | return $this->sendMessage($to, $from, $subject, $htmlContent = '', $plainContent, $attachments, $headers); |
|
94 | } |
||
95 | |||
96 | /** |
||
97 | * {@inheritdoc} |
||
98 | */ |
||
99 | 1 | public function sendHTML($to, $from, $subject, $htmlContent, $attachments = [], $headers = [], $plainContent = '') |
|
100 | { |
||
101 | 1 | return $this->sendMessage($to, $from, $subject, $htmlContent, $plainContent, $attachments, $headers); |
|
102 | } |
||
103 | |||
104 | /** |
||
105 | * @param string $to |
||
106 | * @param string $from |
||
107 | * @param string $subject |
||
108 | * @param string $content |
||
109 | * @param string $plainContent |
||
110 | * @param array $attachments |
||
111 | * @param array $headers |
||
112 | */ |
||
113 | 3 | protected function sendMessage($to, $from, $subject, $content, $plainContent, $attachments, $headers) |
|
114 | { |
||
115 | 3 | $domain = $this->config()->api_domain; |
|
116 | 3 | $client = $this->getMailgunClient(); |
|
117 | 3 | $attachments = $this->prepareAttachments($attachments); |
|
118 | |||
119 | 3 | if (isset($headers['X-Mailgunner-Batch-Message'])) { |
|
120 | 1 | $builder = $client->BatchMessage($domain); |
|
121 | 1 | unset($headers['X-Mailgunner-Batch-Message']); |
|
122 | 1 | } else { |
|
123 | 2 | $builder = $client->MessageBuilder(); |
|
124 | } |
||
125 | |||
126 | try { |
||
127 | 3 | $this->buildMessage($builder, $to, $from, $subject, $content, $plainContent, $attachments, $headers); |
|
128 | |||
129 | 3 | if ($builder instanceof BatchMessage) { |
|
130 | 1 | $builder->finalize(); |
|
131 | 1 | } else { |
|
132 | 2 | $client->sendMessage($domain, $builder->getMessage(), $builder->getFiles()); |
|
133 | } |
||
134 | 3 | } catch (Exception $e) { |
|
135 | // Close and remove any temp files created for attachments |
||
136 | 1 | $this->closeTempFileHandles(); |
|
137 | // Throwing the exception would break SilverStripe's Email API expectations, so we log |
||
138 | // errors and show a message (which is hidden in live mode) |
||
139 | 1 | SS_Log::log('Mailgun error: ' . $e->getMessage(), SS_Log::ERR); |
|
140 | 1 | if (!SapphireTest::is_running_test()) { |
|
141 | Debug::message('Mailgun error: ' . $e->getMessage()); |
||
142 | 1 | } |
|
143 | |||
144 | 1 | return false; |
|
145 | } |
||
146 | |||
147 | 2 | $this->closeTempFileHandles(); |
|
148 | |||
149 | // This is a stupid API :( |
||
150 | 2 | return [$to, $subject, $content, $headers, '']; |
|
151 | } |
||
152 | |||
153 | /** |
||
154 | * @param MessageBuilder $builder |
||
155 | * @param string $to |
||
156 | * @param string $from |
||
157 | * @param string $subject |
||
158 | * @param string $content |
||
159 | * @param string $plainContent |
||
160 | * @param array $attachments |
||
161 | * @param array $headers |
||
162 | */ |
||
163 | 2 | protected function buildMessage( |
|
164 | MessageBuilder $builder, |
||
165 | $to, |
||
166 | $from, |
||
167 | $subject, |
||
168 | $content, |
||
169 | $plainContent, |
||
170 | array $attachments, |
||
171 | array $headers |
||
172 | ) { |
||
173 | // Add base info |
||
174 | 2 | $parsedFrom = $this->parseAddresses($from); |
|
175 | 2 | foreach ($parsedFrom as $email => $name) { |
|
176 | 2 | $builder->setFromAddress($email, ['full_name' => $name]); |
|
177 | 2 | } |
|
178 | |||
179 | 2 | if (empty($plainContent)) { |
|
180 | $plainContent = Convert::xml2raw($content); |
||
181 | } |
||
182 | |||
183 | 2 | $builder->setSubject($subject); |
|
184 | 2 | $builder->setHtmlBody($content); |
|
185 | 2 | $builder->setTextBody($plainContent); |
|
0 ignored issues
–
show
|
|||
186 | |||
187 | // Add attachments |
||
188 | 2 | foreach ($attachments as $attachment) { |
|
189 | 2 | $builder->addAttachment($attachment['filePath'], $attachment['remoteName']); |
|
190 | 2 | } |
|
191 | |||
192 | // Parse Cc & Bcc headers out if they're set |
||
193 | 2 | $ccAddresses = isset($headers['Cc']) ? $headers['Cc'] : ''; |
|
194 | 2 | $bccAddresses = isset($headers['Bcc']) ? $headers['Bcc'] : ''; |
|
195 | |||
196 | // We handle these ourselves, so can remove them from the list of headers |
||
197 | 2 | unset($headers['Cc']); |
|
198 | 2 | unset($headers['Bcc']); |
|
199 | |||
200 | // Add remaining custom headers |
||
201 | 2 | foreach ($headers as $name => $data) { |
|
202 | 2 | $builder->addCustomHeader($name, $data); |
|
203 | 2 | } |
|
204 | |||
205 | // Add recipients. This is done last as the 'BatchMessage' message builder |
||
206 | // will trigger sends for every 1000 addresses |
||
207 | 2 | $to = $this->parseAddresses($to); |
|
208 | 2 | foreach ($to as $email => $name) { |
|
209 | 2 | $builder->addToRecipient($email, ['full_name' => $name]); |
|
210 | 2 | } |
|
211 | |||
212 | 2 | $ccAddresses = $this->parseAddresses($ccAddresses); |
|
213 | 2 | foreach ($ccAddresses as $email => $name) { |
|
214 | 2 | $builder->addCcRecipient($email, ['full_name' => $name]); |
|
215 | 2 | } |
|
216 | |||
217 | 2 | $bccAddresses = $this->parseAddresses($bccAddresses); |
|
218 | 2 | foreach ($bccAddresses as $email => $name) { |
|
219 | 2 | $builder->addBccRecipient($email, ['full_name' => $name]); |
|
220 | 2 | } |
|
221 | 2 | } |
|
222 | |||
223 | /** |
||
224 | * @todo This can't deal with mismatched quotes, or commas in names. |
||
225 | * E.g. "Smith, John" <[email protected]> or "John O'smith" <[email protected]> |
||
226 | * @param string |
||
227 | * @return array |
||
228 | */ |
||
229 | 3 | protected function parseAddresses($addresses) |
|
230 | { |
||
231 | 3 | $parsed = []; |
|
232 | |||
233 | 3 | $expr = '/\s*["\']?([^><,;"\']+)["\']?\s*((?:<[^><,]+>)?)\s*/'; |
|
234 | 3 | if (preg_match_all($expr, $addresses, $matches, PREG_SET_ORDER) > 0) { |
|
235 | 3 | foreach ($matches as $result) { |
|
236 | 3 | if (empty($result[2])) { |
|
237 | // If we couldn't parse out a name |
||
238 | 3 | $parsed[$result[1]] = ''; |
|
239 | 3 | } else { |
|
240 | 3 | $email = trim($result[2], '<>'); |
|
241 | 3 | $parsed[$email] = trim($result[1]); |
|
242 | } |
||
243 | 3 | } |
|
244 | 3 | } |
|
245 | |||
246 | 3 | return $parsed; |
|
247 | } |
||
248 | |||
249 | /** |
||
250 | * Prepare attachments for sending. SilverStripe extracts the content and |
||
251 | * passes that to the mailer, so to save encoding it we just write them all |
||
252 | * to individual files and let Mailgun deal with the rest. |
||
253 | * |
||
254 | * @todo Can we handle this better? |
||
255 | * @param array $attachments |
||
256 | * @return array |
||
257 | */ |
||
258 | 2 | protected function prepareAttachments(array $attachments) |
|
259 | { |
||
260 | 2 | $prepared = []; |
|
261 | |||
262 | 2 | foreach ($attachments as $attachment) { |
|
263 | 2 | $tempFile = $this->writeToTempFile($attachment['contents']); |
|
264 | |||
265 | 2 | $prepared[] = [ |
|
266 | 2 | 'filePath' => $tempFile, |
|
267 | 2 | 'remoteName' => $attachment['filename'] |
|
268 | 2 | ]; |
|
269 | 2 | } |
|
270 | |||
271 | 2 | return $prepared; |
|
272 | } |
||
273 | |||
274 | /** |
||
275 | * @param string $contents |
||
276 | * @return string |
||
277 | */ |
||
278 | 2 | protected function writeToTempFile($contents) |
|
279 | { |
||
280 | 2 | $tempFile = tempnam(sys_get_temp_dir(), 'SS_MG_TMP'); |
|
281 | 2 | $fileHandle = fopen($tempFile, 'r+'); |
|
282 | 2 | fwrite($fileHandle, $contents); |
|
283 | |||
284 | 2 | $this->tempFileHandles[] = [ |
|
285 | 2 | 'handle' => $fileHandle, |
|
286 | 'path' => $tempFile |
||
287 | 2 | ]; |
|
288 | |||
289 | 2 | return $tempFile; |
|
290 | } |
||
291 | |||
292 | /** |
||
293 | * @return void |
||
294 | */ |
||
295 | 1 | protected function closeTempFileHandles() |
|
296 | { |
||
297 | 1 | foreach ($this->tempFileHandles as $key => $data) { |
|
298 | 1 | fclose($data['handle']); |
|
299 | 1 | unlink($data['path']); |
|
300 | 1 | unset($this->tempFileHandles[$key]); |
|
301 | 1 | } |
|
302 | 1 | } |
|
303 | } |
||
304 |
If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check:
If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue.