web-dev-qa-db-ja.com

ログイン用の認証フィルターとサーブレット

ログインに使用するフィルターがあります。フィールド「ユーザー名」と「パスワード」でテキストチェックを実行します。テキストチェックが正しく行われた場合にのみ、リクエストはサーブレットに送信されます。後者は、データベースと対話する必要がある制御を実行します。このチェーンは正しいですか？

authenticationservletsservlet-filterslogin-control
21
paolo2988

序文：コンテナ管理のログインの代わりに、自家製のログインを使用していると思います。すべての方法については、「 データベース内のユーザーとの認証/承認を処理する方法？ 」を参照してください。

フィルター（インターセプター）は、ユーザー名とパスワードのコンボの有効性をチェックするべきではありません。それは、サーブレット（コントローラー）の責任です。

フィルターは単にユーザーがログインしているかどうかを確認するだけで（通常はセッション属性の存在を確認するだけで）、リクエストを続行するか、ログインページにリダイレクトしてブロックします。

@WebFilter("/*")
public class LoginFilter implements Filter {

    @Override
    public void doFilter(ServletRequest req, ServletResponse res, FilterChain chain) throws ServletException, IOException {    
        HttpServletRequest request = (HttpServletRequest) req;
        HttpServletResponse response = (HttpServletResponse) res;
        HttpSession session = request.getSession(false);
        String loginURI = request.getContextPath() + "/login";

        boolean loggedIn = session != null && session.getAttribute("user") != null;
        boolean loginRequest = request.getRequestURI().equals(loginURI);

        if (loggedIn || loginRequest) {
            chain.doFilter(request, response);
        } else {
            response.sendRedirect(loginURI);
        }
    }

    // ...
}

サーブレットは、送信されたデータを収集し、データベースで関連するUserを見つけ、見つかった場合はセッション属性として保存し、ホームページにリダイレクトするか、検証エラーでフォームを再表示する必要があります。

@WebServlet("/login")
public class LoginServlet extends HttpServlet {

    @EJB
    private UserService userService;

    @Override
    protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        request.getRequestDispatcher("/WEB-INF/login.jsp").forward(request, response);
    }

    @Override
    protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        String username = request.getParameter("username");
        String password = request.getParameter("password");
        Map<String, String> messages = new HashMap<String, String>();

        if (username == null || username.isEmpty()) {
            messages.put("username", "Please enter username");
        }

        if (password == null || password.isEmpty()) {
            messages.put("password", "Please enter password");
        }

        if (messages.isEmpty()) {
            User user = userService.find(username, password);

            if (user != null) {
                request.getSession().setAttribute("user", user);
                response.sendRedirect(request.getContextPath() + "/home");
                return;
            } else {
                messages.put("login", "Unknown login, please try again");
            }  
        }

        request.setAttribute("messages", messages);
        request.getRequestDispatcher("/WEB-INF/login.jsp").forward(request, response);
    }

}

こちらもご覧ください：

48
BalusC