blibli-future / detroit

Project Detroit
5 stars 0 forks source link

Hash password #28

Closed adhikasp closed 7 years ago

adhikasp commented 7 years ago

Never ever store a plaintext password.

Some reference:

adhikasp commented 7 years ago

If we implement this, it's a good idea to integrate the hashing method inside of user.setPassword() method too.

DitoRaharjo commented 7 years ago

okay, I already know bcrypt from laravel, but in laravel when I had to authenticated user's password from login and database I use hashCheck(), looks like this

if (Hash::check($pengguna_data['password_lama'], $pengguna->password) ) {
...
}`

is it the same thing with this example with spring? (I read it from your url reference : http://www.baeldung.com/spring-security-registration-password-encoding-bcrypt)

public class SecSecurityConfig extends WebSecurityConfigurerAdapter {

    @Override
    protected void configure(AuthenticationManagerBuilder auth) throws Exception {
        auth.authenticationProvider(authProvider());
    }

    ...
}

@adhikasp

riencus commented 7 years ago

Just asking, don't we (blibli-future) need to save user's data using LDAP? Have any of you done any research on using it @DitoRaharjo @adhikasp ?

DitoRaharjo commented 7 years ago

@arielchristianto We still confused about how to integrate LDAP with our project, so we use Basic Auth instead 💃