mirror of
https://github.com/snapotter-hq/SnapOtter.git
synced 2026-08-03 07:46:42 +02:00
* feat(rbac): add editor role, 3 new permissions, ownership helper * feat(rbac): add audit_log table, apiKeys.permissions column, editor role to schema * feat(rbac): wire requirePermission into all routes, add editor role support * refactor(rbac): replace ad-hoc role checks with permission-based ownership * feat(rbac): add audit log DB writes + query endpoint Dual-write audit events to stdout (existing) and SQLite audit_log table. Add GET /api/v1/audit-log with pagination, action filter, and date range filtering, gated behind audit:read permission. * feat(rbac): add API key permission scoping with ceiling enforcement * feat(rbac): add escalation prevention and last-admin protection * feat(rbac): add editor role to UI, API key permission scoping in settings * test(rbac): add full permission matrix integration test * test(rbac): add editor role E2E tests * feat(rbac): add custom roles with CRUD API and DB-backed permission lookup * feat(rbac): add API key expiration * feat(rbac): add roles management UI and API key expiration to settings * feat(rbac): add audit log UI to settings * fix: remove any cast in API key permission validation * test(rbac): add unit tests for username validation rules * test(rbac): add unit tests for effective permissions and ownership * test(rbac): add comprehensive route permission matrix (all routes × all roles) * test(rbac): add auth route edge case tests (login failures, session expiry, password side effects) * test(rbac): add escalation prevention tests (register, update, self-demote, last-admin) * test(rbac): add ownership enforcement tests (files, pipelines, editor access, cross-user isolation) * test(rbac): add API key edge cases (name validation, delete behavior, key revocation) * test(rbac): add audit log edge cases (all events, pagination clamping, structure) * test(rbac): add custom roles edge case tests (validation, CRUD, functional permissions) * test(rbac): add comprehensive E2E tests (roles UI, audit log, custom role, API key scoping)
180 lines
6.0 KiB
TypeScript
180 lines
6.0 KiB
TypeScript
import { eq } from "drizzle-orm";
|
|
import { afterAll, beforeAll, describe, expect, it } from "vitest";
|
|
import { db, schema } from "../../apps/api/src/db/index.js";
|
|
import { buildTestApp, loginAsAdmin, type TestApp } from "./test-server.js";
|
|
|
|
let testApp: TestApp;
|
|
let adminToken: string;
|
|
|
|
beforeAll(async () => {
|
|
testApp = await buildTestApp();
|
|
adminToken = await loginAsAdmin(testApp.app);
|
|
}, 30_000);
|
|
|
|
afterAll(async () => {
|
|
await testApp.cleanup();
|
|
}, 10_000);
|
|
|
|
describe("API key permission scoping", () => {
|
|
it("creates a key with scoped permissions", async () => {
|
|
const res = await testApp.app.inject({
|
|
method: "POST",
|
|
url: "/api/v1/api-keys",
|
|
headers: { authorization: `Bearer ${adminToken}` },
|
|
payload: { name: "scoped-key", permissions: ["tools:use", "files:own"] },
|
|
});
|
|
expect(res.statusCode).toBe(201);
|
|
const body = JSON.parse(res.body);
|
|
expect(body.permissions).toEqual(["tools:use", "files:own"]);
|
|
expect(body.key).toBeTruthy();
|
|
});
|
|
|
|
it("rejects permissions the user does not have", async () => {
|
|
await testApp.app.inject({
|
|
method: "POST",
|
|
url: "/api/auth/register",
|
|
headers: { authorization: `Bearer ${adminToken}` },
|
|
payload: { username: "scopetest", password: "ScopeTest1", role: "user" },
|
|
});
|
|
db.update(schema.users)
|
|
.set({ mustChangePassword: false })
|
|
.where(eq(schema.users.username, "scopetest"))
|
|
.run();
|
|
|
|
const loginRes = await testApp.app.inject({
|
|
method: "POST",
|
|
url: "/api/auth/login",
|
|
payload: { username: "scopetest", password: "ScopeTest1" },
|
|
});
|
|
const userToken = JSON.parse(loginRes.body).token;
|
|
|
|
const res = await testApp.app.inject({
|
|
method: "POST",
|
|
url: "/api/v1/api-keys",
|
|
headers: { authorization: `Bearer ${userToken}` },
|
|
payload: { name: "bad-scope", permissions: ["users:manage"] },
|
|
});
|
|
expect(res.statusCode).toBe(400);
|
|
});
|
|
|
|
it("scoped API key is restricted to its permissions", async () => {
|
|
const createRes = await testApp.app.inject({
|
|
method: "POST",
|
|
url: "/api/v1/api-keys",
|
|
headers: { authorization: `Bearer ${adminToken}` },
|
|
payload: { name: "readonly-key", permissions: ["tools:use", "settings:read"] },
|
|
});
|
|
const apiKey = JSON.parse(createRes.body).key;
|
|
|
|
const settingsRes = await testApp.app.inject({
|
|
method: "GET",
|
|
url: "/api/v1/settings",
|
|
headers: { authorization: `Bearer ${apiKey}` },
|
|
});
|
|
expect(settingsRes.statusCode).toBe(200);
|
|
|
|
const writeRes = await testApp.app.inject({
|
|
method: "PUT",
|
|
url: "/api/v1/settings",
|
|
headers: { authorization: `Bearer ${apiKey}` },
|
|
payload: { appName: "hacked" },
|
|
});
|
|
expect(writeRes.statusCode).toBe(403);
|
|
});
|
|
|
|
it("null permissions inherits all from user role", async () => {
|
|
const createRes = await testApp.app.inject({
|
|
method: "POST",
|
|
url: "/api/v1/api-keys",
|
|
headers: { authorization: `Bearer ${adminToken}` },
|
|
payload: { name: "full-key" },
|
|
});
|
|
const body = JSON.parse(createRes.body);
|
|
expect(body.permissions).toBeNull();
|
|
|
|
const writeRes = await testApp.app.inject({
|
|
method: "PUT",
|
|
url: "/api/v1/settings",
|
|
headers: { authorization: `Bearer ${body.key}` },
|
|
payload: { testSetting: "value" },
|
|
});
|
|
expect(writeRes.statusCode).toBe(200);
|
|
});
|
|
|
|
it("GET /api/v1/api-keys returns permissions field", async () => {
|
|
const res = await testApp.app.inject({
|
|
method: "GET",
|
|
url: "/api/v1/api-keys",
|
|
headers: { authorization: `Bearer ${adminToken}` },
|
|
});
|
|
const body = JSON.parse(res.body);
|
|
expect(body.apiKeys.length).toBeGreaterThan(0);
|
|
const scopedKey = body.apiKeys.find((k: any) => k.name === "scoped-key");
|
|
expect(scopedKey).toBeDefined();
|
|
expect(scopedKey.permissions).toEqual(["tools:use", "files:own"]);
|
|
});
|
|
});
|
|
|
|
describe("API key expiration", () => {
|
|
it("creates key with expiration", async () => {
|
|
const future = new Date(Date.now() + 86400000).toISOString(); // 24h from now
|
|
const res = await testApp.app.inject({
|
|
method: "POST",
|
|
url: "/api/v1/api-keys",
|
|
headers: { authorization: `Bearer ${adminToken}` },
|
|
payload: { name: "expiring-key", expiresAt: future },
|
|
});
|
|
expect(res.statusCode).toBe(201);
|
|
const body = JSON.parse(res.body);
|
|
expect(body.expiresAt).toBeTruthy();
|
|
});
|
|
|
|
it("rejects past expiration date", async () => {
|
|
const past = new Date(Date.now() - 86400000).toISOString();
|
|
const res = await testApp.app.inject({
|
|
method: "POST",
|
|
url: "/api/v1/api-keys",
|
|
headers: { authorization: `Bearer ${adminToken}` },
|
|
payload: { name: "past-key", expiresAt: past },
|
|
});
|
|
expect(res.statusCode).toBe(400);
|
|
});
|
|
|
|
it("expired key returns 401", async () => {
|
|
// Create a key, then manually set its expiration to the past
|
|
const createRes = await testApp.app.inject({
|
|
method: "POST",
|
|
url: "/api/v1/api-keys",
|
|
headers: { authorization: `Bearer ${adminToken}` },
|
|
payload: { name: "will-expire", expiresAt: new Date(Date.now() + 86400000).toISOString() },
|
|
});
|
|
const apiKey = JSON.parse(createRes.body).key;
|
|
const keyId = JSON.parse(createRes.body).id;
|
|
|
|
// Manually expire the key in DB
|
|
db.update(schema.apiKeys)
|
|
.set({ expiresAt: new Date(Date.now() - 1000) })
|
|
.where(eq(schema.apiKeys.id, keyId))
|
|
.run();
|
|
|
|
const res = await testApp.app.inject({
|
|
method: "GET",
|
|
url: "/api/v1/settings",
|
|
headers: { authorization: `Bearer ${apiKey}` },
|
|
});
|
|
expect(res.statusCode).toBe(401);
|
|
});
|
|
|
|
it("GET returns expiresAt field", async () => {
|
|
const res = await testApp.app.inject({
|
|
method: "GET",
|
|
url: "/api/v1/api-keys",
|
|
headers: { authorization: `Bearer ${adminToken}` },
|
|
});
|
|
const body = JSON.parse(res.body);
|
|
const expiringKey = body.apiKeys.find((k: any) => k.name === "expiring-key");
|
|
expect(expiringKey).toBeDefined();
|
|
expect(expiringKey.expiresAt).toBeTruthy();
|
|
});
|
|
});
|