mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #25591 from nextcloud/backport/25582/stable21 

[stable21] Do not send imip email to invalid recipients
This commit is contained in:
Roeland Jago Douma 2021-02-12 08:44:42 +01:00 committed by GitHub
parent a15066b840 3878bcbc11
commit 32a49e228b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 19 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
apps/dav/lib/CalDAV/Schedule/IMipPlugin.php
View File

@ -178,6 +178,11 @@ class IMipPlugin extends SabreIMipPlugin {
// Strip off mailto:
$sender = substr($iTipMessage->sender, 7);
$recipient = substr($iTipMessage->recipient, 7);
if (!$this->mailer->validateMailAddress($recipient)) {
// Nothing to send if the recipient doesn't have a valid email address
$iTipMessage->scheduleStatus = '5.0; EMail delivery failed';
return;
}
$senderName = $iTipMessage->senderName ?: null;
$recipientName = $iTipMessage->recipientName ?: null;

14
apps/dav/tests/unit/CalDAV/Schedule/IMipPluginTest.php
View File

@ -140,6 +140,7 @@ class IMipPluginTest extends TestCase {
->method('getAppValue')
->with('dav', 'invitation_link_recipients', 'yes')
->willReturn('yes');
$this->mailer->method('validateMailAddress')->willReturn(true);
$message = $this->_testMessage();
$this->_expectSend();
@ -153,6 +154,7 @@ class IMipPluginTest extends TestCase {
->method('getAppValue')
->with('dav', 'invitation_link_recipients', 'yes')
->willReturn('yes');
$this->mailer->method('validateMailAddress')->willReturn(true);
$message = $this->_testMessage();
$this->mailer
@ -163,12 +165,21 @@ class IMipPluginTest extends TestCase {
$this->assertEquals('5.0', $message->getScheduleStatus());
}
public function testInvalidEmailDelivery() {
$this->mailer->method('validateMailAddress')->willReturn(false);
$message = $this->_testMessage();
$this->plugin->schedule($message);
$this->assertEquals('5.0', $message->getScheduleStatus());
}
public function testDeliveryWithNoCommonName() {
$this->config
->expects($this->at(1))
->method('getAppValue')
->with('dav', 'invitation_link_recipients', 'yes')
->willReturn('yes');
$this->mailer->method('validateMailAddress')->willReturn(true);
$message = $this->_testMessage();
$message->senderName = null;
@ -193,6 +204,7 @@ class IMipPluginTest extends TestCase {
$this->config
->method('getAppValue')
->willReturn('yes');
$this->mailer->method('validateMailAddress')->willReturn(true);
$message = $this->_testMessage($veventParams);
@ -227,6 +239,7 @@ class IMipPluginTest extends TestCase {
*/
public function testIncludeResponseButtons(string $config_setting, string $recipient, bool $has_buttons) {
$message = $this->_testMessage([],$recipient);
$this->mailer->method('validateMailAddress')->willReturn(true);
$this->_expectSend($recipient, true, $has_buttons);
$this->config
@ -256,6 +269,7 @@ class IMipPluginTest extends TestCase {
$this->config
->method('getAppValue')
->willReturn('yes');
$this->mailer->method('validateMailAddress')->willReturn(true);
$message = $this->_testMessage(['SUMMARY' => '']);
$this->_expectSend('frodo@hobb.it', true, true,'Invitation: Untitled event');