search

7-Seven-Up / megastore-backend

MIT License
1 stars 0 forks source link

Feat/recover password #55

Closed ValeLattanzi closed 1 month ago

ValeLattanzi commented 1 month ago

Changes:

EzeSosa commented 1 month ago

Una cosa más @ValeLattanzi, probaste que esto no te tire un 403 desde el back? Lo digo porque en nuestra configuración de endpoints protegidos (https://github.com/7-Seven-Up/megastore-backend/blob/main/src/main/java/com/_up/megastore/security/utils/Endpoints.java) las URL que creaste entrarían en las que están habilitadas sólo para administradores.

Deberías agregar tus URL en el arreglo de las white listed

ValeLattanzi commented 1 month ago

@EzeSosa lo probé todo levantando el Back y el front, pero no me dió ese error. Ahora me pongo a ver eso que me decís de las URL

Fedesan14 commented 1 month ago

Te dejo esas correcciones que son más que nada detalles, pero me gustó mucho el laburo.

Unas cositas más:

  1. Habría que hacer el envío de los correos asíncronos porque demoran mucho el procesamiento de las request. Podríamos hacerlo en esta PR si todos estamos de acuerdo @Fedesan14 @delgadomatias @martinjcrosetto @NicoAntu
  2. Si tenés un poco más de tiempo para la próxima estaría genial que expliques un poco el flujo de las requests para la recuperación, una descripción cortita

Good work

Si, coincido que el envío de los emails deberían ser asíncronos, pero podemos hacerlo en otro MR