pastilhasandClaude Opus 5
dcbc9b102c
one sidecar, one permission, one install — and two screens, because Calendar and Contacts are two places you go rather than two sections of one page. the first plugin to declare `screens`, which the platform gained for this. what moved: the radicale sidecar, the four routers, the panel apps, the app-password table and the settings form. the form became a modal on the calendar screen — a plugin ships panels and cannot contribute a settings section, which is the same wall the email plugin hit and the same answer. the routes are /calendar and /contacts, not /caldav/calendar. they were core routes and keeping them is not cosmetic: a bookmark, a dock tile someone arranged, and a persisted layout all address them. only a first-party plugin may choose its own segment; a third-party one is confined to /p/<publisher>/. four paths outside /api come with it, above the account gate, because phones are not browsers: DAVx5, iOS and thunderbird send HTTP basic on every request and have nowhere to put a JWT. that is not "open" — the sync door demands a scoped app password and resolves it to one user before radicale sees anything. two things deliberately unchanged. the table is still dav_app_passwords, not caldav_app_passwords: installs hold real rows and a naming convention is not worth a data migration on a live machine. and the panel names still resolve through the platform's catalogs, so the six locales they had as core screens survive rather than dropping to english to make the refactor tidier. per-user was already true and is worth writing down because the config said otherwise: the sync door resolves an app password to an account, tells radicale that id is the principal, and owner_only does the rest. the comment claiming "the owner is the only principal that exists" had been stale since multi-user landed on 2026-08-07. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Officer-Session: 656f5c85-e49a-4f1f-b9f3-042a04b4d043
47 lines
2.6 KiB
TypeScript
47 lines
2.6 KiB
TypeScript
import { pgTable, serial, integer, text, timestamp, index } from 'drizzle-orm/pg-core';
|
|
import { users } from 'officerdb/auth/schema';
|
|
|
|
// Per-device credentials for CalDAV / CardDAV clients — DAVx5, iOS, macOS, Thunderbird.
|
|
//
|
|
// A new credential type is unavoidable and NextCloud solved it the same way for the same reason: a DAV
|
|
// client cannot log into Officer. It speaks HTTP Basic on every request and has nowhere to put a 30-day
|
|
// JWT, no way to refresh one, and no way to answer a passkey challenge. Without this table the only
|
|
// credential a phone could carry is the account password itself — which then lives in that phone's
|
|
// account manager in recoverable form, and in whatever backs the phone up.
|
|
//
|
|
// So: one password per device, revocable per device, hashed with argon2 like the account password and
|
|
// SHOWN EXACTLY ONCE at creation. Nothing reads the plaintext back, because nothing stores it.
|
|
//
|
|
// SCOPE IS LOAD-BEARING. These are accepted ONLY by the /dav mount. `userMiddleware` must never look at
|
|
// this table — an app password is not a session, and a device that syncs a calendar has no business
|
|
// reaching the wallet or the vault.
|
|
//
|
|
// `revokedAt` rather than a delete, so a revoked credential stays visible in the UI ("this phone had
|
|
// access until Tuesday") instead of silently vanishing. `lastUsedAt` is what makes a stale device
|
|
// noticeable at all.
|
|
export const davAppPasswords = pgTable(
|
|
'dav_app_passwords',
|
|
{
|
|
id: serial('id').primaryKey(),
|
|
userId: integer('user_id')
|
|
.notNull()
|
|
.references(() => users.id, { onDelete: 'cascade' }),
|
|
/** What the owner calls the device — "Pixel 9", "iPad". The only way to tell two rows apart in the UI. */
|
|
label: text('label').notNull(),
|
|
/** argon2 hash. Deliberately not unique: two devices could in principle collide and it would not matter. */
|
|
passwordHash: text('password_hash').notNull(),
|
|
/**
|
|
* First 8 chars of the generated secret, stored in the clear on purpose. It is not enough to
|
|
* authenticate with, and it is the only way for the owner to match a row in this list against the
|
|
* password saved on a device they are looking at.
|
|
*/
|
|
hint: text('hint').notNull(),
|
|
lastUsedAt: timestamp('last_used_at', { withTimezone: true }),
|
|
revokedAt: timestamp('revoked_at', { withTimezone: true }),
|
|
createdAt: timestamp('created_at', { withTimezone: true }).notNull().defaultNow(),
|
|
},
|
|
// Every DAV request authenticates, and a phone polls often. This index is the difference between that
|
|
// being free and it being a sequential scan on every PROPFIND.
|
|
(t) => [index('idx_dav_app_passwords_user').on(t.userId)],
|
|
);
|