質問内容

現在、簡易的なログイン証認機能の実装をしているのですが、ログイン成功後の画面が表示されず困っています。具体的な状況を説明しますと、画像にもある通り、正しいログインIDとパスワードを入力してログインボタンを押すと、本来ならばメニュー画面(menu.jsp)が表示されるところ、画面推移はしているものの何も表示されないという状況です。
画像にもある通り、画面推移後のURLを見たところ、最後に "authentication" という別フォルダが指定されているのが何か今回のエラーと関係があるのではないかと考えているのですが、どこをどのようになおせばいいのか分からなかったので、また質問させて頂くことにしました。
ご教授頂ければ幸いです。宜しくお願いします。

発生している問題の詳細・エラーメッセージ

コンソール

ログイン画面 (index.jsp)

ログイン証認が成功し、画面推移後のページ(menu.jsp)

該当のソースコード

index.jsp

Java
1<%@ page language="java" contentType="text/html; charset=Windows-31J"
2    pageEncoding="Windows-31J" isELIgnored="false"%>
3<%@ taglib prefix="c" uri="http://java.sun.com/jsp/jstl/core" %>
4<html>
5<head><meta charset="UTF-8">
6<link rel="stylesheet" type="text/css" href="<%=request.getContextPath() %>/login.css">
7<title>Insert title here</title>
8</head>
9<body>
10	<h3>ログイン画面</h3>
11    <p>
12	<c:if test="${!empty errorMessages}">
13	<font color="red">ログインIDかパスワードが間違っています。<br>再度入力して下さい。</font>
14	</c:if>
15    <form method="POST" action="authentication" name="login_form">
16    	<div class="login_form_input">
17         ログインID : <input type="text" name="user_id"><br>
18         パスワード : <input type="password" name="user_pass">
19        </div>
20        <button type="submit" name="submit">ログイン</button>  
21    </form>
22</body>
23</html>

LoginServlet.java

Java
1package servlet;
2
3import java.io.IOException;
4import java.sql.Connection;
5import java.sql.DriverManager;
6import java.sql.PreparedStatement;
7import java.sql.ResultSet;
8import java.sql.SQLException;
9import java.util.ArrayList;
10import java.util.List;
11
12import javax.servlet.ServletException;
13import javax.servlet.annotation.WebServlet;
14import javax.servlet.http.HttpServlet;
15import javax.servlet.http.HttpServletRequest;
16import javax.servlet.http.HttpServletResponse;
17
18import authentications.LoginAuthentication;
19
20@WebServlet(urlPatterns = {"/authentication"})
21public class LoginServlet extends HttpServlet{
22	private static final long serialVersionUID = 1L;
23	@Override
24	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws IOException, ServletException {
25		try {
26			String sql = "SELECT user_id FROM imada.mst_user WHERE user_id = ? AND user_pass = ?";
27			Connection connection = DriverManager.getConnection("jdbc:mysql://localhost:3306/imada?serverTimezone=Asia/Tokyo","root","*****");
28			PreparedStatement ps = connection.prepareStatement(sql);
29			LoginAuthentication authen = new LoginAuthentication();
30			authen.setUser_id(request.getParameter("user_id"));
31			authen.setUser_pass(request.getParameter("user_pass"));
32			ps.setString(1,authen.getUser_id());
33			ps.setString(2,authen.getUser_pass());
34			ResultSet rs = ps.executeQuery();
35			List<LoginAuthentication> user = toUserList(rs);
36			if (user.isEmpty() == true) {
37				List<String> messages = new ArrayList<String>();
38				messages.add("ログインに失敗しました。");
39				request.setAttribute("errorMessages", messages);
40				getServletContext().getRequestDispatcher("/index.jsp").forward(request,response);
41			} else if (2 <= user.size()) {
42				throw new IllegalStateException("2 <= userList.size()");
43			} else {		
44				request.getSession().setAttribute("login", user.get(0));
45				getServletContext().getRequestDispatcher("/menu.jsp").forward(request,response);
46					
47			}
48			rs.close();
49			ps.close();
50			connection.close();
51		} catch (Exception e) {
52			e.printStackTrace();
53		}
54	}
55	
56	private List<LoginAuthentication> toUserList(ResultSet rs) throws SQLException{
57		List<LoginAuthentication> ret = new ArrayList<LoginAuthentication>();
58		try {
59			while (rs.next()) {
60				String user_id = rs.getString("user_id");
61				LoginAuthentication user = new LoginAuthentication();
62				user.setUser_id(user_id);
63				ret.add(user);
64			}
65		} catch (Exception e) {
66			e.printStackTrace();
67		}
68		return ret;
69	}
70}

LogoutServlet.java

Java
1package servlet;
2
3import java.io.IOException;
4
5import javax.servlet.ServletException;
6import javax.servlet.annotation.WebServlet;
7import javax.servlet.http.HttpServlet;
8import javax.servlet.http.HttpServletRequest;
9import javax.servlet.http.HttpServletResponse;
10
11@WebServlet(urlPatterns = {"/logout"})
12public class LogoutServlet extends HttpServlet{
13	private static final long serialVersionUID = 1L;	
14	@Override
15	protected void doPost(HttpServletRequest request,HttpServletResponse response) throws IOException, ServletException {		try{
16			request.getSession().invalidate();
17		}catch(Exception e){
18			throw new ServletException(e);
19		}
20		getServletContext().getRequestDispatcher("/index.jsp").forward(request,response);   
21	}
22	
23}

menu.jsp

Java
1<%@ page language="java" contentType="text/html; charset=UTF-8"
2    pageEncoding="UTF-8"%>
3<html>
4<head>
5    <link rel="stylesheet" type="text/css" href="<%=request.getContextPath() %>/menu.css">
6    <title>メニュー画面</title>
7</head>
8<body>
9	ようこそ${login.user_name}さん。
10    <form action="logout" method="post">
11		<input type="submit" name="submit" value="ログアウト"/>
12	</form>
13    <h3>メニュー画面</h3>
14    <ul>
15        <li><a href="#">売上実績参照画面</a></li>
16        <li><a href="#">ユーザー登録画面</a></li>
17        <li><a href="#">売上実績サマリ確認画面</a></li>
18    </ul>
19</body>
20</html>

LoginAuthentication.java

Java
1package authentications;
2
3public class LoginAuthentication {
4	private String user_id;
5	private String user_pass;
6	
7	public String getUser_id() {
8		return user_id;
9	}
10	public void setUser_id(String user_id) {
11		this.user_id = user_id;
12	}
13	public String getUser_pass() {
14		return user_pass;
15	}
16	public void setUser_pass(String user_pass) {
17		this.user_pass = user_pass;
18	}
19}

web.xml

<?xml version="1.0" encoding="Windows-31J"?>

<web-app
  xmlns="https://jakarta.ee/xml/ns/jakartaee_9.xsd"
  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
  xsi:schemaLocation="https://jakarta.ee/xml/ns/jakartaee
  https://jakarta.ee/xml/ns/jakartaee/web-app_5_0.xsd"
  version="5.0">
     
<servlet>
<servlet-name>LoginServlet</servlet-name>
<servlet-class>servlet.LoginServlet</servlet-class>
</servlet>
<servlet>
<servlet-name>LogoutServlet</servlet-name>
<servlet-class>servlet.LogoutServlet</servlet-class>
</servlet>
<servlet>
<servlet-name>LoginAuthentication</servlet-name>
<servlet-class>authentication.LoginAuthentication</servlet-class>
</servlet>
<servlet-mapping>
<servlet-name>LoginServlet</servlet-name>
<url-pattern>/servlet/LoginServlet</url-pattern>
</servlet-mapping>
<servlet-mapping>
<servlet-name>LogoutServlet</servlet-name>
<url-pattern>/servlet/LogoutServlet</url-pattern>
</servlet-mapping>
<servlet-mapping>
<servlet-name>LoginAuthentication</servlet-name>
<url-pattern>/authentication/LoginAuthentication</url-pattern>
</servlet-mapping>
<session-config>
<session-timeout>30</session-timeout>
</session-config>
<welcome-file-list>
<welcome-file>index.jsp</welcome-file>
</welcome-file-list>
</web-app>

補足情報（FW/ツールのバージョンなど）

Windows - 10
Microsoft Edge - 103.0.1264.62
Java - 17
Tomcat - 9
MySQL -8.0
mysql-connector-java - 8.0.29

現在のファイル構造

ビルドパス

行動規範の内容に同意します

回答2件

ベストアンサー

SELECT user_id FROM

user_name
という項目は取得されていないように見受けられます。

あと、パスワードが平文ぽかったり、
モデルでやるべきところをコントローラ（サーブレットクラス）でやってたり
細かいところは気にはなりますね。

投稿2022/07/22 07:49

編集2022/07/22 07:51

m.ts10806

総合スコア80850

RYU

2022/07/22 08:06

とても初歩的なミスで、ご指摘の通り"user_name"ではなく"user_id"でした。正しい変数名に変更したところ、しかっりと表示されました! パスワードの部分は一時的にアスタリスクにしていました。大変助かりました、ありがとうございました!

m.ts10806

2022/07/22 08:10 編集

仕様の問題ですが、セッションに持たせたいならuser_id以外も取得させた方が良いと思います。＞パスワードの部分は一時的にアスタリスクにしていましたいえ、セキュリティの問題です。通常は暗号化した文字列をDBに保存し、ログイン認証時に同じ形式で暗号化した文字列と照合し、パスワード流出を防ぎます。（DBみれるような開発者であってもユーザのパスワードが分かるべきではない。パスワードはあくまで本人だけのものということ）

RYU

2022/07/22 08:56

なるほど、そうなのですね。もっと自分なりに調べてみようと思います。

行動規範の内容に同意します

動作確認してないので、憶測ですみません。

不具合原因

menu.jspへ遷移する処理が抜けている。

サンプルコード

LoginServlet.java
1protected void doPost(HttpServletRequest request, HttpServletResponse response) throws IOException, ServletException {
2...途中省略
3// 最後にmenu.jspへの遷移を追加。
4        RequestDispatcher dispatcher =  request.getRequestDispatcher("/menu.jsp");
5        dispatcher.forward(request, response);
6}