feat: Sitzungen beenden, Verwaltungs-Verlauf -- und ein Kaestchen, das log

DER FEHLER ZUERST, denn er ist meiner von gestern: die Liste der
Meldungsanlaesse stand ZWEIMAL da -- als Standard in meldung.ts und noch einmal
als Weissliste beim Speichern in dash.ts. Beim Einbau des Plattenwaechters habe
ich nur die erste angefasst.

Ergebnis: das Kaestchen "Der Platz auf der Platte wird knapp" liess sich
ankreuzen, der Server warf den Wert beim Speichern weg, und beim naechsten
Laden war es wieder aus. Ohne ein Wort dazu. Nachgemessen am laufenden Dienst:

    geschickt    ["sicherung","verwaiste","platte"]
    gespeichert  ["sicherung","verwaiste"]

Die Liste steht jetzt an EINER Stelle (ANLAESSE in meldung.ts) und wird von
beiden benutzt. Gegengeprueft: alle drei kommen an.

SITZUNGEN BEENDEN. Wer sich an einem fremden Rechner angemeldet hat und es
spaeter merkt, hatte keine Moeglichkeit das zurueckzunehmen -- ein geaendertes
Passwort half nicht, die Sitzungen haengen an einer eigenen Tabelle und
ueberleben es. Der Knopf steht im Kontomenue und nur dann, wenn es ueberhaupt
eine zweite Sitzung gibt. Gemessen: 87 offen, 86 beendet, meine lebt.

Dabei in die eigene Falle getreten: der Weg lag zuerst bei den anderen
/auth-Wegen -- und die stehen mit Absicht VOR der Wache, weil man sich anmelden
koennen muss, ohne angemeldet zu sein. Dort ist c.get('user') leer, und der Weg
antwortete mit "Cannot read properties of undefined". Hono setzt Middleware und
Handler in der Reihenfolge ihrer Anmeldung zusammen; der Pfad sagt darueber
nichts.

VERWALTUNGS-VERLAUF. Der Datei-Verlauf beantwortet "was ist mit den Dateien
passiert". Wer den Discord-Webhook geaendert, einen Token angelegt oder ein
Konto entfernt hat, stand nirgends. Bei zwei Konten verschmerzbar -- sobald ein
drittes ueber die Discord-Rolle von selbst entsteht, ist es die erste Frage.

Eigene Tabelle und nicht events: dort traegt jede Zeile path und size, und eine
Einstellungsaenderung muesste path mit etwas fuellen, das kein Pfad ist.

WERTE STEHEN DORT NIE, nur Schluesselnamen. Unter den Einstellungen liegen das
Discord-Geheimnis und das Nextcloud-Passwort; ein Verlauf, der sie mitschreibt,
macht aus einer Tabelle mit einem Passwort eine Tabelle mit allen, die es je
gab.

Beim Einbau derselbe Fehlertyp wie gestern beim SVG: im SQL-Kommentar standen
Backticks, und das Schema liegt in einem Template-Literal -- ein Backtick
beendet es. Der Uebersetzer hat es gefangen.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
2026-08-12 07:43:57 +02:00
co-authored by Claude Opus 5
parent a91c4261de
commit 29e516559a
11 changed files with 445 additions and 16 deletions
+96 -7
View File
@@ -14,11 +14,13 @@ import {
avatarUrl,
db,
logEvent,
logVerwaltung,
now,
pruneEvents,
pruneSessions,
type Media,
type BenutzerZeile,
type VerwaltungZeile,
type Freigabe,
type MediaEvent,
type PapierkorbEintrag,
@@ -28,6 +30,8 @@ import {
import {
SESSION_COOKIE,
createSession,
countSessions,
destroyOtherSessions,
destroySession,
generateToken,
hashPassword,
@@ -51,7 +55,7 @@ import {
type Vorlage,
} from '../settings.js'
import { discordRoutes, redirectUri } from './discord.js'
import { MELDUNG_KEYS, meldungConfig, testMeldung } from '../meldung.js'
import { ANLAESSE, MELDUNG_KEYS, meldungConfig, testMeldung } from '../meldung.js'
import { holeVonAdresse, HolFehler, type Geholt } from '../holen.js'
import { neuerSchluessel } from './freigabe.js'
import {
@@ -259,7 +263,9 @@ const zeigen = (user: User) => ({
dashRoutes.get('/auth/me', (c) => {
const user = userForSession(getCookie(c, SESSION_COOKIE))
if (!user) return c.json({ error: 'nicht angemeldet' }, 401)
return c.json({ user: zeigen(user) })
// Die Zahl der offenen Sitzungen geht MIT: das Kontomenue zeigt sie an, und
// eine Zahl, die man erst auf einer anderen Seite findet, sieht niemand.
return c.json({ user: zeigen(user), sitzungen: countSessions(user.id) })
})
/** Was die Anmeldeseite wissen muss, BEVOR jemand angemeldet ist.
@@ -332,6 +338,41 @@ const ARTEN: Record<string, string> = {
"AND mime <> 'application/pdf' AND mime NOT LIKE 'text/%' AND mime NOT LIKE 'application/json%')",
}
/** Alle anderen Sitzungen beenden.
*
* WOZU: wer sich an einem fremden Rechner angemeldet hat und es spaeter merkt,
* hatte keine Moeglichkeit, das zurueckzunehmen -- ein geaendertes Passwort
* half nicht, die Sitzungen haengen an einer eigenen Tabelle und ueberleben
* es.
*
* Die EIGENE bleibt: wer drueckt, will sich nicht selbst aussperren.
*
* UND DIESER WEG STEHT HINTER DER WACHE, obwohl er unter /auth/ liegt.
* Zuerst stand er bei den anderen /auth-Wegen -- und die liegen mit Absicht
* DAVOR, weil man sich anmelden koennen muss, ohne angemeldet zu sein. Dort
* ist `c.get('user')` aber leer, und der Weg antwortete mit
* "Cannot read properties of undefined (reading 'id')" statt zu arbeiten.
*
* Hono setzt Middleware und Handler in der REIHENFOLGE ihrer Anmeldung
* zusammen: was vor `use('*')` steht, laeuft ohne sie. Der Pfad sagt darueber
* nichts. */
dashRoutes.post('/auth/sitzungen/beenden', (c) => {
const sid = getCookie(c, SESSION_COOKIE)
if (!sid) return c.json({ error: 'keine Sitzung' }, 401)
const user = c.get('user')
const beendet = destroyOtherSessions(user.id, sid)
if (beendet > 0) {
logVerwaltung(
user.username,
'benutzer',
'geaendert',
user.username,
`${beendet} Sitzung(en) beendet`,
)
}
return c.json({ beendet, offen: countSessions(user.id) })
})
dashRoutes.get('/media', (c) => {
const query = c.req.query('query')?.trim() ?? ''
const limit = Math.min(Number(c.req.query('limit') ?? 60) || 60, 200)
@@ -1045,6 +1086,23 @@ dashRoutes.post('/upload-url', async (c) => {
})
})
/** Der Verwaltungs-Verlauf: wer hat wann was EINGESTELLT.
*
* Getrennt vom Datei-Verlauf, weil es eine andere Frage beantwortet -- und
* weil `events` mit `path` und `size` die falschen Spalten dafuer hat.
*
* WERTE STEHEN HIER NIE, nur Schluesselnamen. Unter den Einstellungen liegen
* das Discord-Geheimnis und das Nextcloud-Passwort. */
dashRoutes.get('/verwaltung', (c) => {
const limit = Math.min(Number(c.req.query('limit') ?? 100) || 100, 500)
const offset = Math.max(Number(c.req.query('offset') ?? 0) || 0, 0)
const items = db
.prepare('SELECT * FROM verwaltung ORDER BY at DESC LIMIT ? OFFSET ?')
.all(limit, offset) as VerwaltungZeile[]
const total = (db.prepare('SELECT COUNT(*) AS n FROM verwaltung').get() as { n: number }).n
return c.json({ items, total, limit, offset })
})
dashRoutes.get('/events', (c) => {
const limit = Math.min(Number(c.req.query('limit') ?? 50) || 50, 500)
const offset = Math.max(Number(c.req.query('offset') ?? 0) || 0, 0)
@@ -1201,13 +1259,16 @@ dashRoutes.post('/settings', async (c) => {
}
}
if (Array.isArray(m.anlaesse)) {
const erlaubt = ['sicherung', 'verwaiste']
// Die Weissliste kommt aus meldung.ts und steht NICHT noch einmal
// hier. Genau diese Verdopplung hat den Plattenwaechter
// unbenutzbar gemacht: dort ergaenzt, hier vergessen.
setSetting(
MELDUNG_KEYS.anlaesse,
m.anlaesse.filter((a) => erlaubt.includes(a)).join(','),
m.anlaesse.filter((a) => (ANLAESSE as readonly string[]).includes(a)).join(','),
)
}
if (!body.discord && !body.sicherung && !body.vorlagen) {
logVerwaltung(c.get('user').username, 'einstellung', 'geaendert', 'Meldungen')
return c.json({ ok: true, meldungen: meldungConfig() })
}
}
@@ -1252,6 +1313,7 @@ dashRoutes.post('/settings', async (c) => {
}
setzeVorlagen(sauber)
if (!body.discord && !body.sicherung) {
logVerwaltung(c.get('user').username, 'einstellung', 'geaendert', 'Adressvorlagen', `${sauber.length} Stueck`)
return c.json({ ok: true, vorlagen: sauber })
}
}
@@ -1354,6 +1416,16 @@ dashRoutes.post('/settings', async (c) => {
else if (v) setSetting(DISCORD_KEYS.clientSecret, v)
}
// Am Ende angekommen heisst: Discord und/oder Sicherung wurden angefasst.
// Aufgeschrieben wird WELCHER Bereich -- nie ein Wert. Darunter liegen das
// Discord-Geheimnis und das Nextcloud-Passwort.
logVerwaltung(
c.get('user').username,
'einstellung',
'geaendert',
[body.discord && 'Discord', body.sicherung && 'Sicherung'].filter(Boolean).join(' + '),
)
const jetzt = discordConfig()
return c.json({
ok: true,
@@ -2037,6 +2109,7 @@ dashRoutes.delete('/benutzer/:id', (c) => {
if (!wer) return c.json({ error: 'nicht gefunden' }, 404)
db.prepare('DELETE FROM users WHERE id = ?').run(id)
logVerwaltung(c.get('user').username, 'benutzer', 'entfernt', wer.username)
return c.json({ ok: true, username: wer.username })
})
@@ -2081,7 +2154,7 @@ dashRoutes.post('/freigaben', async (c) => {
)
.run(schluessel, ordner, (body.titel ?? '').trim(), now())
logEvent('upload', ordner, 0, 'user', c.get('user').username)
logVerwaltung(c.get('user').username, 'freigabe', 'angelegt', ordner, `${drin.n} Dateien`)
return c.json({
id: Number(eintrag.lastInsertRowid),
@@ -2093,8 +2166,13 @@ dashRoutes.post('/freigaben', async (c) => {
})
dashRoutes.delete('/freigaben/:id', (c) => {
const r = db.prepare('DELETE FROM freigaben WHERE id = ?').run(Number(c.req.param('id')))
const id = Number(c.req.param('id'))
const vorher = db.prepare('SELECT ordner FROM freigaben WHERE id = ?').get(id) as
| { ordner: string }
| undefined
const r = db.prepare('DELETE FROM freigaben WHERE id = ?').run(id)
if (r.changes === 0) return c.json({ error: 'nicht gefunden' }, 404)
logVerwaltung(c.get('user').username, 'freigabe', 'entfernt', vorher?.ordner ?? String(id))
return c.json({ ok: true })
})
@@ -2331,6 +2409,8 @@ dashRoutes.post('/tokens', async (c) => {
expiresAt,
)
logVerwaltung(c.get('user').username, 'token', 'angelegt', name.trim(), cleanPrefix || 'ueberall')
// Der Klartext geht genau EINMAL raus. Danach steht nur noch der Hash in
// der Datenbank, und auch wir koennen ihn nicht mehr zeigen.
return c.json({
@@ -2461,6 +2541,8 @@ dashRoutes.patch('/tokens/:id', async (c) => {
FROM tokens WHERE id = ?`,
)
.get(id) as Omit<Token, 'hash'>
logVerwaltung(c.get('user').username, 'token', 'geaendert', neu.name, spalten.join(', '))
return c.json({
...neu,
abgelaufen: Boolean(neu.expires_at && neu.expires_at < now()),
@@ -2469,8 +2551,15 @@ dashRoutes.patch('/tokens/:id', async (c) => {
})
dashRoutes.delete('/tokens/:id', (c) => {
const info = db.prepare('DELETE FROM tokens WHERE id = ?').run(Number(c.req.param('id')))
const id = Number(c.req.param('id'))
// Erst den Namen holen: nach dem DELETE ist er weg, und "Token 7 entfernt"
// sagt in einem halben Jahr niemandem mehr etwas.
const vorher = db.prepare('SELECT name FROM tokens WHERE id = ?').get(id) as
| { name: string }
| undefined
const info = db.prepare('DELETE FROM tokens WHERE id = ?').run(id)
if (info.changes === 0) return c.json({ error: 'nicht gefunden' }, 404)
logVerwaltung(c.get('user').username, 'token', 'entfernt', vorher?.name ?? String(id))
return c.json({ ok: true })
})