SpringSecurityでハッシュ化したパスワードを照合

Question

前提・実現したいこと

SpringSecurityのPasswordEncoderであらかじめハッシュ化してDBに登録しておいたパスワードと画面上で入力したパスワードを照合したいです。（パスワード変更画面を作っていて、現在のパスワードと照合するため）
しかしうまく行かず、ソース上の「passwordEncoder.matches」でif文したときに不一致判定になってしまいます。。。。
ご教授お願いいたします。

該当のソースコード

こちらが照合をしてるソースです。(DB上のソースは一旦ベタガキしてます。)
画面から入力した値はpasswordChange.getNowPassword()で取得できています。

package com.kproject01.controller;

import java.security.Principal;

import javax.validation.Valid;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.context.annotation.Bean;
import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
import org.springframework.security.crypto.password.PasswordEncoder;
import org.springframework.stereotype.Controller;
import org.springframework.validation.BindingResult;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.ModelAttribute;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RequestMethod;
import org.springframework.web.servlet.ModelAndView;

import com.kproject01.entity.PasswordChange;
import com.kproject01.entity.UserEdit;

@Controller
public class PasswordChangeController {

	@Autowired
	PasswordEncoder passwordEncoder;
	
	/**
	 * 変更処理
	 * @param passwordChange
	 * @param bindingResult
	 * @param mav
	 * @param principal
	 * @return
	 */
	@RequestMapping(value = "password-change-go", method = RequestMethod.POST)
	public ModelAndView passwordChangeGo(@Valid @ModelAttribute PasswordChange passwordChange, BindingResult bindingResult, ModelAndView mav, Principal principal){
		System.out.println("DB上パスワード" + "$2a$10$rXo1P2N6HIrTA.735CAA0eFMfAgAD00ABIhzHioCr.Xe7HnVadTsi");
		//DB上のパスワードベタガキ
		String dbPass = "$2a$10$rXo1P2N6HIrTA.735CAA0eFMfAgAD00ABIhzHioCr.Xe7HnVadTsi";
		//入力したパスワード
		String inPass = passwordChange.getNowPassword();
		//パスワード照合
		if (passwordEncoder.matches(dbPass, inPass)) {
            System.out.println("一致");
        }else{
        	System.out.println("不一致");
        }
		return null;	
	}
}

一応コンフィグファイルは以下です。

package com.kproject01.config;

import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;
import org.springframework.security.config.annotation.web.builders.HttpSecurity;
import org.springframework.security.config.annotation.web.builders.WebSecurity;
import org.springframework.security.config.annotation.web.configuration.EnableWebSecurity;
import org.springframework.security.config.annotation.web.configuration.WebSecurityConfigurerAdapter;
import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
import org.springframework.security.crypto.password.PasswordEncoder;

@Configuration
@EnableWebSecurity
public class SecurityConfig extends WebSecurityConfigurerAdapter {

	@Override
	public void configure(WebSecurity web) throws Exception {
		web.ignoring().antMatchers("/webjars/**", "/favicon.ico", "/css/**",
				"/js/**", "/img/**");
	}

	@Override
	protected void configure(HttpSecurity http) throws Exception {
		http.authorizeRequests().antMatchers("/")
				.permitAll().antMatchers("/admin").hasRole("ADMIN")
				.anyRequest().authenticated().and().formLogin()
				.loginProcessingUrl("/authenticate").loginPage("/")
				.failureUrl("/?error").defaultSuccessUrl("/user-top", true)
				.usernameParameter("username").passwordParameter("password")
				.and().logout().logoutSuccessUrl("/").and();
	}

	@Bean
    PasswordEncoder passwordEncoder() {
        return new BCryptPasswordEncoder();
    }

}

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

java8
SpringBoot2.0

Answer 1

matchesメソッドに渡す引数が逆ではないかと思います。

passwordEncoder.matches(dbPass, inPass)

Interface PasswordEncoderのJavaDocをみると

matches(java.lang.CharSequence rawPassword, java.lang.String encodedPassword)

Verify the encoded password obtained from storage matches the submitted raw password after it too is encoded.

上記に引用したように第1引数が入力された生のパスワード(raw password)で、第2引数がDBなどに保存しているエンコードされたパスワード(encoded password)です。

一度、引数の順番を変えて試してみてください。

Comments

[takahiro00]

引数の順番を変えたらうまくできました！
ありがとうございます。