質問編集履歴

必要そうな情報を追記しました。

2020/05/25 12:21

投稿

yys949

スコア9

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -274,12 +274,6 @@
 	@Autowired
 	private LoginUserDao userDao;
-	/**
-	 * UserDetailsServiceインタフェースの実装メソッド
-	 * フォームから取得したユーザ名でDBを検索し、合致するものが存在したとき、
-	 * パスワード、権限情報と共にUserDetailsオブジェクトを生成
-	 * コンフィグクラスで上入力値とDBから取得したパスワードと比較し、ログイン判定を行う
-	 */
 	@Override
 	public UserDetails loadUserByUsername(String userId) throws UsernameNotFoundException {
@@ -288,9 +282,6 @@
 		if (user == null) {
 			throw new UsernameNotFoundException("User" + userId + "was not found in the database");
 		}
-		//権限のリスト
-		//AdminやUserなどが存在するが、今回は利用しないのでUSERのみを仮で設定
-		//権限を利用する場合は、DB上で権限テーブル、ユーザ権限テーブルを作成し管理が必要
 		List<GrantedAuthority> grantList = new ArrayList<GrantedAuthority>();
 		GrantedAuthority authority = new SimpleGrantedAuthority("USERS");
 		grantList.add(authority);
@@ -298,7 +289,6 @@
 		//rawDataのパスワードは渡すことができないので、暗号化
 		BCryptPasswordEncoder encoder = new BCryptPasswordEncoder();
-		//UserDetailsはインタフェースなのでUserクラスのコンストラクタで生成したユーザオブジェクトを
 		UserDetails userDetails = (UserDetails)new User(user.getUserName(), encoder.encode(user.getPassword()),grantList);
 		return userDetails;

必要そうな情報を追記しました。

2020/05/25 12:21

投稿

yys949

スコア9

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -168,7 +168,144 @@
 </body>
 </html>
 ```
+```UserEntity.java
+@Entity
+@Table(name = "USERS")
+public class UserEntity implements UserDetails{
+	/**
+	 *
+	 */
+	private static final long serialVersionUID = 1L;
+	@Id
+	@Column(name = "USER_ID")
+	@NotEmpty
+	private String userId;
+	@Column(name = "USER_NAME")
+	@NotEmpty
+	private String userName;
+	@Column(name = "ADMIN_FLG")
+	private int adminFlg;
+	@Column(name = "PASSWORD")
+	@NotEmpty
+	private String password;
+	public String getUserId() {
+		return userId;
+	}
+	public void setUserId(String userId) {
+		this.userId = userId;
+	}
+	public String getUserName() {
+		return userName;
+	}
+	public void setUserName(String userName) {
+		this.userName = userName;
+	}
+	public int getAdminFlg() {
+		return adminFlg;
+	}
+	public void setAdminFlg(int adminFlg) {
+		this.adminFlg = adminFlg;
+	}
+	public String getPassword() {
+		return password;
+	}
+	public void setPassword(String password) {
+		this.password = password;
+	}
+	@Override
+	public Collection<? extends GrantedAuthority> getAuthorities() {
+		// TODO 自動生成されたメソッド・スタブ
+		return null;
+	}
+	@Override
+	public String getUsername() {
+		// TODO 自動生成されたメソッド・スタブ
+		return null;
+	}
+	@Override
+	public boolean isAccountNonExpired() {
+		// TODO 自動生成されたメソッド・スタブ
+		return false;
+	}
+	@Override
+	public boolean isAccountNonLocked() {
+		// TODO 自動生成されたメソッド・スタブ
+		return false;
+	}
+	@Override
+	public boolean isCredentialsNonExpired() {
+		// TODO 自動生成されたメソッド・スタブ
+		return false;
+	}
+	@Override
+	public boolean isEnabled() {
+		// TODO 自動生成されたメソッド・スタブ
+		return false;
+	}
+}
+```
+```JpaUserDetailsServiceImpl.java
+@Component
+@Service
+public class JpaUserDetailsServiceImpl implements UserDetailsService {
+	//DBからユーザ情報を検索するメソッドを実装したクラス
+	@Autowired
+	private LoginUserDao userDao;
+	/**
+	 * UserDetailsServiceインタフェースの実装メソッド
+	 * フォームから取得したユーザ名でDBを検索し、合致するものが存在したとき、
+	 * パスワード、権限情報と共にUserDetailsオブジェクトを生成
+	 * コンフィグクラスで上入力値とDBから取得したパスワードと比較し、ログイン判定を行う
+	 */
+	@Override
+	public UserDetails loadUserByUsername(String userId) throws UsernameNotFoundException {
+		UserEntity user = userDao.findUser(userId);
+		if (user == null) {
+			throw new UsernameNotFoundException("User" + userId + "was not found in the database");
+		}
+		//権限のリスト
+		//AdminやUserなどが存在するが、今回は利用しないのでUSERのみを仮で設定
+		//権限を利用する場合は、DB上で権限テーブル、ユーザ権限テーブルを作成し管理が必要
+		List<GrantedAuthority> grantList = new ArrayList<GrantedAuthority>();
+		GrantedAuthority authority = new SimpleGrantedAuthority("USERS");
+		grantList.add(authority);
+		//rawDataのパスワードは渡すことができないので、暗号化
+		BCryptPasswordEncoder encoder = new BCryptPasswordEncoder();
+		//UserDetailsはインタフェースなのでUserクラスのコンストラクタで生成したユーザオブジェクトを
+		UserDetails userDetails = (UserDetails)new User(user.getUserName(), encoder.encode(user.getPassword()),grantList);
+		return userDetails;
+	}
+}
+```
 ### 試したこと
 上述した通り、Principalを使うとユーザー名だけは取得できました。

必要そうな情報を追記しました。

2020/05/25 12:20

投稿

yys949

スコア9

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -30,7 +30,7 @@
 	 */
 	@PostMapping("/index")
 	public String loginFormSubmit(@Valid @ModelAttribute UserForm loginUser,
-									BindingResult result) {
+									BindingResult result, Model model) {
 		if(result.hasErrors()) {
 			return "/login";
@@ -43,14 +43,21 @@
 			return "/login";
 		}
+		setLoginInfo(userInfo);
 		model.addAttribute(loginUser);
 		return "redirect:/index";
 	}
+	private void setLoginInfo(UserEntity userEntityResult) {
+		this.userForm.setUserId(userEntityResult.getUserId());
+		this.userForm.setPassword(userEntityResult.getPassword());
+		this.userForm.setUserName(userEntityResult.getUserName());
+	}
 	@GetMapping("/index")
 	public String afterLogin(@ModelAttribute UserForm loginUser, Model model) {
@@ -63,7 +70,52 @@
 }
 ```
+```UserService.java
+@Service
+@Transactional
+public class UserService {
+	@Autowired
+	private UserRepository userRepository;
+	@Autowired
+	LoginUserDao loginUserDao;
+	@Autowired
+	private BCryptPasswordEncoder passwordEncoder;
+	public UserEntity create(UserEntity account, String rawPassword) {
+		String encodePassword = passwordEncoder.encode(rawPassword);
+		account.setPassword(encodePassword);
+		return userRepository.save(account);
+	}
+	public UserEntity loginCheck(String userId, String password) {
+		return loginUserDao.findUser(userId);
+	}
+}
+```
+```LoginUserDao.java
+@Repository
+public class LoginUserDao {
+	@Autowired
+	EntityManager em;
+	public UserEntity findUser(String userId) {
+		String query = "";
+		query += "SELECT * ";
+		query += "FROM USERS ";
+		query += "WHERE USER_ID = :userId ";
+		return (UserEntity)em.createNativeQuery(query, UserEntity.class).setParameter("userId", userId)
+				.getSingleResult();
+	}
+}
+```
 ```login.html
 <!DOCTYPE html>
 <html xmlns:th="http://thymeleaf.org">
@@ -126,7 +178,9 @@
 @ModelAttributeを両方に書けばいくかと思って書いたのですがデバッグするとUserFormがnullになっていました。
+上記のような感じで書いてあります。
-ほかにどの情報を書けばいいかわからなかったのでとりあえずControllerの処理とhtmlのほうだけ書きましたが必要な情報あれば教えてください。
+ログイン先の画面への移行はできたのですが、そもそものuserInfoとしたところがユーザー情報をすべて返すように作ったと思ったのにnullが返ってきています。
+dao内の書き方が悪いのでしょうか。
 全然わからなくなってしまったため教えていただきたいです。
 お手数おかけしますがよろしくお願いします。

誤字

2020/05/25 12:05

投稿

yys949

スコア9

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -54,7 +54,7 @@
 	@GetMapping("/index")
 	public String afterLogin(@ModelAttribute UserForm loginUser, Model model) {
-		model.addAttribute("userId", userId);
+		model.addAttribute("loginUser", loginUser);
 		return "/index";
 	}