CIDWA/pcpuma_unam_api
13
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

SEC-001: migrar hCaptcha a Cloudflare Turnstile en API (prod) #3

Merged
edreitega merged 3 commits from dev into master 2026-06-23 18:58:13 +00:00
Conversation 0 Commits 3 Files Changed 6
+86 -55
edreitega commented 2026-06-23 18:57:56 +00:00
Collaborator
Copy Link
Copy Source

Summary

Promoción a master del cambio SEC-001 en la API: reemplazo de HcaptchaGuard por TurnstileGuard con verificación vía Cloudflare siteverify.

Cambios

  • Nuevo src/guards/turnstile.guard.ts (header HTTP turnstile).
  • TurnstileGuard en login admin, operador, usuario y registro.
  • Eliminados hcaptcha.guard.ts y dependencia hcaptcha.

Validación en pruebas (PMODUL1)

  • Login usuario, operador y admin OK
  • Registro OK
  • Widget Turnstile visible
  • Header turnstile en requests
  • STATE=pruebas con siteverify activo

Variables de entorno en producción (antes del deploy)

  • Añadir: TURNSTILE_SECRET_KEY
  • Eliminar: HCAPTCHA_KEY
  • Verificar: STATE=produccion

Deploy coordinado

No desplegar solo la API. Debe subir junto con pcpuma_unam_usuario y pcpuma_unam_operador en la misma ventana.

Test plan post-deploy

  • Login admin / operador / usuario en modulospcpuma.unam.mx
  • Registro usuario
  • Token inválido → 403
  • pm2 reload api-prod sin errores en logs
## Summary Promoción a `master` del cambio **SEC-001** en la API: reemplazo de `HcaptchaGuard` por `TurnstileGuard` con verificación vía Cloudflare siteverify. ## Cambios - Nuevo `src/guards/turnstile.guard.ts` (header HTTP `turnstile`). - `TurnstileGuard` en login admin, operador, usuario y registro. - Eliminados `hcaptcha.guard.ts` y dependencia `hcaptcha`. ## Validación en pruebas (PMODUL1) - [x] Login usuario, operador y admin OK - [x] Registro OK - [x] Widget Turnstile visible - [x] Header `turnstile` en requests - [x] `STATE=pruebas` con siteverify activo ## Variables de entorno en producción (antes del deploy) - Añadir: `TURNSTILE_SECRET_KEY` - Eliminar: `HCAPTCHA_KEY` - Verificar: `STATE=produccion` ## Deploy coordinado **No desplegar solo la API.** Debe subir junto con `pcpuma_unam_usuario` y `pcpuma_unam_operador` en la misma ventana. ## Test plan post-deploy - [ ] Login admin / operador / usuario en `modulospcpuma.unam.mx` - [ ] Registro usuario - [ ] Token inválido → 403 - [ ] `pm2 reload api-prod` sin errores en logs
edreitega merged commit 4660214cd7 into master 2026-06-23 18:58:13 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
Arturo LeydiMoon (Leydi Monserrat Jiménez Pineda) Saavedra29 (Saavedra Trejo) diegocg (CANTERO GARCIA DIEGO) donovan (JIMENEZ ROSAS DONOVAN JHOSETE) dscadmin edreitega (Jair Edrei Téllez García) eithan (Eithan Hernandez) evenegas (Venegas Reyes Emilio) gabrielagh (Gabriela Gómez Hernández) jalvarado (Jorge Miguel Alvarado Reyes) mcervantesag (Manuel Antonio Cervantes Aguirre) omar.monroy (Omar Ulises Monroy Alarcon) puma99 (Lemuel Helon Marquez Rosas) smoreno (Santiago Moreno Lozano)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: CIDWA/pcpuma_unam_api#3
Delete Branch "dev"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?