Made sendmail command configurable

For BookStackApp#4001 Added simple test to cover config option.
CaffeineSheep · Feb 17, 2023 · af31a6f · af31a6f
1 parent 08b3950
commit af31a6f
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/.env.example.complete b/.env.example.complete
@@ -80,6 +80,9 @@ MAIL_USERNAME=null
 MAIL_PASSWORD=null
 MAIL_ENCRYPTION=null
 
+# Command to use when email is sent via sendmail
+MAIL_SENDMAIL_COMMAND="/usr/sbin/sendmail -bs"
+
 # Cache & Session driver to use
 # Can be 'file', 'database', 'memcached' or 'redis'
 CACHE_DRIVER=file

diff --git a/app/Config/mail.php b/app/Config/mail.php
@@ -38,7 +38,7 @@
 
         'sendmail' => [
             'transport' => 'sendmail',
-            'path' => '/usr/sbin/sendmail -bs',
+            'path' => env('MAIL_SENDMAIL_COMMAND', '/usr/sbin/sendmail -bs'),
         ],
 
         'log' => [

diff --git a/tests/Unit/ConfigTest.php b/tests/Unit/ConfigTest.php
@@ -96,6 +96,11 @@ public function test_snappy_paper_size_options_are_limited()
         $this->checkEnvConfigResult('EXPORT_PAGE_SIZE', 'a4', 'snappy.pdf.options.page-size', 'A4');
     }
 
+    public function test_sendmail_command_is_configurage()
+    {
+        $this->checkEnvConfigResult('MAIL_SENDMAIL_COMMAND', '/var/sendmail -o', 'mail.mailers.sendmail.path', '/var/sendmail -o');
+    }
+
     /**
      * Set an environment variable of the given name and value
      * then check the given config key to see if it matches the given result.