¿Qué buscas en una revisión de código segura?

Respuesta directa

Qué debe cubrir una buena respuesta

Cómo funciona

Una revisión segura sigue a los datos. Cada valor que viene de fuera, parámetros de solicitud, cabeceras, ficheros, mensajes, respuestas de terceros, se rastrea por el código para ver dónde se valida, cómo influye en decisiones de autorización, si llega a una consulta, un comando, una plantilla, un deserializador, una ruta de fichero o una solicitud saliente, y qué pasa si está malformado o es hostil. Junto al rastreo, el revisor comprueba autenticación y manejo de sesiones, que la autorización se aplica en cada objeto y no solo en el endpoint, que secretos y claves se manejan bien, que la criptografía usa bibliotecas contrastadas con parámetros sensatos, que las operaciones concurrentes no pueden adelantar a una comprobación, y que los logs ni filtran datos sensibles ni omiten eventos de seguridad. Los cambios de dependencias reciben su propia mirada.

Ejemplo

Al revisar un endpoint que genera informes PDF, el revisor rastreó el parámetro de nombre de plantilla y lo encontró concatenado en una ruta de fichero, permitiendo traversal a cualquier fichero legible por el servicio. Un segundo hallazgo fue una URL de imagen descargada en el servidor sin lista de permitidos, un SSRF hacia la red interna. Ambos se corrigieron antes de fusionar con una lista de IDs de plantilla y un descargador restringido a direcciones públicas y un conjunto de hosts.

Errores frecuentes

• Revisar solo el diff sin seguir adónde van los datos.
• Buscar inyección pero no autorización ausente sobre IDs de objeto.
• Aprobar una actualización de dependencia sin mirar qué cambió.

Una estructura útil para responder

Responde desde tu propia experiencia.

No memorices un guion: usa solo ejemplos que puedas explicar con honestidad en preguntas posteriores.