Issue #1301: ログインエラー表示改善 + ブルートフォース攻撃対策 + AJAX統一対応

.gitignore

@@ -19,15 +19,22 @@
 /html/upload/temp_image/*
 /html/upload/temp_plugin/*
 /node_modules/*
+/.yarn/*
+/.pnp.cjs
+/.pnp.loader.mjs
 /phpunit.xml
 /playwright-report/*
 /reports/*
 /test-results/*
 /tests/tmp/*
 /vagrant/*
 /videos/*
+/scripts/*
+/screenshots/*
 composer.phar
 *.cache
 *.log
 *.tpl.php
+eccube_db
+/data/eccube.db
 !.gitkeep

data/Smarty/templates/default/frontparts/bloc/login.tpl

@@ -20,6 +20,50 @@
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
  *}-->
 
+<!--{if !$tpl_login}-->
+<script type="text/javascript">//<![CDATA[
+    $(function(){
+        $('#login_form').submit(function(e) {
+            var checkLogin = eccube.checkLoginFormInputted('login_form');
+            if (checkLogin == false) {
+                return false;
+            }
+
+            // AJAX対応
+            e.preventDefault();
+            $.ajax({
+                type: "POST",
+                url: "<!--{$smarty.const.ROOT_URLPATH}-->frontparts/login_check.php",
+                data: $('#login_form').serialize(),
+                cache: false,
+                dataType: "json",
+                error: function(xhr, textStatus, errorThrown) {
+                    if (xhr.status === 401) {
+                        try {
+                            var result = JSON.parse(xhr.responseText);
+                            if (result.error) {
+                                alert(result.error);
+                            }
+                        } catch (e) {
+                            alert('通信エラーが発生しました。');
+                        }
+                    } else {
+                        alert('通信エラーが発生しました。');
+                    }
+                },
+                success: function(result) {
+                    if (result.success) {
+                        location.href = result.success;
+                    }
+                }
+            });
+
+            return false;
+        });
+    });
+//]]></script>
+<!--{/if}-->
+
 <!--{strip}-->
     <div class="block_outer">
         <div id="login_area">
@@ -29,6 +73,7 @@
                 <input type="hidden" name="mode" value="login" />
                 <input type="hidden" name="url" value="<!--{$smarty.server.REQUEST_URI|h}-->" />
                 <div class="block_body">
+                    <div id="login_error_area" class="attention" style="margin: 5px; padding: 5px; background-color: #ffe6e6; border: 1px solid #ff9999;<!--{if !$arrErr.login}--> display: none;<!--{/if}-->"><!--{$arrErr.login|h|nl2br}--></div>
                     <!--{if $tpl_login}-->
                         <p>ようこそ<br />
                             <span class="user_name"><!--{$arrCustomer|format_name|h}--> 様</span><br />

data/Smarty/templates/default/frontparts/bloc/login_header.tpl

@@ -25,15 +25,45 @@
                 }
             });
 
-        $('#header_login_form').submit(function() {
+        $('#header_login_form').submit(function(e) {
             if (!$login_email.val()
                 || $login_email.val() == 'メールアドレス') {
                 if ($('#header_login_area input[name=login_pass]').val()) {
                     alert('メールアドレス/パスワードを入力して下さい。');
                 }
                 return false;
             }
-            return true;
+
+            // AJAX対応
+            e.preventDefault();
+            $.ajax({
+                type: "POST",
+                url: "<!--{$smarty.const.ROOT_URLPATH}-->frontparts/login_check.php",
+                data: $('#header_login_form').serialize(),
+                cache: false,
+                dataType: "json",
+                error: function(xhr, textStatus, errorThrown) {
+                    if (xhr.status === 401) {
+                        try {
+                            var result = JSON.parse(xhr.responseText);
+                            if (result.error) {
+                                alert(result.error);
+                            }
+                        } catch (e) {
+                            alert('通信エラーが発生しました。');
+                        }
+                    } else {
+                        alert('通信エラーが発生しました。');
+                    }
+                },
+                success: function(result) {
+                    if (result.success) {
+                        location.href = result.success;
+                    }
+                }
+            });
+
+            return false;
         });
     });
 //]]></script>
@@ -46,6 +76,7 @@
                 <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->" />
                 <input type="hidden" name="url" value="<!--{$smarty.server.REQUEST_URI|h}-->" />
                 <div class="block_body clearfix">
+                    <div id="header_login_error_area" class="attention" style="margin: 5px 10px; padding: 5px; background-color: #ffe6e6; border: 1px solid #ff9999;<!--{if !$arrErr.login}--> display: none;<!--{/if}-->"><!--{$arrErr.login|h|nl2br}--></div>
                     <!--{if $tpl_login}-->
                         <p class="btn">
                             ようこそ <span class="user_name"><!--{$arrCustomer|format_name|h}--> 様</span>

data/Smarty/templates/default/mypage/login.tpl

@@ -22,17 +22,57 @@
  */
 *}-->
 
+<script type="text/javascript">
+    function ajaxMyPageLogin() {
+        var checkLogin = eccube.checkLoginFormInputted('login_mypage');
+
+        if (checkLogin == false) {
+            return false;
+        }
+
+        $.ajax({
+            type: "POST",
+            url: "<!--{$smarty.const.ROOT_URLPATH}-->frontparts/login_check.php",
+            data: $('#login_mypage').serialize(),
+            cache: false,
+            dataType: "json",
+            error: function(xhr, textStatus, errorThrown) {
+                if (xhr.status === 401) {
+                    try {
+                        var result = JSON.parse(xhr.responseText);
+                        if (result.error) {
+                            $('#login_error_area').html(result.error.replace(/\n/g, '<br>')).show();
+                        }
+                    } catch (e) {
+                        alert('通信エラーが発生しました。');
+                    }
+                } else {
+                    alert('通信エラーが発生しました。');
+                }
+            },
+            success: function(result) {
+                if (result.success) {
+                    location.href = result.success;
+                }
+            }
+        });
+
+        return false;
+    }
+</script>
+
 <div id="undercolumn">
     <h2 class="title"><!--{$tpl_title|h}--></h2>
     <div id="undercolumn_login">
-        <form name="login_mypage" id="login_mypage" method="post" action="<!--{$smarty.const.HTTPS_URL}-->frontparts/login_check.php" onsubmit="return eccube.checkLoginFormInputted('login_mypage')">
+        <form name="login_mypage" id="login_mypage" method="post" action="?" onsubmit="return ajaxMyPageLogin()">
             <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->" />
             <input type="hidden" name="mode" value="login" />
             <input type="hidden" name="url" value="<!--{$smarty.server.REQUEST_URI|h}-->" />
 
             <div class="login_area">
                 <h3>会員登録がお済みのお客様</h3>
                 <p class="inputtext">会員の方は、登録時に入力されたメールアドレスとパスワードでログインしてください。</p>
+                <div id="login_error_area" class="attention" style="margin-bottom: 10px;<!--{if !$arrErr.login}--> display: none;<!--{/if}-->"><!--{$arrErr.login|h|nl2br}--></div>
                 <div class="inputbox">
                     <dl class="formlist clearfix">
                         <!--{assign var=key value="login_email"}-->

data/Smarty/templates/default/shopping/index.tpl

@@ -20,16 +20,56 @@
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
  *}-->
 
+<script type="text/javascript">
+    function ajaxLogin() {
+        var checkLogin = eccube.checkLoginFormInputted('member_form');
+
+        if (checkLogin == false) {
+            return false;
+        }
+
+        $.ajax({
+            type: "POST",
+            url: "<!--{$smarty.const.ROOT_URLPATH}-->shopping/index.php",
+            data: $('#member_form').serialize(),
+            cache: false,
+            dataType: "json",
+            error: function(xhr, textStatus, errorThrown) {
+                if (xhr.status === 401) {
+                    try {
+                        var result = JSON.parse(xhr.responseText);
+                        if (result.error) {
+                            $('#login_error_area').html(result.error).show();
+                        }
+                    } catch (e) {
+                        alert('通信エラーが発生しました。');
+                    }
+                } else {
+                    alert('通信エラーが発生しました。');
+                }
+            },
+            success: function(result) {
+                if (result.success) {
+                    location.href = '<!--{$smarty.const.ROOT_URLPATH}-->shopping/' + result.success;
+                }
+            }
+        });
+
+        return false;
+    }
+</script>
+
 <div id="undercolumn">
     <div id="undercolumn_login">
         <h2 class="title"><!--{$tpl_title|h}--></h2>
-        <form name="member_form" id="member_form" method="post" action="?" onsubmit="return eccube.checkLoginFormInputted('member_form')">
+        <form name="member_form" id="member_form" method="post" action="?" onsubmit="return ajaxLogin()">
             <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->" />
             <input type="hidden" name="mode" value="login" />
 
             <div class="login_area">
                 <h3>会員登録がお済みのお客様</h3>
                 <p class="inputtext">会員の方は、登録時に入力されたメールアドレスとパスワードでログインしてください。</p>
+                <div id="login_error_area" class="attention" style="margin-bottom: 10px;<!--{if !$arrErr.login}--> display: none;<!--{/if}-->"><!--{$arrErr.login|h|nl2br}--></div>
                 <div class="inputbox">
                     <dl class="formlist clearfix">
                         <!--{assign var=key value="login_email"}-->

data/Smarty/templates/mobile/mypage/login.tpl

@@ -27,6 +27,9 @@
         <input type="hidden" name="mode" value="login">
         <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->">
         <input type="hidden" name="url" value="<!--{$smarty.server.REQUEST_URI|h}-->">
+        <!--{if $arrErr.login}-->
+        <font color="#FF0000"><!--{$arrErr.login|h|nl2br}--></font><br>
+        <!--{/if}-->
         <!--{if !$tpl_valid_phone_id}-->
             ●メールアドレス<br>
             <!--{assign var=key value="login_email"}-->

data/Smarty/templates/mobile/shopping/index.tpl

@@ -38,6 +38,9 @@
     <form name="member_form" id="member_form" method="post" action="?">
         <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->">
         <input type="hidden" name="mode" value="login">
+        <!--{if $arrErr.login}-->
+        <font color="#FF0000"><!--{$arrErr.login|h|nl2br}--></font><br>
+        <!--{/if}-->
         <!--{if !$tpl_valid_phone_id}-->
             ■以前にご注文された方<br>
             (モバイル又はPCでご登録済み)<br>

data/Smarty/templates/sphone/frontparts/bloc/login_header.tpl

@@ -21,6 +21,9 @@
  *}-->
 
 <nav class="header_navi">
+    <!--{if $arrErr.login}-->
+        <div class="attention" style="margin: 5px; padding: 5px; background-color: #ffe6e6; border: 1px solid #ff9999;"><!--{$arrErr.login|h|nl2br}--></div>
+    <!--{/if}-->
     <!--{if $tpl_login}-->
         <p class="guest">ようこそ <a href="<!--{$smarty.const.HTTPS_URL}-->mypage/login.php"><!--{$arrCustomer|format_name|h}--> 様</a></p>
         <!--{if $smarty.const.USE_POINT !== false}-->

data/Smarty/templates/sphone/shopping/index.tpl

@@ -39,14 +39,23 @@
                 data: postData,
                 cache: false,
                 dataType: "json",
-                error: function(XMLHttpRequest, textStatus, errorThrown){
-                    alert(textStatus);
+                error: function(xhr, textStatus, errorThrown){
+                    if (xhr.status === 401) {
+                        try {
+                            var result = JSON.parse(xhr.responseText);
+                            if (result.error) {
+                                alert(result.error);
+                            }
+                        } catch (e) {
+                            alert('通信エラーが発生しました。');
+                        }
+                    } else {
+                        alert('通信エラーが発生しました。');
+                    }
                 },
                 success: function(result){
                     if (result.success) {
                         location.href = '<!--{$smarty.const.ROOT_URLPATH}-->shopping/' + result.success;
-                    } else {
-                        alert(result.login_error);
                     }
                 }
             });
@@ -56,7 +65,7 @@
 
 <section id="slidewindow">
     <h2 class="title"><!--{$tpl_title|h}--></h2>
-    <form name="member_form" id="member_form" method="post" action="javascript:;" onSubmit="return ajaxLogin()">
+    <form name="member_form" id="member_form" method="post" action="?" onSubmit="return ajaxLogin()">
         <input type="hidden" name="<!--{$smarty.const.TRANSACTION_ID_NAME}-->" value="<!--{$transactionid}-->" />
         <input type="hidden" name="mode" value="login" />
         <div class="login_area">