mirror of
https://github.com/Termix-SSH/Termix.git
synced 2026-11-04 17:45:24 +00:00
* fix: keep concurrent ssh pool connects from overwriting each other (#1619) * fix: stop auto-accepting changed jump host keys (#1620) A jump hop whose key no longer matched the pinned fingerprint was silently accepted and the pin overwritten. Background status probes reconnect to jump hosts every cycle, so a MITM on that hop got the credentials with no user interaction. Changed jump host keys now go through the same path as any other host: prompt when there is a terminal socket, otherwise refuse. * fix(electron): validate external login URL scheme and callback port (#1621) * fix(plugins): read artifact manifest with the same tar parser as the unpack (#1622) The consent check read the first manifest.json with a hand-written tar reader, while the unpack keeps the last one and also follows pax and long-name headers. An archive with two manifests could pass the check with one capability list and get granted another. Read the manifest with node-tar, count only plain files, and refuse archives that hold more than one manifest.json. * fix: only trust the server's origin in the desktop login frame (#1624) The desktop sign-in iframe accepted AUTH_SUCCESS and system browser requests from whatever page it had navigated to, and posted replies (including 2FA temp tokens) to any origin. Messages must now come from the server's origin and replies are posted to it. The web UI also treated any framing page as the desktop app and posted the JWT to it. It now also requires the Termix-Desktop user agent. * fix: check trusted proxy login against nginx's peer, not nginx (#1625) The backend only sees the bundled nginx on loopback, so the allowlist could only work by trusting 127.0.0.1, which let any client that reached nginx log in as anyone by sending the identity headers. nginx now passes its own TCP peer (before real_ip rewriting) in X-Termix-Proxy-Peer, and loopback connections are judged by that address instead. * fix: keep foreign keys on during user data import (#1626) Import credentials before hosts and remap the file's credential ids to the ones created here, so hosts and protocol logins no longer point at another server's ids. With rows written in dependency order, the import no longer turns off SQLite foreign keys on the shared connection. The uploaded file is now removed on every exit path. * fix: enforce unique usernames (#1628) * fix: enforce unique usernames Registration checked for a taken name, hashed the password, then inserted, with nothing in the schema enforcing uniqueness. Concurrent sign-ups could create two accounts with one name, after which login, password reset, proxy login and admin lookups by username picked between them arbitrarily. Add a unique index on users.username for every dialect. Existing duplicates are renamed (later rows get a -<id> suffix, the oldest keeps the name) rather than deleted. Registration and admin user creation now return the same 409 when the insert loses the race. * fix: avoid username collisions during migration * fix: don't auto-enable synced plugins signed only by a server-sent key (#1623) * fix: don't auto-enable synced plugins signed only by a server-sent key Registry keys a linked server sends still let its plugins load on the desktop, but no longer count as a signer for switching them on or copying their grants. Keys the desktop added itself keep their trust. * fix: enforce local plugin trust across remote source labels * fix: require positive local trust for synced plugin keys * fix(desktop): harden external editor and session cookie IPC (#1627) * fix(desktop): stop the renderer choosing the external editor binary or reading the session cookie open-external-editor now launches only the editor saved from a file dialog and refuses to hand executable file types to the OS opener. The unused get-session-cookie and wait-session-cookie channels, which returned the httpOnly jwt, are removed. * fix: validate setting keys before editor authorization * fix(sidebar): sort hosts by IP address numerically (#1629) * fix(desktop): find spawn-helper in universal macOS builds (#1630) The universal build ships app-arm64.asar and app-x64.asar instead of app.asar, so node-pty's helper path rewrite never pointed at the unpacked copy and every local terminal failed with posix_spawnp failed. * fix(sync): send shared hosts with text columns in their stored form (#1631) resolveHostById turns forceKeyboardInteractive into a boolean, and the shared-host export passed it on as is. A desktop could not bind the boolean into the text column, so every shared host failed to apply. * fix: drop ssh_data.vault_profile_id so removing the vault plugin keeps hosts writable (#1633) The vault plugin renames vault_profiles to p_vault_profiles, and SQLite rewrites ssh_data's foreign key to follow it. Uninstalling the plugin then dropped p_vault_profiles and left ssh_data pointing at a missing table, so creating, cloning and deleting hosts failed with "no such table: main.p_vault_profiles". Postgres and MySQL refused the uninstall instead. The column was already copied into the vault plugin's host settings by 2.9, so retire it on every engine. The SQLite boot drop also repairs databases already in the broken state; MySQL drops the column's foreign key first. * fix(sync): let a linked desktop enable mirrored plugins it does not trust on its own (#1632) * fix(sync): let a linked desktop enable mirrored plugins it does not trust on its own * fix(sync): switch off an untrusted mirrored plugin when the server updates it * fix: require local consent for mirrored plugin activation * docs: describe consent fields on the plugin state endpoint * fix(i18n): keep Indonesian technical terms consistent (#1636) Use the English terms Indonesian users know for tunnel, snippet, deploy, file manager, remote desktop, jump host, passphrase, sidebar, toolbar, endpoint, browser and SSH agent, instead of mixing literal translations (terowongan, cuplikan, terapkan, bilah sisi, titik akhir, ...) with the English words. Match the split shortcuts to the split screen menu. Also fix the user data export description, which said secrets are encrypted while the English says they are decrypted. * fix(settings): leave user manage view when switching admin sections (#1649) The admin panel stays mounted across Settings pages, so an open user manage view kept replacing every other admin section until Back to users was clicked. Reset it whenever the selected section changes. * fix(sync): don't delete a local row when the server can't send its copy (#1650) * fix(sync): don't delete a local row when the server can't send its copy A push conflict on a live record whose server row failed to serialize came back as row: null, deleted: false, and the desktop treated that as a delete. The server now rejects that op as unavailable, the desktop never deletes on a conflict without a row, and serialization failures are logged. * fix(sync): retry temporarily unavailable records * fix: smooth host editor jumps to lower sections * chore: add changelog entries for 26.10.2 fixes * fix: stop building ssh2 native crypto in the docker image (#1644) * chore: test that an update keeps a changed auth type (#1601) * feat: option to hide the sidebar after opening a host (#1652) * feat: sort the dashboard host status card (Plugin-Host-Metrics#6) * fix: let the hover side menu overlay content instead of resizing it (Plugin-Remote-Desktop#2) * fix: say the server is down and keep retrying instead of a database error (#1637) * fix: show why a client tunnel failed to start (Plugin-Tunnels#2) * fix(sdk): skip comment-only chunks when splitting migrations * feat: setting to let the hover side menu push content again * fix: use the vite proxy for plugin sockets on any dev port * feat(sdk): let a plugin raise its public route rate limit * chore: update translations * chore: add the overview video to the readme * chore: update the overview video and add a play button to its thumbnail * feat: plugin builds bundle worker entries and import wasm and fonts as bytes * feat: plugin packs include LICENSE and NOTICE * chore: drop guacd from compose files and dev runner * chore: update remote desktop origin comment for the built-in engine * fix: show plugin names and descriptions in the UI language (#1658) * fix: show plugin names and descriptions in the UI language Read plugin.name and plugin.description from each plugin's own locale namespace, falling back to the manifest, everywhere core shows a plugin's name: the Plugins manager, onboarding picker, host editor tabs and sections, host filters and bulk actions, beta updates, interface presets and plugin crash/loading notices. * docs(sdk): document plugin.name and plugin.description locale keys --------- Co-authored-by: ZacharyZcR <zkl205305@gmail.com> * fix(hosts): reject or infer a missing authType on host create (#1664) Creating an SSH host through the API without authType hit the NOT NULL auth_type column and came back as a 500. The create route now infers the type from the credential, key or password in the body, falls back to the host defaults, and otherwise answers 400. A non-string authType is a 400 on create and update. * chore: changelog for merged plugin name and host auth type PRs * fix: move macOS split pane navigation to Cmd+Option+Arrow * fix: pick a free port for the desktop backend when 30001 is taken (#1663) * feat: build the Windows desktop app for ARM64 (#1615) * fix(sync): keep the changes cursor within the 32-bit seq column (#1665) The desktop learned the server's entity types by asking for changes after Number.MAX_SAFE_INTEGER. sync_records.seq is a 32-bit integer on Postgres, which rejects that bound value, so every pass needing the probe failed with 22003 out of range. The server now clamps the cursor to the largest seq the column holds and answers 400 for a cursor or limit that is not a non-negative integer. The desktop asks for a one-row page from its saved cursor instead. * fix(audit): coalesce identical plugin routine audits so they cannot flush the log (#1667) Successful plugin_as_user and plugin_service_* entries written by a plugin with the same user, action, resource and details are recorded once per 10 minutes. Failures and every other action are still written each time. * feat(plugins): publish host.created when a host is added (#1668) Plugins that keep an inventory of hosts had no way to hear about a new one short of polling. Core now publishes host.created with the same { hostId, userId } payload as host.updated wherever a host row is created: the host create route, ctx.hosts.create, JSON and SSH config bulk import, database file import, and sync writing a host it did not have yet. * fix: show the login for signed-out full-screen view links and return to them (#1669) A ?view= link opened without a session spun forever: the gate rendered the bare view when the session check failed, so the plugin runtime never started and the view never registered. A regular browser now keeps the link in sessionStorage, shows the normal login and opens the link again after any login, including redirect logins. Only same-origin paths with a view are restored. The desktop app, embedded sign-in frames and guest views keep their current behavior. * fix(desktop): apply linked server proxy headers and certificate exception to WebSockets (#1670) * fix(desktop): send linked server proxy headers on WebSocket upgrades isLinkedOrigin compared the request origin with the link's http(s) origin, so the terminal WebSocket (wss://host/...) never matched and its upgrade went out without the proxy headers and basic auth. Behind a forward-auth proxy that upgrade was redirected to the login and the terminal closed with "connection closed unexpectedly". ws and wss now count as http and https for the comparison, the same mapping the WebSocket handshake itself uses, with host and port still matched exactly. * fix(desktop): allow a linked server's self-signed certificate for its WebSockets isInvalidCertificateAllowedForUrl compared the URL with a local getOrigin copy, so wss://host never matched the link's https://host origin and an allowed self-signed certificate did not cover the terminal and tunnel sockets. It now uses linkedServer.isLinkedOrigin, which maps ws and wss the same way as the proxy headers, and the local copy is removed. * feat(sdk): let plugins mint a short-lived user session and read a user's locale (#1666) ctx.auth.issueUserToken(userId, { purpose, ttlSeconds }) gives background work a session for calling Termix's own API as an existing user. It needs users:impersonate, defaults to 300s and is capped at 900s, is stored as a normal session row marked as plugin-issued, and every issue is audited. Such sessions are refused on routes that change the user's password, second factors, API keys or linked logins. ctx.users.getLocale(userId) reads the user's UI language under users:read. SDK 1.0.11, plugin API 1.3.0. * chore: changelog for merged sync, audit, event and desktop PRs * feat: add shell.rightEdge slot so plugin panels can push the page over * chore: pin bundled plugins to the latest releases * chore: lint, format, and bump version to 26.10.2 --------- Co-authored-by: ZacharyZcR <zacharyzcr1984@gmail.com> Co-authored-by: Ferdina Kusumah <19812460+FerdinaKusumah@users.noreply.github.com> Co-authored-by: ZacharyZcR <zkl205305@gmail.com> Co-authored-by: Lukas <80832688+0skater0@users.noreply.github.com>
495 lines
16 KiB
TypeScript
495 lines
16 KiB
TypeScript
/**
|
|
* HTTP for plugins: one mount point, one set of middleware, no ports.
|
|
*
|
|
* A plugin calls ctx.http.router() and gets an Express Router that core mounts
|
|
* at /plugin-api/<id>/. Everything a route needs in front of it runs here
|
|
* rather than in the plugin:
|
|
*
|
|
* 1. auth, unless the path was declared public;
|
|
* 2. the actor, so ctx and core APIs know who is calling;
|
|
* 3. an enabled check, so a disabled plugin answers 503 rather than 404;
|
|
* 4. body limits;
|
|
* 5. an error wrapper that logs against the plugin and leaks nothing.
|
|
*
|
|
* The wrapper never touches the response body, so SSE, file downloads and
|
|
* multipart uploads stream through untouched.
|
|
*/
|
|
|
|
import express, {
|
|
type NextFunction,
|
|
type Request,
|
|
type Response,
|
|
type Router,
|
|
} from "express";
|
|
import type {
|
|
PluginMiddleware,
|
|
PluginRouterOptions,
|
|
} from "@termix-ssh/plugin-sdk/backend";
|
|
import type { PluginManifest } from "@termix-ssh/plugin-sdk/manifest";
|
|
import { pluginLogger } from "../utils/logger.js";
|
|
import { AuthManager } from "../utils/auth-manager.js";
|
|
import { runAsActor } from "./actor.js";
|
|
import { hasCapability } from "./permissions.js";
|
|
import { declaredPermissions, resolvePermission } from "./rbac.js";
|
|
|
|
/** Matches what core's own routes accept, so moving a route changes nothing. */
|
|
const DEFAULT_BODY_LIMIT = "2mb";
|
|
|
|
/** Routers by plugin id, read by the dispatcher in plugin-api-routes.ts. */
|
|
const routers = new Map<string, Router>();
|
|
|
|
/**
|
|
* Requests per minute one address may make to one plugin's public routes.
|
|
* They run without a login, so they are the cheap way to hammer a plugin.
|
|
*/
|
|
const MAX_PLUGIN_PUBLIC_RATE_LIMIT = 10_000;
|
|
|
|
/** The admin's limit when set, else the plugin's own, else 120. */
|
|
function publicRateLimit(pluginLimit?: number): number {
|
|
const parsed = Number(process.env.TERMIX_PLUGIN_PUBLIC_RATE_LIMIT);
|
|
if (Number.isFinite(parsed) && parsed > 0) return parsed;
|
|
if (Number.isFinite(pluginLimit) && (pluginLimit as number) > 0) {
|
|
return Math.min(pluginLimit as number, MAX_PLUGIN_PUBLIC_RATE_LIMIT);
|
|
}
|
|
return 120;
|
|
}
|
|
const PUBLIC_RATE_WINDOW_MS = 60_000;
|
|
const publicHits = new Map<string, { count: number; resetAt: number }>();
|
|
|
|
/** True when this request is over the public route limit. */
|
|
export function overPublicRateLimit(
|
|
pluginId: string,
|
|
ip: string,
|
|
now = Date.now(),
|
|
pluginLimit?: number,
|
|
): boolean {
|
|
if (publicHits.size > 10_000) {
|
|
for (const [hitKey, entry] of publicHits) {
|
|
if (entry.resetAt <= now) publicHits.delete(hitKey);
|
|
}
|
|
}
|
|
const hitKey = `${pluginId}|${ip}`;
|
|
const entry = publicHits.get(hitKey);
|
|
if (!entry || entry.resetAt <= now) {
|
|
publicHits.set(hitKey, { count: 1, resetAt: now + PUBLIC_RATE_WINDOW_MS });
|
|
return false;
|
|
}
|
|
entry.count++;
|
|
return entry.count > publicRateLimit(pluginLimit);
|
|
}
|
|
|
|
/** Test seam. */
|
|
export function resetPublicRateLimits(): void {
|
|
publicHits.clear();
|
|
}
|
|
|
|
/** Declared public paths by plugin id, for the admin plugin details. */
|
|
const publicRoutes = new Map<string, string[]>();
|
|
|
|
export function getPluginPublicHttpRoutes(pluginId: string): string[] {
|
|
return [...(publicRoutes.get(pluginId) ?? [])];
|
|
}
|
|
|
|
export function getPluginRouter(pluginId: string): Router | undefined {
|
|
return routers.get(pluginId);
|
|
}
|
|
|
|
export function unregisterPluginHttp(pluginId: string): void {
|
|
publicRoutes.delete(pluginId);
|
|
if (!routers.delete(pluginId)) return;
|
|
pluginLogger.info(`Unmounted /plugin-api/${pluginId}`, {
|
|
operation: "plugin_http_unmount",
|
|
});
|
|
}
|
|
|
|
/**
|
|
* Whether the plugin is currently running.
|
|
*
|
|
* Injected rather than imported so this module does not depend on the loader,
|
|
* which imports ctx.ts, which would close a cycle.
|
|
*/
|
|
type IsEnabled = (pluginId: string) => boolean;
|
|
|
|
let isPluginEnabled: IsEnabled = () => true;
|
|
|
|
export function setPluginEnabledCheck(check: IsEnabled): void {
|
|
isPluginEnabled = check;
|
|
}
|
|
|
|
/**
|
|
* Whether a plugin with this id is installed at all, running or not. A
|
|
* disabled plugin has no router or socket routes left, so this is what tells
|
|
* "disabled" (503) apart from "no such plugin" (404).
|
|
*/
|
|
let isInstalled: IsEnabled = () => false;
|
|
|
|
export function setPluginInstalledCheck(check: IsEnabled): void {
|
|
isInstalled = check;
|
|
}
|
|
|
|
export function isPluginInstalled(pluginId: string): boolean {
|
|
return isInstalled(pluginId);
|
|
}
|
|
|
|
/** Test seam, so one test's stub does not leak into the next. */
|
|
export function resetPluginHttp(): void {
|
|
routers.clear();
|
|
publicRoutes.clear();
|
|
isPluginEnabled = () => true;
|
|
isInstalled = () => false;
|
|
}
|
|
|
|
function normalizePublicPath(path: string): string {
|
|
const trimmed = path.trim();
|
|
const withSlash = trimmed.startsWith("/") ? trimmed : `/${trimmed}`;
|
|
return withSlash.length > 1 && withSlash.endsWith("/")
|
|
? withSlash.slice(0, -1)
|
|
: withSlash;
|
|
}
|
|
|
|
/**
|
|
* Compiles a declared public path into a matcher.
|
|
*
|
|
* Express-style ":param" segments are supported, because the routes that have
|
|
* to be public are usually keyed by a token ("/webhook/:token"). A parameter
|
|
* matches exactly one segment and never a slash, so ":token" cannot swallow
|
|
* the rest of the path and turn one declared route into a prefix hole.
|
|
*
|
|
* A trailing "/*" is the one deliberate prefix: it matches the path itself and
|
|
* anything below it, for a proxied page that loads its own subpaths.
|
|
*/
|
|
function compilePublicPath(path: string): RegExp {
|
|
const prefix = path.endsWith("/*");
|
|
const escaped = (prefix ? path.slice(0, -2) : path)
|
|
.split("/")
|
|
.map((segment) => {
|
|
if (segment.startsWith(":")) return "[^/]+";
|
|
return segment.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
|
|
})
|
|
.join("/");
|
|
return new RegExp(prefix ? `^${escaped}(?:/.*)?$` : `^${escaped}$`);
|
|
}
|
|
|
|
interface PublicPath {
|
|
declared: string;
|
|
pattern: RegExp;
|
|
}
|
|
|
|
interface RouterLayer {
|
|
route?: {
|
|
path?: unknown;
|
|
_handlesMethod?: (method: string) => boolean;
|
|
};
|
|
match?: (path: string) => boolean;
|
|
}
|
|
|
|
function segments(path: string): string[] {
|
|
return normalizePublicPath(path).split("/").slice(1);
|
|
}
|
|
|
|
/**
|
|
* Whether a declared public path names this registered route: literal
|
|
* segments must be equal and a ":param" must sit where the route has one, so
|
|
* "/webhook/:token" never covers a sibling "/webhook/config".
|
|
*/
|
|
function declaredCoversRoute(declared: string, routePath: string): boolean {
|
|
const prefix = declared.endsWith("/*");
|
|
const want = segments(prefix ? declared.slice(0, -2) : declared);
|
|
const have = segments(routePath);
|
|
if (prefix ? have.length < want.length : have.length !== want.length) {
|
|
return false;
|
|
}
|
|
return want.every((segment, index) =>
|
|
segment.startsWith(":")
|
|
? have[index].startsWith(":")
|
|
: segment.toLowerCase() === have[index].toLowerCase(),
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Whether the request is for a declared public path.
|
|
*
|
|
* Full-path match only, and only for the route that will actually answer:
|
|
* the first route in the plugin's router that matches the path and method
|
|
* has to be the one the declaration names. So a public "/webhook/:token"
|
|
* registered for POST does not open GET, and does not open a literal sibling
|
|
* route registered next to it. A trailing "/*" may also be served by
|
|
* router.use(), which has no route to compare against.
|
|
*/
|
|
function isPublicRequest(
|
|
req: Request,
|
|
publicPaths: PublicPath[],
|
|
inner: Router,
|
|
): boolean {
|
|
if (publicPaths.length === 0) return false;
|
|
const path = normalizePublicPath(req.path || "/");
|
|
const candidates = publicPaths.filter(({ pattern }) => pattern.test(path));
|
|
if (candidates.length === 0) return false;
|
|
|
|
const stack = (inner as unknown as { stack?: RouterLayer[] }).stack ?? [];
|
|
for (const layer of stack) {
|
|
if (!layer.match?.(req.path || "/")) continue;
|
|
if (!layer.route) {
|
|
if (candidates.some(({ declared }) => declared.endsWith("/*"))) {
|
|
return true;
|
|
}
|
|
continue;
|
|
}
|
|
if (!layer.route._handlesMethod?.(req.method)) continue;
|
|
const routePath = layer.route.path;
|
|
return (
|
|
typeof routePath === "string" &&
|
|
candidates.some(({ declared }) =>
|
|
declaredCoversRoute(declared, routePath),
|
|
)
|
|
);
|
|
}
|
|
return false;
|
|
}
|
|
|
|
export interface CreatePluginRouterArgs {
|
|
manifest: PluginManifest;
|
|
options?: PluginRouterOptions;
|
|
/** Records a runtime error against the plugin's error budget. */
|
|
reportError: (error: unknown) => void;
|
|
}
|
|
|
|
export function createPluginRouter({
|
|
manifest,
|
|
options,
|
|
reportError,
|
|
}: CreatePluginRouterArgs): Router {
|
|
const pluginId = manifest.id;
|
|
if (routers.has(pluginId)) {
|
|
throw new Error(
|
|
`Plugin ${pluginId} already has a router; call ctx.http.router() once`,
|
|
);
|
|
}
|
|
const declaredPublic = (options?.public ?? []).map((path) =>
|
|
normalizePublicPath(path),
|
|
);
|
|
const publicPaths = declaredPublic.map((declared) => ({
|
|
declared,
|
|
pattern: compilePublicPath(declared),
|
|
}));
|
|
publicRoutes.set(pluginId, declaredPublic);
|
|
|
|
if (declaredPublic.length > 0) {
|
|
// Audited once at registration: the set of unauthenticated routes a plugin
|
|
// has is worth seeing even if nobody ever calls them.
|
|
pluginLogger.warn(
|
|
`Plugin ${pluginId} serves ${declaredPublic.length} route(s) without authentication: ${declaredPublic.join(", ")}`,
|
|
{ operation: "plugin_http_public" },
|
|
);
|
|
void writePublicRouteAudit(manifest, declaredPublic);
|
|
}
|
|
|
|
const outer = express.Router();
|
|
const inner = express.Router();
|
|
|
|
outer.use((req: Request, res: Response, next: NextFunction) => {
|
|
if (isPluginEnabled(pluginId)) {
|
|
next();
|
|
return;
|
|
}
|
|
res.status(503).json({ error: "This feature is not available", pluginId });
|
|
});
|
|
|
|
// The grant lives in the database, so it cannot be checked in the
|
|
// synchronous router() call a plugin makes inside activate. Checked per
|
|
// request instead: a plugin whose network:serve was revoked keeps its mount
|
|
// point and refuses every request through it, which is the same outcome a
|
|
// revoke should have and takes effect without a restart.
|
|
outer.use((req: Request, res: Response, next: NextFunction) => {
|
|
void (async () => {
|
|
try {
|
|
const granted = await hasCapability(
|
|
pluginId,
|
|
"network:serve",
|
|
manifest.capabilities,
|
|
);
|
|
if (!granted) {
|
|
res.status(403).json({
|
|
error: "Plugin is not granted the network:serve capability",
|
|
pluginId,
|
|
});
|
|
return;
|
|
}
|
|
next();
|
|
} catch (error) {
|
|
next(error);
|
|
}
|
|
})();
|
|
});
|
|
|
|
const authenticate = AuthManager.getInstance().createAuthMiddleware();
|
|
|
|
outer.use((req: Request, res: Response, next: NextFunction) => {
|
|
if (isPublicRequest(req, publicPaths, inner)) {
|
|
const ip = req.ip || req.socket?.remoteAddress || "unknown";
|
|
if (
|
|
overPublicRateLimit(pluginId, ip, Date.now(), options?.publicRateLimit)
|
|
) {
|
|
res.status(429).json({ error: "Too many requests" });
|
|
return;
|
|
}
|
|
pluginLogger.info(`Unauthenticated request to ${pluginId}${req.path}`, {
|
|
operation: "plugin_http_public_request",
|
|
});
|
|
next();
|
|
return;
|
|
}
|
|
authenticate(req, res, next);
|
|
});
|
|
|
|
if (!options?.rawBody) {
|
|
const limit = options?.bodyLimit ?? DEFAULT_BODY_LIMIT;
|
|
// Both parsers ignore a body they do not claim, so multipart still reaches
|
|
// multer untouched. database.ts skips its own global parser for
|
|
// /plugin-api so this limit is the one that actually applies.
|
|
outer.use(express.json({ limit }));
|
|
outer.use(express.urlencoded({ limit, extended: true }));
|
|
}
|
|
|
|
// After auth, so the actor is the user core authenticated rather than
|
|
// whatever the request claimed.
|
|
outer.use((req: Request, _res: Response, next: NextFunction) => {
|
|
const { userId, sessionId } = req as Request & {
|
|
userId?: string;
|
|
sessionId?: string;
|
|
};
|
|
if (!userId) {
|
|
next();
|
|
return;
|
|
}
|
|
runAsActor(userId, "request", () => next(), sessionId);
|
|
});
|
|
|
|
outer.use(inner);
|
|
|
|
outer.use(
|
|
(
|
|
error: unknown,
|
|
req: Request,
|
|
res: Response,
|
|
_next: NextFunction,
|
|
): void => {
|
|
void _next;
|
|
const status = clientErrorStatus(error);
|
|
const authenticated = !!(req as Request & { userId?: string }).userId;
|
|
// A bad body or a 4xx is the caller's fault, and an anonymous caller
|
|
// must never be able to spend the plugin's error budget: five junk
|
|
// requests to a public route would otherwise switch the plugin off.
|
|
if (status === null && authenticated) reportError(error);
|
|
if (status !== null) {
|
|
if (!res.headersSent) {
|
|
res.status(status).json({ error: "Invalid request" });
|
|
} else {
|
|
res.end();
|
|
}
|
|
return;
|
|
}
|
|
pluginLogger.error(
|
|
`Plugin ${pluginId} route ${req.method} ${req.path} failed`,
|
|
error instanceof Error ? error : new Error(String(error)),
|
|
{ operation: "plugin_http_error" },
|
|
);
|
|
if (res.headersSent) {
|
|
// Mid-stream: the only honest thing left is to end it.
|
|
res.end();
|
|
return;
|
|
}
|
|
res.status(500).json({ error: "Plugin request failed" });
|
|
},
|
|
);
|
|
|
|
routers.set(pluginId, outer);
|
|
pluginLogger.info(`Mounted /plugin-api/${pluginId}`, {
|
|
operation: "plugin_http_mount",
|
|
});
|
|
|
|
return inner;
|
|
}
|
|
|
|
/** The status of an error that is the caller's fault, or null. */
|
|
function clientErrorStatus(error: unknown): number | null {
|
|
const status =
|
|
(error as { status?: unknown; statusCode?: unknown } | null)?.status ??
|
|
(error as { statusCode?: unknown } | null)?.statusCode;
|
|
return typeof status === "number" && status >= 400 && status < 500
|
|
? status
|
|
: null;
|
|
}
|
|
|
|
async function writePublicRouteAudit(
|
|
manifest: PluginManifest,
|
|
paths: string[],
|
|
): Promise<void> {
|
|
try {
|
|
const { logAudit } = await import("../utils/audit-logger.js");
|
|
await logAudit({
|
|
userId: null,
|
|
username: `plugin:${manifest.id}`,
|
|
action: "plugin_http_public_routes",
|
|
resourceType: "plugin",
|
|
resourceId: manifest.id,
|
|
resourceName: manifest.name,
|
|
details: `unauthenticated routes: ${paths.join(", ")}`,
|
|
success: true,
|
|
});
|
|
} catch {
|
|
// Auditing must never stop a plugin starting.
|
|
}
|
|
}
|
|
|
|
/**
|
|
* Builds ctx.rbac.require.
|
|
*
|
|
* A plugin may only gate on a permission it declares itself. Without that a
|
|
* route could require admin.users.manage and borrow someone else's authority,
|
|
* which is the escalation shape the manifest validator already blocks for
|
|
* role defaults.
|
|
*
|
|
* The 401/403 bodies match requirePermission(), so a plugin route denies the
|
|
* same way a core route does and a client needs one code path.
|
|
*/
|
|
export function createRbacMiddleware(
|
|
manifest: PluginManifest,
|
|
permission: string,
|
|
): PluginMiddleware {
|
|
const required = resolvePermission(manifest, permission);
|
|
const declared = declaredPermissions(manifest);
|
|
|
|
if (!declared.has(required)) {
|
|
throw new Error(
|
|
`Plugin ${manifest.id} cannot require permission "${permission}": it is not declared in contributes.permissions`,
|
|
);
|
|
}
|
|
|
|
return ((req: Request, res: Response, next: NextFunction) => {
|
|
void (async () => {
|
|
const userId = (req as Request & { userId?: string }).userId;
|
|
if (!userId) {
|
|
res.status(401).json({ error: "Not authenticated" });
|
|
return;
|
|
}
|
|
|
|
try {
|
|
const { PermissionManager } =
|
|
await import("../utils/permission-manager.js");
|
|
const allowed = await PermissionManager.getInstance().hasPermission(
|
|
userId,
|
|
required,
|
|
);
|
|
if (!allowed) {
|
|
res.status(403).json({ error: "Insufficient permissions", required });
|
|
return;
|
|
}
|
|
next();
|
|
} catch (error) {
|
|
next(error);
|
|
}
|
|
})();
|
|
}) as unknown as PluginMiddleware;
|
|
}
|