Merge pull request #1994 from kounoike/pr-show-email-settings-error

Show SMTP Error message in testing email settings
This commit is contained in:
Naoki Takezoe
2018-05-02 16:50:56 +09:00
committed by GitHub

View File

@@ -15,6 +15,7 @@ import gitbucket.core.util.StringUtil._
import gitbucket.core.util.SyntaxSugars._
import gitbucket.core.util.{AdminAuthenticator, Mailer}
import org.apache.commons.io.IOUtils
import org.apache.commons.mail.EmailException
import org.json4s.jackson.Serialization
import org.scalatra._
import org.scalatra.forms._
@@ -312,7 +313,8 @@ trait SystemSettingsControllerBase extends AccountManagementControllerBase {
"Test mail has been sent to: " + form.testAddress
} catch {
case e: Exception => "[Error] " + e.toString
case e: EmailException => s"[Error] ${e.getCause}"
case e: Exception => "[Error] " + e.toString
}
})