HALLAZGO 01Prioridad alta · ejemplo
InyecciónPosible SQL Injection en una búsqueda
app/Reportes.php · línea 84
- Qué observamos
- El código concatena un parámetro de búsqueda dentro de una consulta SQL.
- Impacto posible
- Si el dato llega sin controles suficientes, podría alterar la consulta y permitir acceso a información fuera de lo previsto.
- Recomendación
- Usar consultas parametrizadas, revisar permisos de la cuenta de base de datos y validar las entradas.
- Cómo confirmarlo
- Revisar el recorrido completo del parámetro y probar búsquedas válidas e inválidas en staging antes de confirmar el hallazgo.
HALLAZGO 02Prioridad alta · ejemplo
Control de accesoRevisar permisos antes de exportar
app/Exportaciones.php · línea 42
- Qué observamos
- La función de exportación recibe un identificador de empresa. En el fragmento revisado no se ve la comprobación de pertenencia.
- Impacto posible
- Si el control no existe en otra capa, un usuario podría solicitar una exportación de otra empresa.
- Recomendación
- Comprobar permisos sobre el recurso en el servidor y limitar la exportación a la empresa autorizada.
- Cómo confirmarlo
- Confirmar controles en middleware y probar dos cuentas con permisos diferentes. El fragmento por sí solo no confirma una vulnerabilidad.
HALLAZGO 03Prioridad media · ejemplo
ConfiguraciónDesactivar errores detallados en producción
config/app.php · línea 17
- Qué observamos
- La configuración de ejemplo habilita la presentación de errores detallados.
- Impacto posible
- Si se utiliza en producción, podría revelar rutas internas y detalles de implementación.
- Recomendación
- Separar la configuración por entorno, mostrar un mensaje genérico al usuario y registrar los detalles en un destino con acceso restringido.
- Cómo confirmarlo
- Verificar la configuración efectiva del servidor y comprobar el manejo de errores en staging.