Mail: updated peer verify option name and added test

diff --git a/.env.example.complete b/.env.example.complete
index 8c0accf1de68d2402942a235c1e4363b9c5f77a9..9ce33f58f82b77900eb7ec20bb27e1fdae315f3d 100644 (file)
--- a/.env.example.complete
+++ b/.env.example.complete
@@ -79,7 +79,7 @@ MAIL_PORT=1025
 MAIL_USERNAME=null
 MAIL_PASSWORD=null
 MAIL_ENCRYPTION=null
 MAIL_USERNAME=null
 MAIL_PASSWORD=null
 MAIL_ENCRYPTION=null

-MAIL_VERIFY_PEER=true
+MAIL_VERIFY_SSL=true

 
 # Command to use when email is sent via sendmail
 MAIL_SENDMAIL_COMMAND="/usr/sbin/sendmail -bs"
 
 # Command to use when email is sent via sendmail
 MAIL_SENDMAIL_COMMAND="/usr/sbin/sendmail -bs"

diff --git a/app/Config/mail.php b/app/Config/mail.php
index 6cd5ee28fbb3b9ccbe6e5f1cd64790e2675492a8..87514aa409809c875cbe5523404214c440e71dba 100644 (file)
--- a/app/Config/mail.php
+++ b/app/Config/mail.php
@@ -32,7 +32,7 @@ return [
             'encryption' => env('MAIL_ENCRYPTION', 'tls'),
             'username' => env('MAIL_USERNAME'),
             'password' => env('MAIL_PASSWORD'),
             'encryption' => env('MAIL_ENCRYPTION', 'tls'),
             'username' => env('MAIL_USERNAME'),
             'password' => env('MAIL_PASSWORD'),

-            'verify_peer' => env('MAIL_VERIFY_PEER', true),
+            'verify_peer' => env('MAIL_VERIFY_SSL', true),

             'timeout' => null,
             'local_domain' => env('MAIL_EHLO_DOMAIN'),
         ],
             'timeout' => null,
             'local_domain' => env('MAIL_EHLO_DOMAIN'),
         ],

diff --git a/phpunit.xml b/phpunit.xml
index 8a526a70428258ed3ba4d5148baea83c478a507f..704372c5c64f123a42702d62809653ceda95199c 100644 (file)
--- a/phpunit.xml
+++ b/phpunit.xml
@@ -27,6 +27,7 @@
     <server name="DB_CONNECTION" value="mysql_testing"/>
     <server name="BCRYPT_ROUNDS" value="4"/>
     <server name="MAIL_DRIVER" value="array"/>
     <server name="DB_CONNECTION" value="mysql_testing"/>
     <server name="BCRYPT_ROUNDS" value="4"/>
     <server name="MAIL_DRIVER" value="array"/>

+    <server name="MAIL_VERIFY_SSL" value="true"/>

     <server name="LOG_CHANNEL" value="single"/>
     <server name="AUTH_METHOD" value="standard"/>
     <server name="AUTH_AUTO_INITIATE" value="false"/>
     <server name="LOG_CHANNEL" value="single"/>
     <server name="AUTH_METHOD" value="standard"/>
     <server name="AUTH_AUTO_INITIATE" value="false"/>

diff --git a/tests/Unit/ConfigTest.php b/tests/Unit/ConfigTest.php
index 9966a4fb15114683393cb46e603da2e6431aa6d2..10376751633c5e225a1451cc361ddb41870ae2e5 100644 (file)
--- a/tests/Unit/ConfigTest.php
+++ b/tests/Unit/ConfigTest.php
@@ -3,6 +3,8 @@
 namespace Tests\Unit;
 
 use Illuminate\Support\Facades\Log;
 namespace Tests\Unit;
 
 use Illuminate\Support\Facades\Log;

+use Illuminate\Support\Facades\Mail;
+use Symfony\Component\Mailer\Transport\Smtp\EsmtpTransport;

 use Tests\TestCase;
 
 /**
 use Tests\TestCase;
 
 /**


@@ -96,11 +98,30 @@ class ConfigTest extends TestCase
         $this->checkEnvConfigResult('EXPORT_PAGE_SIZE', 'a4', 'snappy.pdf.options.page-size', 'A4');
     }
 
         $this->checkEnvConfigResult('EXPORT_PAGE_SIZE', 'a4', 'snappy.pdf.options.page-size', 'A4');
     }
 

-    public function test_sendmail_command_is_configurage()
+    public function test_sendmail_command_is_configurable()

     {
         $this->checkEnvConfigResult('MAIL_SENDMAIL_COMMAND', '/var/sendmail -o', 'mail.mailers.sendmail.path', '/var/sendmail -o');
     }
 
     {
         $this->checkEnvConfigResult('MAIL_SENDMAIL_COMMAND', '/var/sendmail -o', 'mail.mailers.sendmail.path', '/var/sendmail -o');
     }
 

+    public function test_mail_disable_ssl_verification_alters_mailer()
+    {
+        $getStreamOptions = function (): array {
+            /** @var EsmtpTransport $transport */
+            $transport = Mail::mailer('smtp')->getSymfonyTransport();
+            return $transport->getStream()->getStreamOptions();
+        };
+
+        $this->assertEmpty($getStreamOptions());
+
+
+        $this->runWithEnv('MAIL_VERIFY_SSL', 'false', function () use ($getStreamOptions) {
+            $options = $getStreamOptions();
+            $this->assertArrayHasKey('ssl', $options);
+            $this->assertFalse($options['ssl']['verify_peer']);
+            $this->assertFalse($options['ssl']['verify_peer_name']);
+        });
+    }
+

     /**
      * Set an environment variable of the given name and value
      * then check the given config key to see if it matches the given result.
     /**
      * Set an environment variable of the given name and value
      * then check the given config key to see if it matches the given result.