Change implementation to add validation rules, of which 'required' is shown in the frontend

src/Forum/ForumServiceProvider.php

@@ -168,6 +168,8 @@ class ForumServiceProvider extends AbstractServiceProvider
                     $this->app
                 );
                 $validator->whenSettingsSaving($event);
+
+                $this->app->make(ValidateMailConfiguration::class)->whenSettingsSaving($event);
             }
         );
     }

src/Forum/ValidateMailConfiguration.php

@@ -0,0 +1,91 @@
+<?php
+
+/*
+ * This file is part of Flarum.
+ *
+ * (c) Toby Zerner <toby.zerner@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Flarum\Forum;
+
+use Flarum\Foundation\Application;
+use Flarum\Mail\DriverInterface;
+use Flarum\Mail\NullDriver;
+use Flarum\Settings\Event\Saving;
+use Flarum\Settings\SettingsRepositoryInterface;
+use Flarum\Settings\SettingsServiceProvider;
+use Illuminate\Contracts\Container\Container;
+use Illuminate\Support\Arr;
+use Illuminate\Validation\Validator;
+
+class ValidateMailConfiguration
+{
+    /**
+     * @var SettingsServiceProvider
+     */
+    protected $settings;
+
+    /**
+     * @var Application
+     */
+    protected $container;
+
+    /**
+     * @param Container $container
+     * @param SettingsRepositoryInterface $settings
+     */
+    public function __construct(Container $container, SettingsRepositoryInterface $settings)
+    {
+        $this->container = $container;
+        $this->settings = $settings;
+    }
+
+    public function whenSettingsSaving(Saving $event)
+    {
+        if (! isset($event->settings['mail_driver'])) {
+            return;
+        }
+
+        $driver = $this->getDriver($event->settings);
+
+        $this->getValidator($driver, $event->settings)->validate();
+    }
+
+    public function getWorkingDriver()
+    {
+        $settings = $this->settings->all();
+        $driver = $this->getDriver($settings);
+        $validator = $this->getValidator($driver, $settings);
+
+        return$validator->passes()
+            ? $driver
+            : $this->container->make(NullDriver::class);
+    }
+
+    /**
+     * @param DriverInterface $driver
+     * @param array $settings
+     * @return Validator
+     */
+    protected function getValidator($driver, $settings)
+    {
+        $rules = $driver->availableSettings();
+        $settings = Arr::only($settings, array_keys($rules));
+
+        return $this->container->make('validator')->make($settings, $rules);
+    }
+
+    protected function getDriver($settings)
+    {
+        $drivers = $this->container->make('mail.supported_drivers');
+        $specifiedDriver = Arr::get($settings, 'mail_driver');
+        $driverClass = Arr::get($drivers, $specifiedDriver);
+
+        return $specifiedDriver && $driverClass
+            ? $this->container->make($driverClass)
+            : $this->container->make(NullDriver::class);
+    }
+}

src/Mail/DriverInterface.php

@@ -31,11 +31,6 @@ interface DriverInterface
      */
     public function availableSettings(): array;
 
-    /**
-     * Provide a list of required settings for this driver.
-     */
-    public function requiredFields(): array;
-
     /**
      * Build a mail transport based on Flarum's current settings.
      */

src/Mail/LogDriver.php

@@ -31,11 +31,6 @@ class LogDriver implements DriverInterface
         return [];
     }
 
-    public function requiredFields(): array
-    {
-        return [];
-    }
-
     public function buildTransport(SettingsRepositoryInterface $settings): Swift_Transport
     {
         return new LogTransport($this->logger);

src/Mail/MailServiceProvider.php

@@ -9,6 +9,7 @@
 
 namespace Flarum\Mail;
 
+use Flarum\Forum\ValidateMailConfiguration;
 use Flarum\Foundation\AbstractServiceProvider;
 use Flarum\Settings\SettingsRepositoryInterface;
 use Illuminate\Mail\Mailer;
@@ -30,18 +31,7 @@ class MailServiceProvider extends AbstractServiceProvider
         });
 
         $this->app->singleton('mail.driver', function () {
-            $settings = $this->app->make(SettingsRepositoryInterface::class);
-            $drivers = $this->app->make('mail.supported_drivers');
-
-            $driver = $this->app->make($drivers[$settings->get('mail_driver')]);
-
-            // check that all required fields have been filled
-            $settings = $this->app->make(SettingsRepositoryInterface::class);
-            $valid = $driver && array_reduce($driver->requiredFields(), function ($carry, $field) use ($settings) {
-                return $carry && ! empty($settings->get($field));
-            }, true);
-
-            return $valid ? $driver : $this->app->make(NullDriver::class);
+            return $this->app->make(ValidateMailConfiguration::class)->getWorkingDriver();
         });
 
         $this->app->alias('mail.driver', DriverInterface::class);

src/Mail/MailgunDriver.php

@@ -19,20 +19,11 @@ class MailgunDriver implements DriverInterface
     public function availableSettings(): array
     {
         return [
-            'mail_mailgun_secret' => '', // the secret key
-            'mail_mailgun_domain' => '', // the API base URL
-            'mail_mailgun_region' => [ // region's endpoint
-                'api.mailgun.net' => 'US',
-                'api.eu.mailgun.net' => 'EU',
-            ],
+            'mail_mailgun_secret' => 'required', // the secret key
+            'mail_mailgun_domain' => 'required|regex:/^(?!\-)(?:[a-zA-Z\d\-]{0,62}[a-zA-Z\d]\.){1,126}(?!\d+)[a-zA-Z\d]{1,63}$/', // the API base URL
         ];
     }
 
-    public function requiredFields(): array
-    {
-        return $this->availableSettings();
-    }
-
     public function buildTransport(SettingsRepositoryInterface $settings): Swift_Transport
     {
         return new MailgunTransport(

src/Mail/MandrillDriver.php

@@ -19,15 +19,10 @@ class MandrillDriver implements DriverInterface
     public function availableSettings(): array
     {
         return [
-            'mail_mandrill_secret' => '',
+            'mail_mandrill_secret' => 'required',
         ];
     }
 
-    public function requiredFields(): array
-    {
-        return $this->availableSettings();
-    }
-
     public function buildTransport(SettingsRepositoryInterface $settings): Swift_Transport
     {
         return new MandrillTransport(

src/Mail/NullDriver.php

@@ -20,11 +20,6 @@ class NullDriver implements DriverInterface
         return [];
     }
 
-    public function requiredFields(): array
-    {
-        return [];
-    }
-
     public function buildTransport(SettingsRepositoryInterface $settings): Swift_Transport
     {
         return new Swift_NullTransport();

src/Mail/SendmailDriver.php

@@ -20,11 +20,6 @@ class SendmailDriver implements DriverInterface
         return [];
     }
 
-    public function requiredFields(): array
-    {
-        return [];
-    }
-
     public function buildTransport(SettingsRepositoryInterface $settings): Swift_Transport
     {
         return new Swift_SendmailTransport;

src/Mail/SesDriver.php

@@ -19,17 +19,12 @@ class SesDriver implements DriverInterface
     public function availableSettings(): array
     {
         return [
-            'mail_ses_key' => '',
-            'mail_ses_secret' => '',
-            'mail_ses_region' => '',
+            'mail_ses_key' => 'required',
+            'mail_ses_secret' => 'required',
+            'mail_ses_region' => 'required',
         ];
     }
 
-    public function requiredFields(): array
-    {
-        return $this->availableSettings();
-    }
-
     public function buildTransport(SettingsRepositoryInterface $settings): Swift_Transport
     {
         $config = [

src/Mail/SmtpDriver.php

@@ -18,20 +18,11 @@ class SmtpDriver implements DriverInterface
     public function availableSettings(): array
     {
         return [
-            'mail_host' => '', // a hostname, IPv4 address or IPv6 wrapped in []
-            'mail_port' => '', // a number, defaults to 25
-            'mail_encryption' => '', // "tls" or "ssl"
-            'mail_username' => '',
-            'mail_password' => '',
-        ];
-    }
-
-    public function requiredFields(): array
-    {
-        return [
-            'mail_host',
-            'mail_username',
-            'mail_password',
+            'mail_host' => 'required', // a hostname, IPv4 address or IPv6 wrapped in []
+            'mail_port' => 'nullable|integer', // a number, defaults to 25
+            'mail_encryption' => 'nullable|in:tls,ssl', // "tls" or "ssl"
+            'mail_username' => 'required',
+            'mail_password' => 'required',
         ];
     }