• Joined on 2026-08-03
ai-bot commented on pull request jpbeldomenico/taller_automotriz#8 2026-08-05 19:15:08 -03:00
fix(whatsapp): pasar las credenciales de Meta al contenedor

Qué cambia
Se agregan al docker-compose.yml las cuatro variables de entorno necesarias para el bot de WhatsApp (WHATSAPP_PHONE_NUMBER_ID, WHATSAPP_TOKEN, WHATSAPP_VERIFY_TOKEN,…

ai-bot commented on pull request jpbeldomenico/taller_automotriz#7 2026-08-05 16:47:28 -03:00
fix(auth): comprobar el rol de la sesion en todas las rutas

Qué cambia
Se agrega validación de rol en la verificación de sesión: verificarSesion y verifyCookie ahora reciben el rol esperado (`"admin"

ai-bot commented on pull request jpbeldomenico/taller_automotriz#7 2026-08-05 16:47:07 -03:00
fix(auth): comprobar el rol de la sesion en todas las rutas

Inline review

ai-bot commented on pull request jpbeldomenico/taller_automotriz#7 2026-08-05 16:47:07 -03:00
fix(auth): comprobar el rol de la sesion en todas las rutas

Inline review

ai-bot commented on pull request jpbeldomenico/taller_automotriz#7 2026-08-05 16:47:07 -03:00
fix(auth): comprobar el rol de la sesion en todas las rutas

Mismo caso: se quitó el check de rol explícito. Si verificarSesion no valida el rol, esto abre acceso indebido. Verificá la implementación en auth-admin.ts.

ai-bot commented on pull request jpbeldomenico/taller_automotriz#7 2026-08-05 16:47:07 -03:00
fix(auth): comprobar el rol de la sesion en todas las rutas

Inline review

ai-bot commented on pull request jpbeldomenico/taller_automotriz#7 2026-08-05 16:47:07 -03:00
fix(auth): comprobar el rol de la sesion en todas las rutas

Inline review

ai-bot commented on pull request jpbeldomenico/taller_automotriz#7 2026-08-05 16:47:07 -03:00
fix(auth): comprobar el rol de la sesion en todas las rutas

Inline review

ai-bot commented on pull request jpbeldomenico/taller_automotriz#7 2026-08-05 16:47:07 -03:00
fix(auth): comprobar el rol de la sesion en todas las rutas

El cast (payload as { role?: string }) asume que el JWT siempre tiene un campo role. ¿Confirmaste que los tokens generados al loguear (admin, jefe, cliente) incluyen ese claim? Si no, la validación siempre fallará y nadie podrá acceder.

ai-bot commented on pull request jpbeldomenico/taller_automotriz#6 2026-08-05 15:58:09 -03:00
feat: el cliente puede ver y modificar sus datos de contacto

Qué cambia
Se agrega la funcionalidad completa para que el cliente logueado vea y edite su perfil (nombre, apellido, email, teléfono). Incluye: endpoint GET/PATCH /api/cliente/perfil con…

ai-bot commented on pull request jpbeldomenico/taller_automotriz#6 2026-08-05 15:57:24 -03:00
feat: el cliente puede ver y modificar sus datos de contacto

Inline review

ai-bot commented on pull request jpbeldomenico/taller_automotriz#6 2026-08-05 15:57:24 -03:00
feat: el cliente puede ver y modificar sus datos de contacto

Inline review

ai-bot commented on pull request jpbeldomenico/taller_automotriz#6 2026-08-05 15:57:24 -03:00
feat: el cliente puede ver y modificar sus datos de contacto

Importás verificarSesion desde auth-admin.ts para validar la cookie cliente_session. Verificá que esa función valide correctamente el rol de cliente y no solo la existencia de la sesión de admin. Si reutiliza lógica de admin, podría haber escalada de privilegios.

ai-bot commented on pull request jpbeldomenico/taller_automotriz#6 2026-08-05 15:57:24 -03:00
feat: el cliente puede ver y modificar sus datos de contacto

Inline review

ai-bot commented on pull request jpbeldomenico/taller_automotriz#6 2026-08-05 15:57:24 -03:00
feat: el cliente puede ver y modificar sus datos de contacto

Inline review

ai-bot commented on pull request jpbeldomenico/taller_automotriz#6 2026-08-05 15:57:24 -03:00
feat: el cliente puede ver y modificar sus datos de contacto

Inline review

ai-bot commented on pull request jpbeldomenico/taller_automotriz#6 2026-08-05 15:57:24 -03:00
feat: el cliente puede ver y modificar sus datos de contacto

Guardás el teléfono en formato local en el campo telefono y en formato WhatsApp en telefonoNorm. El GET lee telefonoNorm primero y hace fallback a telefono. Si telefonoNorm existe, telefono es redundante. Confirmá que el schema tenga @@unique([telefonoNorm]) y @@unique([email]) para que la BD sea la fuente de verdad.

ai-bot commented on pull request jpbeldomenico/taller_automotriz#6 2026-08-05 15:57:24 -03:00
feat: el cliente puede ver y modificar sus datos de contacto

Condición de carrera: la verificación de unicidad de email/teléfono y el update no son atómicos. Dos requests simultáneos pueden pasar la verificación y fallar en el update (o peor, si no hay constraint única en BD, duplicar datos). Usá una transacción de Prisma o confia en el constraint único y atrapá el error P2002.

ai-bot commented on pull request jpbeldomenico/taller_automotriz#5 2026-08-05 14:59:58 -03:00
fix: guardar el telefono en el formato que usa WhatsApp

Qué cambia
Se agrega validación y normalización de teléfonos argentinos en el registro de clientes: el frontend muestra prefijo fijo "+54 9" y acepta solo 10 dígitos (característica sin 0,…

ai-bot commented on pull request jpbeldomenico/taller_automotriz#5 2026-08-05 14:59:34 -03:00
fix: guardar el telefono en el formato que usa WhatsApp

Inline review