reformat code

This commit is contained in:
akuksin
2020-02-22 23:39:48 +01:00
parent 9a378b5763
commit d55ee992f3
18 changed files with 284 additions and 292 deletions

View File

@@ -6,7 +6,7 @@ import org.springframework.boot.autoconfigure.SpringBootApplication;
@SpringBootApplication @SpringBootApplication
public class PasswordEncodingSpringBootApplication { public class PasswordEncodingSpringBootApplication {
public static void main(String[] args) { public static void main(String[] args) {
SpringApplication.run(PasswordEncodingSpringBootApplication.class, args); SpringApplication.run(PasswordEncodingSpringBootApplication.class, args);
} }
} }

View File

@@ -9,20 +9,20 @@ import org.springframework.transaction.annotation.Transactional;
@Service @Service
public class JdbcUserDetailPasswordService implements UserDetailsPasswordService { public class JdbcUserDetailPasswordService implements UserDetailsPasswordService {
private final UserRepository userRepository; private final UserRepository userRepository;
private final UserDetailsMapper userDetailsMapper; private final UserDetailsMapper userDetailsMapper;
public JdbcUserDetailPasswordService( public JdbcUserDetailPasswordService(
UserRepository userRepository, UserDetailsMapper userDetailsMapper) { UserRepository userRepository, UserDetailsMapper userDetailsMapper) {
this.userRepository = userRepository; this.userRepository = userRepository;
this.userDetailsMapper = userDetailsMapper; this.userDetailsMapper = userDetailsMapper;
} }
@Override @Override
public UserDetails updatePassword(UserDetails user, String newPassword) { public UserDetails updatePassword(UserDetails user, String newPassword) {
UserCredentials userCredentials = userRepository.findByUsername(user.getUsername()); UserCredentials userCredentials = userRepository.findByUsername(user.getUsername());
userCredentials.setPassword(newPassword); userCredentials.setPassword(newPassword);
return userDetailsMapper.toUserDetails(userCredentials); return userDetailsMapper.toUserDetails(userCredentials);
} }
} }

View File

@@ -10,19 +10,19 @@ import org.springframework.transaction.annotation.Transactional;
@Transactional @Transactional
public class JdbcUserDetailsService implements UserDetailsService { public class JdbcUserDetailsService implements UserDetailsService {
private final UserRepository userRepository; private final UserRepository userRepository;
private final UserDetailsMapper userDetailsMapper; private final UserDetailsMapper userDetailsMapper;
public JdbcUserDetailsService( public JdbcUserDetailsService(
UserRepository userRepository, UserDetailsMapper userDetailsMapper) { UserRepository userRepository, UserDetailsMapper userDetailsMapper) {
this.userRepository = userRepository; this.userRepository = userRepository;
this.userDetailsMapper = userDetailsMapper; this.userDetailsMapper = userDetailsMapper;
} }
@Override @Override
public UserDetails loadUserByUsername(String username) throws UsernameNotFoundException { public UserDetails loadUserByUsername(String username) throws UsernameNotFoundException {
UserCredentials userCredentials = userRepository.findByUsername(username); UserCredentials userCredentials = userRepository.findByUsername(username);
return userDetailsMapper.toUserDetails(userCredentials); return userDetailsMapper.toUserDetails(userCredentials);
} }
} }

View File

@@ -16,16 +16,17 @@ import java.util.Set;
@Table(name = "users") @Table(name = "users")
public class UserCredentials { public class UserCredentials {
@Id private String username; @Id
private String username;
private String password; private String password;
boolean enabled; boolean enabled;
@ElementCollection @ElementCollection
@JoinTable( @JoinTable(
name = "authorities", name = "authorities",
joinColumns = {@JoinColumn(name = "username")}) joinColumns = {@JoinColumn(name = "username")})
@Column(name = "authority") @Column(name = "authority")
private Set<String> roles; private Set<String> roles;
} }

View File

@@ -6,5 +6,5 @@ import org.springframework.transaction.annotation.Transactional;
@Transactional @Transactional
public interface UserRepository extends JpaRepository<UserCredentials, String> { public interface UserRepository extends JpaRepository<UserCredentials, String> {
UserCredentials findByUsername(String username); UserCredentials findByUsername(String username);
} }

View File

@@ -29,84 +29,75 @@ import java.util.Map;
@EnableWebSecurity @EnableWebSecurity
public class SecurityConfiguration extends WebSecurityConfigurerAdapter { public class SecurityConfiguration extends WebSecurityConfigurerAdapter {
private final UserRepository userRepository; private final UserRepository userRepository;
private final UserDetailsMapper userDetailsMapper; private final UserDetailsMapper userDetailsMapper;
private final BcCryptWorkFactorService bcCryptWorkFactorService; private final BcCryptWorkFactorService bcCryptWorkFactorService;
public SecurityConfiguration( public SecurityConfiguration(
UserRepository userRepository, UserRepository userRepository,
UserDetailsMapper userDetailsMapper, UserDetailsMapper userDetailsMapper,
BcCryptWorkFactorService bcCryptWorkFactorService) { BcCryptWorkFactorService bcCryptWorkFactorService) {
this.userRepository = userRepository; this.userRepository = userRepository;
this.userDetailsMapper = userDetailsMapper; this.userDetailsMapper = userDetailsMapper;
this.bcCryptWorkFactorService = bcCryptWorkFactorService; this.bcCryptWorkFactorService = bcCryptWorkFactorService;
} }
@Override @Override
protected void configure(HttpSecurity httpSecurity) throws Exception { protected void configure(HttpSecurity httpSecurity) throws Exception {
httpSecurity httpSecurity
.csrf() .csrf()
.disable() .disable()
.authorizeRequests() .authorizeRequests()
.antMatchers("/registration") .antMatchers("/registration")
.permitAll() .permitAll()
.anyRequest() .anyRequest()
.authenticated() .authenticated()
.and() .and()
.httpBasic(); .httpBasic();
httpSecurity.headers().frameOptions().disable(); httpSecurity.headers().frameOptions().disable();
} }
@Autowired @Autowired
public void configureGlobal(AuthenticationManagerBuilder auth) throws Exception { public void configureGlobal(AuthenticationManagerBuilder auth) throws Exception {
auth.authenticationProvider(daoAuthenticationProvider()).eraseCredentials(false); auth.authenticationProvider(daoAuthenticationProvider()).eraseCredentials(false);
} }
@Bean @Bean
public PasswordEncoder passwordEncoder() { public PasswordEncoder passwordEncoder() {
// we must user deprecated encoder to support their encoding // we must user deprecated encoder to support their encoding
String encodingId = "bcrypt"; String encodingId = "bcrypt";
Map<String, PasswordEncoder> encoders = new HashMap<>(); Map<String, PasswordEncoder> encoders = new HashMap<>();
encoders.put( encoders.put(encodingId, new BCryptPasswordEncoder(bcCryptWorkFactorService.calculateStrength()));
encodingId, new BCryptPasswordEncoder(bcCryptWorkFactorService.calculateStrength())); encoders.put("ldap", new org.springframework.security.crypto.password.LdapShaPasswordEncoder());
encoders.put("ldap", new org.springframework.security.crypto.password.LdapShaPasswordEncoder()); encoders.put("MD4", new org.springframework.security.crypto.password.Md4PasswordEncoder());
encoders.put("MD4", new org.springframework.security.crypto.password.Md4PasswordEncoder()); encoders.put("MD5", new org.springframework.security.crypto.password.MessageDigestPasswordEncoder("MD5"));
encoders.put( encoders.put("noop", org.springframework.security.crypto.password.NoOpPasswordEncoder.getInstance());
"MD5", encoders.put("pbkdf2", new Pbkdf2PasswordEncoder());
new org.springframework.security.crypto.password.MessageDigestPasswordEncoder("MD5")); encoders.put("scrypt", new SCryptPasswordEncoder());
encoders.put( encoders.put("SHA-1", new org.springframework.security.crypto.password.MessageDigestPasswordEncoder("SHA-1"));
"noop", org.springframework.security.crypto.password.NoOpPasswordEncoder.getInstance()); encoders.put("SHA-256", new org.springframework.security.crypto.password.MessageDigestPasswordEncoder("SHA-256"));
encoders.put("pbkdf2", new Pbkdf2PasswordEncoder()); encoders.put("sha256", new org.springframework.security.crypto.password.StandardPasswordEncoder());
encoders.put("scrypt", new SCryptPasswordEncoder()); encoders.put("argon2", new Argon2PasswordEncoder());
encoders.put(
"SHA-1",
new org.springframework.security.crypto.password.MessageDigestPasswordEncoder("SHA-1"));
encoders.put(
"SHA-256",
new org.springframework.security.crypto.password.MessageDigestPasswordEncoder("SHA-256"));
encoders.put(
"sha256", new org.springframework.security.crypto.password.StandardPasswordEncoder());
encoders.put("argon2", new Argon2PasswordEncoder());
return new DelegatingPasswordEncoder(encodingId, encoders); return new DelegatingPasswordEncoder(encodingId, encoders);
} }
@Bean @Bean
public UserDetailsPasswordService userDetailsPasswordService() { public UserDetailsPasswordService userDetailsPasswordService() {
return new JdbcUserDetailPasswordService(userRepository, userDetailsMapper); return new JdbcUserDetailPasswordService(userRepository, userDetailsMapper);
} }
public UserDetailsService userDetailsService() { public UserDetailsService userDetailsService() {
return new JdbcUserDetailsService(userRepository, userDetailsMapper); return new JdbcUserDetailsService(userRepository, userDetailsMapper);
} }
@Bean @Bean
public DaoAuthenticationProvider daoAuthenticationProvider() { public DaoAuthenticationProvider daoAuthenticationProvider() {
DaoAuthenticationProvider daoAuthenticationProvider = new DaoAuthenticationProvider(); DaoAuthenticationProvider daoAuthenticationProvider = new DaoAuthenticationProvider();
daoAuthenticationProvider.setPasswordEncoder(passwordEncoder()); daoAuthenticationProvider.setPasswordEncoder(passwordEncoder());
daoAuthenticationProvider.setUserDetailsPasswordService(userDetailsPasswordService()); daoAuthenticationProvider.setUserDetailsPasswordService(userDetailsPasswordService());
daoAuthenticationProvider.setUserDetailsService(userDetailsService()); daoAuthenticationProvider.setUserDetailsService(userDetailsService());
return daoAuthenticationProvider; return daoAuthenticationProvider;
} }
} }

View File

@@ -4,15 +4,15 @@ import org.springframework.security.crypto.argon2.Argon2PasswordEncoder;
public class Argon2Example { public class Argon2Example {
public String encode(String plainPassword) { public String encode(String plainPassword) {
int saltLength = 16; // salt length in bytes int saltLength = 16; // salt length in bytes
int hashLength = 32; // hash length in bytes int hashLength = 32; // hash length in bytes
int parallelism = 1; // currently is not supported int parallelism = 1; // currently is not supported
int memory = 4096; // memory costs int memory = 4096; // memory costs
int iterations = 3; int iterations = 3;
Argon2PasswordEncoder argon2PasswordEncoder = Argon2PasswordEncoder argon2PasswordEncoder =
new Argon2PasswordEncoder(saltLength, hashLength, parallelism, memory, iterations); new Argon2PasswordEncoder(saltLength, hashLength, parallelism, memory, iterations);
return argon2PasswordEncoder.encode(plainPassword); return argon2PasswordEncoder.encode(plainPassword);
} }
} }

View File

@@ -6,10 +6,10 @@ import java.security.SecureRandom;
public class BCryptExample { public class BCryptExample {
public String encode(String plainPassword) { public String encode(String plainPassword) {
int strength = 10; int strength = 10;
BCryptPasswordEncoder bCryptPasswordEncoder = BCryptPasswordEncoder bCryptPasswordEncoder =
new BCryptPasswordEncoder(strength, new SecureRandom()); new BCryptPasswordEncoder(strength, new SecureRandom());
return bCryptPasswordEncoder.encode(plainPassword); return bCryptPasswordEncoder.encode(plainPassword);
} }
} }

View File

@@ -4,14 +4,14 @@ import org.springframework.security.crypto.password.Pbkdf2PasswordEncoder;
public class Pbkdf2Example { public class Pbkdf2Example {
public String encode(String plainPassword) { public String encode(String plainPassword) {
String pepper = "pepper"; // secret key used by password encoding String pepper = "pepper"; // secret key used by password encoding
int iterations = 200000; // number of hash iteration int iterations = 200000; // number of hash iteration
int hashWidth = 256; // hash with in bits int hashWidth = 256; // hash with in bits
Pbkdf2PasswordEncoder pbkdf2PasswordEncoder = Pbkdf2PasswordEncoder pbkdf2PasswordEncoder =
new Pbkdf2PasswordEncoder(pepper, iterations, hashWidth); new Pbkdf2PasswordEncoder(pepper, iterations, hashWidth);
return pbkdf2PasswordEncoder.encode(plainPassword); return pbkdf2PasswordEncoder.encode(plainPassword);
} }
} }

View File

@@ -4,15 +4,15 @@ import org.springframework.security.crypto.scrypt.SCryptPasswordEncoder;
public class SCryptExample { public class SCryptExample {
public String encode(String plainPassword) { public String encode(String plainPassword) {
int cpuCost = (int) Math.pow(2, 14); // factor to increase CPU costs int cpuCost = (int) Math.pow(2, 14); // factor to increase CPU costs
int memoryCost = 8; // factor to increases memory usage int memoryCost = 8; // factor to increases memory usage
int parallelization = 1; // currently nor supported by Spring Security int parallelization = 1; // currently nor supported by Spring Security
int keyLength = 32; // key length in bytes int keyLength = 32; // key length in bytes
int saltLength = 64; // salt length in bytes int saltLength = 64; // salt length in bytes
SCryptPasswordEncoder sCryptPasswordEncoder = SCryptPasswordEncoder sCryptPasswordEncoder =
new SCryptPasswordEncoder(cpuCost, memoryCost, parallelization, keyLength, saltLength); new SCryptPasswordEncoder(cpuCost, memoryCost, parallelization, keyLength, saltLength);
return sCryptPasswordEncoder.encode(plainPassword); return sCryptPasswordEncoder.encode(plainPassword);
} }
} }

View File

@@ -13,19 +13,19 @@ import org.springframework.stereotype.Component;
@Component @Component
public class PasswordMigration { public class PasswordMigration {
@Bean @Bean
public ApplicationListener<AuthenticationSuccessEvent> authenticationSuccessListener( public ApplicationListener<AuthenticationSuccessEvent> authenticationSuccessListener(
PasswordEncoder encoder, UserDetailsPasswordService userDetailsPasswordService) { PasswordEncoder encoder, UserDetailsPasswordService userDetailsPasswordService) {
return (AuthenticationSuccessEvent event) -> { return (AuthenticationSuccessEvent event) -> {
Authentication authentication = event.getAuthentication(); Authentication authentication = event.getAuthentication();
User user = (User) authentication.getPrincipal(); User user = (User) authentication.getPrincipal();
String encodedPassword = user.getPassword(); String encodedPassword = user.getPassword();
if (encodedPassword.startsWith("{SHA-1}")) { if (encodedPassword.startsWith("{SHA-1}")) {
CharSequence clearTextPassword = (CharSequence) authentication.getCredentials(); CharSequence clearTextPassword = (CharSequence) authentication.getCredentials();
String newPassword = encoder.encode(clearTextPassword); String newPassword = encoder.encode(clearTextPassword);
userDetailsPasswordService.updatePassword(user, newPassword); userDetailsPasswordService.updatePassword(user, newPassword);
} }
((UsernamePasswordAuthenticationToken) authentication).eraseCredentials(); ((UsernamePasswordAuthenticationToken) authentication).eraseCredentials();
}; };
} }
} }

View File

@@ -11,6 +11,6 @@ import lombok.NoArgsConstructor;
@AllArgsConstructor @AllArgsConstructor
public class Car { public class Car {
private String name; private String name;
private String color; private String color;
} }

View File

@@ -8,11 +8,11 @@ import java.util.Set;
@RestController @RestController
public class CarResources { public class CarResources {
// we use this endpoint as authentication test // we use this endpoint as authentication test
@GetMapping("/cars") @GetMapping("/cars")
public Set<Car> cars() { public Set<Car> cars() {
return Set.of( return Set.of(
Car.builder().name("vw").color("black").build(), Car.builder().name("vw").color("black").build(),
Car.builder().name("bmw").color("white").build()); Car.builder().name("bmw").color("white").build());
} }
} }

View File

@@ -11,6 +11,6 @@ import lombok.NoArgsConstructor;
@NoArgsConstructor @NoArgsConstructor
public class UserCredentialsDto { public class UserCredentialsDto {
private String username; private String username;
private String password; private String password;
} }

View File

@@ -16,24 +16,24 @@ import java.util.Set;
@Transactional @Transactional
public class UserResources { public class UserResources {
private final UserRepository userRepository; private final UserRepository userRepository;
private final PasswordEncoder passwordEncoder; private final PasswordEncoder passwordEncoder;
public UserResources(UserRepository userRepository, PasswordEncoder passwordEncoder) { public UserResources(UserRepository userRepository, PasswordEncoder passwordEncoder) {
this.userRepository = userRepository; this.userRepository = userRepository;
this.passwordEncoder = passwordEncoder; this.passwordEncoder = passwordEncoder;
} }
@PostMapping("/registration") @PostMapping("/registration")
@ResponseStatus(code = HttpStatus.CREATED) @ResponseStatus(code = HttpStatus.CREATED)
public void register(@RequestBody UserCredentialsDto userCredentialsDto) { public void register(@RequestBody UserCredentialsDto userCredentialsDto) {
UserCredentials user = UserCredentials user =
UserCredentials.builder() UserCredentials.builder()
.enabled(true) .enabled(true)
.username(userCredentialsDto.getUsername()) .username(userCredentialsDto.getUsername())
.password(passwordEncoder.encode(userCredentialsDto.getPassword())) .password(passwordEncoder.encode(userCredentialsDto.getPassword()))
.roles(Set.of("USER")) .roles(Set.of("USER"))
.build(); .build();
userRepository.save(user); userRepository.save(user);
} }
} }

View File

@@ -9,93 +9,93 @@ import java.util.concurrent.TimeUnit;
@Component @Component
public class BcCryptWorkFactorService { public class BcCryptWorkFactorService {
private static final String TEST_PASSWORD = "my password"; private static final String TEST_PASSWORD = "my password";
private static final int GOAL_MILLISECONDS_PER_PASSWORD = 1000; private static final int GOAL_MILLISECONDS_PER_PASSWORD = 1000;
private static final int MIN_STRENGTH = 4; private static final int MIN_STRENGTH = 4;
private static final int MAX_STRENGTH = 31; private static final int MAX_STRENGTH = 31;
/** /**
* Calculates the strength (a.k.a. log rounds) for the BCrypt Algorithm, so that password encoding * Calculates the strength (a.k.a. log rounds) for the BCrypt Algorithm, so that password encoding
* takes about 1s. This method uses the divide-and-conquer algorithm. * takes about 1s. This method uses the divide-and-conquer algorithm.
*/ */
public BcryptWorkFactor calculateStrengthDivideAndConquer() { public BcryptWorkFactor calculateStrengthDivideAndConquer() {
return calculateStrengthDivideAndConquer( return calculateStrengthDivideAndConquer(
new BcryptWorkFactor(MIN_STRENGTH, Integer.MIN_VALUE), new BcryptWorkFactor(MIN_STRENGTH, Integer.MIN_VALUE),
new BcryptWorkFactor(MAX_STRENGTH, Integer.MAX_VALUE)); new BcryptWorkFactor(MAX_STRENGTH, Integer.MAX_VALUE));
}
private BcryptWorkFactor calculateStrengthDivideAndConquer(
BcryptWorkFactor smallFactor, BcryptWorkFactor bigFactor) {
if (bigFactor.getStrength() - smallFactor.getStrength() == 1) {
return getClosestStrength(smallFactor, bigFactor);
} }
int midStrength =
(bigFactor.getStrength() - smallFactor.getStrength()) / 2 + smallFactor.getStrength(); private BcryptWorkFactor calculateStrengthDivideAndConquer(
long duration = calculateDuration(midStrength); BcryptWorkFactor smallFactor, BcryptWorkFactor bigFactor) {
BcryptWorkFactor midFactor = new BcryptWorkFactor(midStrength, duration); if (bigFactor.getStrength() - smallFactor.getStrength() == 1) {
if (duration < GOAL_MILLISECONDS_PER_PASSWORD) { return getClosestStrength(smallFactor, bigFactor);
return calculateStrengthDivideAndConquer(midFactor, bigFactor); }
int midStrength =
(bigFactor.getStrength() - smallFactor.getStrength()) / 2 + smallFactor.getStrength();
long duration = calculateDuration(midStrength);
BcryptWorkFactor midFactor = new BcryptWorkFactor(midStrength, duration);
if (duration < GOAL_MILLISECONDS_PER_PASSWORD) {
return calculateStrengthDivideAndConquer(midFactor, bigFactor);
}
return calculateStrengthDivideAndConquer(smallFactor, midFactor);
} }
return calculateStrengthDivideAndConquer(smallFactor, midFactor);
}
private BcryptWorkFactor getClosestStrength( private BcryptWorkFactor getClosestStrength(
BcryptWorkFactor smallFactor, BcryptWorkFactor bigFactor) { BcryptWorkFactor smallFactor, BcryptWorkFactor bigFactor) {
if (isPreviousDurationCloserToGoal(smallFactor.getDuration(), bigFactor.getDuration())) { if (isPreviousDurationCloserToGoal(smallFactor.getDuration(), bigFactor.getDuration())) {
return smallFactor; return smallFactor;
}
return bigFactor;
} }
return bigFactor;
}
private long calculateDuration(int strength) { private long calculateDuration(int strength) {
BCryptPasswordEncoder bCryptPasswordEncoder = new BCryptPasswordEncoder(strength); BCryptPasswordEncoder bCryptPasswordEncoder = new BCryptPasswordEncoder(strength);
Stopwatch stopwatch = Stopwatch.createStarted(); Stopwatch stopwatch = Stopwatch.createStarted();
bCryptPasswordEncoder.encode(TEST_PASSWORD); bCryptPasswordEncoder.encode(TEST_PASSWORD);
stopwatch.stop(); stopwatch.stop();
return stopwatch.elapsed(TimeUnit.MILLISECONDS); return stopwatch.elapsed(TimeUnit.MILLISECONDS);
}
/**
* Calculates the strength (a.k.a. log rounds) for the BCrypt Algorithm, so that password encoding
* takes about 1s. This method iterates over strength from 4 to 31 and calculates the duration of
* password encoding for every value of strength. It returns the first strength, that takes more
* than 1s
*/
public int calculateStrength() {
for (int strength = MIN_STRENGTH; strength <= MAX_STRENGTH; strength++) {
long duration = calculateDuration(strength);
if (duration >= GOAL_MILLISECONDS_PER_PASSWORD) {
return strength;
}
} }
throw new RuntimeException(
String.format(
"Could not find suitable round number for bcrypt encoding. The encoding with %d rounds"
+ " takes less than %d ms.",
MAX_STRENGTH, GOAL_MILLISECONDS_PER_PASSWORD));
}
/** /**
* @param previousDuration duration from previous iteration * Calculates the strength (a.k.a. log rounds) for the BCrypt Algorithm, so that password encoding
* @param currentDuration duration of current iteration * takes about 1s. This method iterates over strength from 4 to 31 and calculates the duration of
* @param strength current strength * password encoding for every value of strength. It returns the first strength, that takes more
* @return return the current strength, if current duration is closer to * than 1s
* GOAL_MILLISECONDS_PER_PASSWORD, otherwise current strength-1. */
*/ public int calculateStrength() {
int getStrength(long previousDuration, long currentDuration, int strength) { for (int strength = MIN_STRENGTH; strength <= MAX_STRENGTH; strength++) {
if (isPreviousDurationCloserToGoal(previousDuration, currentDuration)) {
return strength - 1; long duration = calculateDuration(strength);
} else { if (duration >= GOAL_MILLISECONDS_PER_PASSWORD) {
return strength; return strength;
}
}
throw new RuntimeException(
String.format(
"Could not find suitable round number for bcrypt encoding. The encoding with %d rounds"
+ " takes less than %d ms.",
MAX_STRENGTH, GOAL_MILLISECONDS_PER_PASSWORD));
} }
}
/** /**
* return true, if previousDuration is closer to the goal than currentDuration, false otherwise. * @param previousDuration duration from previous iteration
*/ * @param currentDuration duration of current iteration
boolean isPreviousDurationCloserToGoal(long previousDuration, long currentDuration) { * @param strength current strength
return Math.abs(GOAL_MILLISECONDS_PER_PASSWORD - previousDuration) * @return return the current strength, if current duration is closer to
< Math.abs(GOAL_MILLISECONDS_PER_PASSWORD - currentDuration); * GOAL_MILLISECONDS_PER_PASSWORD, otherwise current strength-1.
} */
int getStrength(long previousDuration, long currentDuration, int strength) {
if (isPreviousDurationCloserToGoal(previousDuration, currentDuration)) {
return strength - 1;
} else {
return strength;
}
}
/**
* return true, if previousDuration is closer to the goal than currentDuration, false otherwise.
*/
boolean isPreviousDurationCloserToGoal(long previousDuration, long currentDuration) {
return Math.abs(GOAL_MILLISECONDS_PER_PASSWORD - previousDuration)
< Math.abs(GOAL_MILLISECONDS_PER_PASSWORD - currentDuration);
}
} }

View File

@@ -7,6 +7,6 @@ import lombok.Data;
@AllArgsConstructor @AllArgsConstructor
public class BcryptWorkFactor { public class BcryptWorkFactor {
private int strength; private int strength;
private long duration; private long duration;
} }

View File

@@ -9,33 +9,33 @@ import java.util.concurrent.TimeUnit;
@Component @Component
public class Pbkdf2WorkFactorService { public class Pbkdf2WorkFactorService {
private static final String TEST_PASSWORD = "my password"; private static final String TEST_PASSWORD = "my password";
private static final String NO_ADDITIONAL_SECRET = ""; private static final String NO_ADDITIONAL_SECRET = "";
private static final int GOAL_MILLISECONDS_PER_PASSWORD = 1000; private static final int GOAL_MILLISECONDS_PER_PASSWORD = 1000;
private static final int HASH_WIDTH = 256; private static final int HASH_WIDTH = 256;
private static final int ITERATION_STEP = 5000; private static final int ITERATION_STEP = 5000;
/** /**
* Finds the number of Iteration for the {@link Pbkdf2PasswordEncoder} to get the duration of * Finds the number of Iteration for the {@link Pbkdf2PasswordEncoder} to get the duration of
* password encoding close to 1s. The Calculation does not use any secret (pepper) and applies * password encoding close to 1s. The Calculation does not use any secret (pepper) and applies
* hash algorithm SHA256. * hash algorithm SHA256.
*/ */
public int calculateIteration() { public int calculateIteration() {
int iterationNumber = 150000; int iterationNumber = 150000;
while (true) { while (true) {
Pbkdf2PasswordEncoder pbkdf2PasswordEncoder = Pbkdf2PasswordEncoder pbkdf2PasswordEncoder =
new Pbkdf2PasswordEncoder(NO_ADDITIONAL_SECRET, iterationNumber, HASH_WIDTH); new Pbkdf2PasswordEncoder(NO_ADDITIONAL_SECRET, iterationNumber, HASH_WIDTH);
Stopwatch stopwatch = Stopwatch.createStarted(); Stopwatch stopwatch = Stopwatch.createStarted();
pbkdf2PasswordEncoder.encode(TEST_PASSWORD); pbkdf2PasswordEncoder.encode(TEST_PASSWORD);
stopwatch.stop(); stopwatch.stop();
long duration = stopwatch.elapsed(TimeUnit.MILLISECONDS); long duration = stopwatch.elapsed(TimeUnit.MILLISECONDS);
if (duration > GOAL_MILLISECONDS_PER_PASSWORD) { if (duration > GOAL_MILLISECONDS_PER_PASSWORD) {
return iterationNumber; return iterationNumber;
} }
iterationNumber += ITERATION_STEP; iterationNumber += ITERATION_STEP;
}
} }
}
} }