mirror of
https://origin.cursor.com/mrdevmx/panels.git
synced 2026-10-09 15:13:17 +00:00
Fase 2 (driver): - api/pg.ts: adaptador delgado sobre postgres.js (prepare/get/all/run, placeholders ? -> $n, withTenant con set_config para RLS), con parsers de tipo custom (numeric/date/timestamp(tz)/bigint) para que el resto del codigo heredado de SQLite (fechas/montos como string, ids como number) siga funcionando sin reescribir cada call-site a mano. - api/platform_db.ts, api/iam_db.ts (nuevo), api/db.ts: pools separados por base/esquema (panels_platform, panels_product.iam, panels_product.core), owner pool para bootstrap/scripts/lookups administrativos que cruzan tenant a proposito. - api/redis.ts: clientes iam/core separados (ACL panels_iam_redis / panels_core_redis). - api/sessions.ts + auth.ts: sesiones ahora en Redis (cookie = id opaco, no HMAC autocontenido); revocacion real (logout, cambio de password). - api/storage.ts (Fase 4c): documentos/PDFs via Contabo Object Storage (S3), con fallback a disco local si no hay credenciales S3 (dev). - api/scope.ts: middleware withCoreScope/requireCoreAuth que abre la transaccion con app.tenant_id fijado (RLS) para cada request. - api/cache.ts (Fase 4e): cache Redis con tenant_id obligatorio en la llave; aplicado a /v1/catalogs. Fase 3 (reescritura SQL, ~80 endpoints en main.ts/companies.ts/budget.ts/ payroll.ts/payroll_http.ts/excel.ts/saas.ts/smtp.ts): - Todo async/await, sintaxis Postgres (COALESCE, ~ regex, ON CONFLICT, now()/current_date, booleanos reales, RETURNING via lastInsertId()). - IDOR cross-tenant cerrado: GET/PATCH /v1/projects/:id, /v1/workers/:id ya no dependen de que el handler recuerde el WHERE tenant_id -- Row Level Security lo hace estructuralmente (verificado con un segundo tenant real: 404 en vez de fuga de datos). - API key ya no ve todos los tenants: ahora exige X-Tenant-Id explicito. Fase 3b (tests): api/test_helpers.ts corre cada test en una transaccion que siempre se revierte, contra el mismo baseline de Liquibase que produccion (ya no un esquema SQLite escrito a mano). payroll_test.ts reescrito con fixtures reales; 11/11 pasan contra Postgres. Fase 4 (IAM/RBAC): iam.roles/permissions/role_permissions formalizados (ver db/iam ya en fase 1); uploaded_by/created_by ahora son snapshot desnormalizado (uploaded_by_id/name); seed() en runtime eliminado, reemplazado por scripts/bootstrap-admin.ts (one-shot). Fase 4d (zona horaria): nuevo endpoint /v1/configuracion (GET/PUT), PAYROLL_TZ hardcodeado reemplazado por tenant_settings.timezone, document_validity.ts ya no usa new Date() crudo. Verificado end-to-end contra Postgres+Redis reales: login, sesiones, catalogos con cache, alta de trabajador, subida/descarga de documento cifrado, y el fix de IDOR probado con un segundo tenant real (403/404 en vez de fuga de datos). Co-authored-by: alberto.martinez <alberto.martinez@mrdev.mx>
54 lines
1.8 KiB
TypeScript
54 lines
1.8 KiB
TypeScript
import { createPool, PgDb, type postgres } from "./pg.ts";
|
|
|
|
/**
|
|
* Fase 3b: estrategia de tests con Postgres efímero.
|
|
*
|
|
* Los tests ya NO crean un esquema paralelo a mano (como hacía
|
|
* `new Database(":memory:")` de @db/sqlite): corren contra el MISMO
|
|
* baseline de Liquibase que producción (ver db/core/changesets), en la
|
|
* base de desarrollo local. Aislamiento entre tests: cada test corre
|
|
* dentro de una transacción que SIEMPRE se revierte al terminar (nunca
|
|
* hace commit), así que nunca contamina la base ni a otros tests, sin
|
|
* necesidad de TRUNCATE.
|
|
*
|
|
* Requiere DATABASE_URL_CORE_OWNER (o DATABASE_URL_CORE) apuntando a una
|
|
* base con las migraciones ya aplicadas -- ver db/provision/dev-local.sh
|
|
* y `./db/update.sh core --context-filter=dev`.
|
|
*/
|
|
|
|
let pool: postgres.Sql | null = null;
|
|
|
|
function getPool(): postgres.Sql {
|
|
if (!pool) {
|
|
const url = Deno.env.get("DATABASE_URL_CORE_OWNER") || Deno.env.get("DATABASE_URL_CORE");
|
|
if (!url) {
|
|
throw new Error(
|
|
"Define DATABASE_URL_CORE_OWNER (o DATABASE_URL_CORE) para correr los tests contra Postgres -- ver db/provision/dev-local.sh",
|
|
);
|
|
}
|
|
pool = createPool(url, { max: 5 });
|
|
}
|
|
return pool;
|
|
}
|
|
|
|
const ROLLBACK = Symbol("test-rollback");
|
|
|
|
/** Corre `fn(db)` dentro de una transacción que SIEMPRE se revierte. */
|
|
export async function withTestDb<T>(fn: (db: PgDb) => Promise<T>): Promise<T> {
|
|
const sql = getPool();
|
|
let result: T = undefined as unknown as T;
|
|
try {
|
|
await sql.begin(async (tx) => {
|
|
result = await fn(new PgDb(tx as unknown as postgres.Sql));
|
|
throw ROLLBACK;
|
|
});
|
|
} catch (e) {
|
|
if (e !== ROLLBACK) throw e;
|
|
}
|
|
return result;
|
|
}
|
|
|
|
export async function closeTestPool(): Promise<void> {
|
|
await pool?.end({ timeout: 1 });
|
|
pool = null;
|
|
}
|