Skip to content

REC-208: Vencimiento de sesión#87

Open
agustin1996ra wants to merge 1 commit into
masterfrom
REC-208
Open

REC-208: Vencimiento de sesión#87
agustin1996ra wants to merge 1 commit into
masterfrom
REC-208

Conversation

@agustin1996ra
Copy link
Copy Markdown

@agustin1996ra agustin1996ra commented Mar 5, 2026

Requerimiento

https://proyectos.andes.gob.ar/browse/REC-208

Funcionalidad desarrollada

  1. Cambio de asignación de fecha de expiración de token de sesión
  2. Quitar env del archivo config, solo dependemos del archivo .env
  3. Forzar expiracion de sesion para lo token con fecha de creación en el futuro.

UserStories llegó a completarse

  • Si
  • No

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en el entorno de Producción

TOKEN_LIFETIME = "12"

Warning

En el archivo env tambien agregar la variable
JWT_SECRET = "..."

andes/recetar-app#134

@agustin1996ra agustin1996ra force-pushed the REC-208 branch 2 times, most recently from 8da6962 to 34526dc Compare March 5, 2026 18:01
@github-actions github-actions Bot added the Andes label Mar 17, 2026
@MCele MCele removed the Andes label Mar 25, 2026
@MCele
Copy link
Copy Markdown

MCele commented Mar 25, 2026

Agus lo probé y me quedó una duda, lo planteo por acá, pero si no charlenlo con los chicos, quizás lo probé mal. En mi .env la variable TOKEN_LIFETIME = "1". Una vez que abro la y empiezo a navegar veo el token expira luego de una hora (exp), pero si pasado ese tiempo vuelvo a abrir el navegador (sin loguearme nuevamente) se actualiza ese token con una hora más. Eso es correcto?
No entiendo si deberia cerrarse sesión en ese caso o el cierre de sesión es siempre despues de las x horas definidas, de todos modos en ese caso no se con qué compararía porque ambas fechas que contiene el token se actualizaron.

Otra cosita a tener en cuenta cuando se pase a prod es la variable que apunta a la base, estaria bueno chequeuar que existe en el entorno. Dicho sea de pase me parece que estaría bueno unificarlas, porque hay dos variables y no le encuentro sentido en nuestro caso. Igual esto ultimo puede ser en otra tarea, lo comento porque hace tiempo lo noté y vi que ya unificaste en una de ellas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes done Cambios realizados

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants