R-428 sivel2_gen. mejora seguridad en Devise #64
Conversation
| @@ -0,0 +1,2 @@ | |||
| <%= f.input :password %> | |||
| <%= f.input :password_confirmation %> | |||
There was a problem hiding this comment.
Favor quitar confirmación de clave. No me parece necesaria para creación de un usuario por parte de un administrador que es este contexto.
| @@ -118,7 +118,8 @@ def lista_params_sip | |||
| :idioma, | |||
| :email, | |||
| :tema_id, | |||
| :encrypted_password, | |||
| :encrypted_password, | |||
| :password_confirmation, | |||
lib/sip/concerns/models/usuario.rb
Outdated
| validates_presence_of :password, :on=>:create | ||
| validates_confirmation_of :password, :on=>:create | ||
| validate :password_complexity | ||
| def password_complexity |
There was a problem hiding this comment.
Nombres de funciones en español
| @@ -81,8 +82,14 @@ def fechadeshabilitacion_posible? | |||
| end | |||
| end | |||
|
|
|||
| validates_presence_of :encrypted_password, :on=>:create | |||
| validates_confirmation_of :encrypted_password, :on=>:create | |||
| validates_presence_of :password, :on=>:create | |||
There was a problem hiding this comment.
No me parece que cambie encrypted_password por password
|
No veo sus pruebas tras ese cambio, por favor probar como afecta sivel2, cor1440 y si_jrscol |
|
Prueba: (probado en sivel2, sijrscol y cor1440). Se quitó lo de confirmación de contraseña. sin embargo se probó que la validación de presencia de password solo funciona con password y no con encrpted passwors. pienso que es por que se realiza la validación antes de encriptarse. igualmente se prueba la complejidad de password. IMG_0054.MOV |
|
Se realizaron las siguientes implementaciones para fortalecer seguridad en la aplicación: