[BORNIOS]

Resumen

Versión 3.1.0: retención de respaldos, endurecimiento del backend y rediseño de la interfaz.

Retención de respaldos

• Días de conservación y mínimo de recientes (por defecto desactivada, 0 días; los últimos 5 siempre se conservan).
• Solo borra archivos con el nombre que genera el plugin (<base>_aaaammdd_hhmmss.sql[.zip]).
• Se aplica tras cada backup, con la tarea PostgreSQL Backup Retention (diaria 03:00) y con «Depurar respaldos ahora» (vista previa + confirmación).

Endurecimiento del backend

• Consola SQL en transacción READ ONLY con timeouts; rechaza comillas de dólar.
• La caché invalida también las escrituras de SaveChanges y al confirmar transacciones.
• Guardar configuración valida la conexión si PostgreSQL está activo, escribe database.xml de forma atómica (+ .bak) y no devuelve la contraseña en claro.
• Search/Instant ahora exige usuario autenticado (antes anónimo). Cambio de comportamiento: un consumidor externo sin sesión dejará de funcionar.
• min/max(uuid) solo se marcan creados tras un intento exitoso; pg_dump/psql se terminan al cancelar; aviso de copias obsoletas del plugin.

Interfaz

• Mismo diseño que JellyTV, responsive, sin estilos inline de tema.
• Resultados como notificaciones; modal de confirmación homogéneo; pestaña Mantenimiento reordenada.

Pruebas

• dotnet test: 122 correctas, 29 omitidas sin conexión de pruebas.
• Con PostgreSQL local (bases temporales): 145 correctas, 1 omitida (antes de añadir los 5 tests de retención).
• Interfaz revisada con capturas a 1920, 1280 y 390 px. La retención no se probó contra un servidor real ni en Linux/macOS (el código no depende del sistema operativo).

Después de aprobar

Release v3.1.0 con notas y actualización de manifest.json.

[BORNIOS]

CodeQL: alerta #54 (cs/sql-injection) descartada como falso positivo

Dónde: QueryConsoleService.cs, new NpgsqlCommand(statement, connection, transaction).

Por qué se silencia y no se corrige: la consola SQL del panel existe para ejecutar el SQL que escribe un administrador. El texto llega del usuario por diseño y no se puede parametrizar; eliminar el flujo equivaldría a eliminar la función. No es una alerta nueva: es la #48 de main, que seguía abierta. En este PR aparece como nueva solo porque la sentencia cambió de línea.

Mitigaciones que sí están en el código (varias son nuevas en este PR):

• El endpoint exige la política de elevación (solo administradores).
• Una única sentencia por consulta y solo de lectura (SELECT, WITH, TABLE, VALUES, EXPLAIN, SHOW), con lista de palabras bloqueadas sobre el texto sin literales.
• Nuevo: se ejecuta dentro de una transacción READ ONLY con statement_timeout y lock_timeout, de modo que el servidor rechaza cualquier escritura, DDL o avance de secuencia aunque el análisis de texto fallara.
• Nuevo: se rechazan las comillas de dólar ($$), que permitían esconder una segunda sentencia al analizador.
• Límite de filas y de tiempo, y cada ejecución queda en el log del plugin.
• Pruebas de regresión en QueryConsoleTests.

CodeQL no reconoce estas barreras como sanitizadores, por eso se descarta con esta justificación.