SpringSecurityを使ったログイン認証がしたい

Question

前提・実現したいこと

SpringSecurityを使ったログイン認証機能を作っています。
新しいユーザーの登録機能を実装したのですが、その時にパスワードはハッシュ化して保存するようにしました。
ハッシュ化したのはいいんですが、認証ができずに困っています。

@Component
@Service
public class JpaUserDetailsServiceImpl implements UserDetailsService {
	
	//DBからユーザ情報を検索するメソッドを実装したクラス
	@Autowired
	private LoginUserDao userDao;

	@Autowired
	HttpSession session;
	
	
	
	/**
	 * 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");
		}
		
		session.setAttribute("userId", userId);
		
		List<GrantedAuthority> grantList = new ArrayList<GrantedAuthority>();
		GrantedAuthority authority = new SimpleGrantedAuthority("USERS");
		grantList.add(authority);
		
		BCryptPasswordEncoder encoder = new BCryptPasswordEncoder();
		
		UserDetails userDetails = (UserDetails)new User(user.getUserName(),encoder.encode(user.getPassword()),grantList);
		
		return userDetails;
	}	
}

@Configuration
@EnableWebSecurity
public class WebSecurityConfig extends WebSecurityConfigurerAdapter {
	
	@Autowired
    private JpaUserDetailsServiceImpl userDetailsService;
	
	@Autowired
	private SuccessHandler successHandler;
	
	@Autowired
	private FailureHandler failureHandler;
	
	@Override
	public void configure(WebSecurity web) throws Exception {
		web.ignoring().antMatchers(
						"/images/**",
						"/css/**",
						"/js/**"
						);
	}
	
	@Override
	protected void configure(HttpSecurity http) throws Exception {
		http
			.authorizeRequests()
				.antMatchers("/add/**").permitAll()
				.anyRequest().authenticated()
				.and()
			.formLogin()
				.loginPage("/login")
				.loginProcessingUrl("/index")
				.failureUrl("/login?error")
				.defaultSuccessUrl("/index")
				.usernameParameter("userId")
				.passwordParameter("password").permitAll()
				.successHandler(successHandler)
				.failureHandler(failureHandler)
				.and()
			.logout()
				.logoutUrl("/logout")
				.logoutSuccessUrl("/login?logout")
				.permitAll();
	}
	
	@Bean
	PasswordEncoder passwordEncod() {
		return new BCryptPasswordEncoder();
	}

	@Autowired
	public void configure(AuthenticationManagerBuilder auth) throws Exception{
        auth.userDetailsService(userDetailsService).passwordEncoder(passwordEncoder());
   　　 }

}

@Controller
public class LoginController {

	@Autowired
	UserForm userForm;

	@Autowired
	UserService userService;

	@Autowired
	private LoginUserDao userDao;

	@Autowired
	HttpSession session;

	@GetMapping({ "/", "/login" })
	public String loginForm(UserForm userForm, Model model) {

		return "/login";
	}

	@GetMapping("/index")
	public String afterLogin(Model model) {

		String userId = (String) session.getAttribute("userId");
		UserEntity userInfo = userDao.findUser(userId);
		model.addAttribute("loginUser", userInfo);
		
		System.out.println(userInfo.getPassword());

		return "/index";
	}

}

WebSecurityConfig.javaでBCryptPasswordEncoderを設定して、JpaUserDetailsServiceImplで入力されたパスワードをハッシュ化し取得しするようにしています。
ただ今のままだと、入力されたパスワードと保存されているパスワードそれぞれのハッシュ化した値が異なるのか認証されません。
平文だと認証するのですが、、、

ハッシュ化し保存したパスワードで認証を行うにはどこを修正すればうまくいくのでしょうか。

全然見当がつかず、申し訳ないのですが、教えていただけないでしょうか。
よろしくお願いします。

Comments

[退会済みユーザー]

ぱっとみ「JpaUserDetailsServiceImpl 」は呼ばれてますか？

[yys949]

失礼しました。

ソースコピペしてきたつもりが読んでいるところを書いていなかったです。

[退会済みユーザー]

> UserDetails userDetails = (UserDetails)new User(user.getUserName(),encoder.encode(user.getPassword()),grantList);
暗号化済みのを暗号化してる？
それとも平文を暗号化してる？

[yys949]

よくみたら暗号化済みをさらに暗号化していました。

すでに暗号化済みで取得できていたのに平文を暗号化して照合させているものと勘違いしてさらに暗号化させていたためおかしなことになっているようでした。

Answer 1

asahina1979様の

UserDetails userDetails = (UserDetails)new User(user.getUserName(),encoder.encode(user.getPassword()),grantList);

暗号化済みのを暗号化してる？
それとも平文を暗号化してる？

とのコメントより、よく確認したところ暗号化済みのものをまた暗号化していました。

BCryptPasswordEncoder encoder = new BCryptPasswordEncoder();

        UserDetails userDetails = (UserDetails)new User(user.getUserName(),encoder.encode(user.getPassword()),grantList);

としていたところを、

//		BCryptPasswordEncoder encoder = new BCryptPasswordEncoder();
		
		//UserDetailsはインタフェースなのでUserクラスのコンストラクタで生成したユーザオブジェクトを
		UserDetails userDetails = (UserDetails)new User(user.getUserName(), user.getPassword(),grantList);

とし暗号化したものを取得するだけにしたら問題なく認証ができました。