Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incidencia - Emails - Evitar que se autocomplete la contraseña #85

Merged
merged 3 commits into from
Feb 19, 2024

Conversation

ManuSinergiaCRM
Copy link
Collaborator

@ManuSinergiaCRM ManuSinergiaCRM commented Jan 25, 2024

SuiteCRM PR: salesagility#10497


Se detecta, tanto en la vista de Configuración General de Email como en la vista de edición del módulo de Cuentas de correo salientes, que si el navegador tiene almacenada la contraseña, el autocompletado de este campo se realiza ya que a pesar de que aparece el enlace de Cambiar Contraseña y parece que el input de tipo password no forma parte del formulario, este si se encuentra pero con la propiedad display: none.

Si la contraseña guardada en el navegador está desactualizada o es errónea, al realizarse el autocompletado de la contraseña genera que se produzca un error de autenticación en el envío del correo de prueba aunque visiblemente se vea el enlace de Cambiar contraseña.

Este PR resuelve esta incidencia haciendo que no se realice ese autocompletado a través de añadir la propiedad autocomplete='new-password' al campo de tipo password.

Esta incidencia también se reproduce en SuiteCRM aunque hemos comprobado que en la versión 7.14.2, donde SA ha realizado cambios profundos en los módulos de Emails, la incidencia del módulo de Cuentas de correo salientes no se soluciona como en este PR y se puede solucionar de la siguiente manera:

  • En el fichero: modules/OutboundEmailAccounts/vardefs.php --> Añadir la siguiente propiedad al campo mail_smtppass en la línea 286: 'autocomplete' => 'new-password',

  • En el fichero: include/SugarFields/Fields/Password/EditView.tpl --> Añadir la siguiente propiedad en la línea 51: autocomplete='{{$vardef.autocomplete}}'

Pruebas

  1. Acceder a la Vista de Configuración General de Email y guardar una primera configuración con la contraseña correcta + almacenar la contraseña en el navegador (Tras guardar el registro suele mostrarse un cuadro de dialogo para almacenar el usuario y la contraseña en el navegador)
  2. Salir del registro y volver a acceder. Pulsar en el botón de Cambiar contraseña y comprobar que no se recupera la contraseña aunque esta esté almacenada en el navegador.
  3. Ir al módulo de Cuentas de correo salientes y acceder a la vista de edición del registros de system. Pulsar en el botón de Cambiar contraseña y comprobar que tampoco se recupera la contraseña aunque esta esté almacenada en el navegador.
  4. Salir del registro
  5. Acceder al gestor de contraseñas del navegador y modificar la contraseña a una errónea.
  6. Acceder a ambas vistas de edición y, con el input de contraseña "oculto" y mostrándose el texto de "Cambiar Contraseña", comprobar que el envío de prueba se realiza correctamente ya que debe recuperar la contraseña de la base de datos.

@ManuSinergiaCRM ManuSinergiaCRM added bug Something isn't working Correo Email Module not urgent labels Jan 25, 2024
@ManuSinergiaCRM ManuSinergiaCRM self-assigned this Jan 25, 2024
Copy link

github-actions bot commented Jan 25, 2024

Actions executed at: 2024-02-19 16:48:13.

@enricsinergia
Copy link
Collaborator

¿Por qué no se aplica la solución de SA? Por lo que comentas es prácticamente la misma pero aplicada de distinta forma

@ManuSinergiaCRM
Copy link
Collaborator Author

ManuSinergiaCRM commented Jan 31, 2024

¿Por qué no se aplica la solución de SA? Por lo que comentas es prácticamente la misma pero aplicada de distinta forma

Es debido a que en la versión 7.12.12 el campo mail_smtppass es de tipo varchar mientras que en la 7.14.2 es de tipo password. Además, me dijo Alberto que SuiteCRM ha realizado más cambios en áreas relacionadas con la gestión de emails para la versión 7.14.2.

En consecuencia, como ya teníamos solución para la versión actual y no sé que generará cambiar el tipo de este campo... Se dejó el integrar la modificación propuesta para la versión 7.14.2 para cuando nos actualicemos a ella, si es que SuiteCRM no lo ha corregido antes ya que sí les vamos a crear un PR con la solución para dicha versión.

Copy link
Collaborator

@enricsinergia enricsinergia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(A)provado

Probado con Chrome y Firefox en Linux

@AlbertoSTIC AlbertoSTIC requested review from ainaraRT and removed request for AlbertoSTIC February 5, 2024 16:28
Copy link
Collaborator

@ainaraRT ainaraRT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(A)Probado

@AlbertoSTIC AlbertoSTIC merged commit 39c9c8d into develop Feb 19, 2024
1 of 3 checks passed
@AlbertoSTIC AlbertoSTIC deleted the bug/preventPasswordAutofill branch February 19, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Correo Email Module not urgent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants