Merge branch 'imprv/can-update-SES-setting-in-the-mail-setting-form' into imprv/show-mailer-status

src/client/js/services/AdminAppContainer.js

@@ -287,7 +287,7 @@ export default class AdminAppContainer extends Container {
    * @memberOf AdminAppContainer
    */
   async sendTestEmail() {
-    await this.appContainer.apiv3.get('/app-settings/smtp-test');
+    return this.appContainer.apiv3.post('/app-settings/smtp-test');
   }
 
   /**

src/server/routes/apiv3/app-settings.js

@@ -486,14 +486,14 @@ module.exports = (crowi) => {
    *    /app-settings/smtp-test:
    *      post:
    *        tags: [AppSettings]
-   *        operationId: getSmtpTest
+   *        operationId: postSmtpTest
    *        summary: /app-settings/smtp-setting
    *        description: Send test mail for smtp
    *        responses:
    *          200:
    *            description: Succeeded to send test mail for smtp
    */
-  router.get('/smtp-test', loginRequiredStrictly, adminRequired, async(req, res) => {
+  router.post('/smtp-test', loginRequiredStrictly, adminRequired, async(req, res) => {
     try {
       const mailConfig = {
         to: req.user.email,