feat: Favicon, Token bearbeiten, und Hochladen per Adresse

FAVICON. /favicon.ico antwortete mit 200 und text/html -- die SPA-Rueckfall-
route lieferte index.html als Icon aus. Ein leeres Blatt im Tab und 2 KB umsonst
bei jedem Aufruf, nachgemessen bevor es hier steht.

Jetzt ein SVG in den Farben der Oberflaeche (Grund #0b0d10, Akzent #4ea3ff,
dieselbe Plattenform wie in der Kopfzeile), dazu PNG in 32, 180 und 512 -- AUS
DERSELBEN SVG-Datei gerechnet und nicht zweimal gezeichnet. Plus ein
web-manifest, damit das Symbol auf einem Telefon-Startbildschirm stimmt.

Der erste Anlauf war kaputt, und zwar fuer JEDEN Parser: im SVG-Kommentar stand
"--color-grund", und XML verbietet den doppelten Bindestrich in Kommentaren.
Aufgefallen beim Rechnen der PNG, nicht erst im Browser.

TOKEN BEARBEITEN. Kontingent, Groessengrenze, Arten, Ablauf und Praefix liessen
sich nur beim ANLEGEN setzen -- wer einem bestehenden Token nachtraeglich eine
Grenze geben wollte, musste ihn neu anlegen und damit den Schluessel in jedem
Skript tauschen. Fuer eine Zahl in einer Tabelle der falsche Preis.

PATCH /api/dash/tokens/:id, jedes Feld einzeln. WEGGELASSEN heisst UNVERAENDERT,
null heisst ausdruecklich "keine Grenze" -- ohne diesen Unterschied liesse sich
eine einmal gesetzte Grenze nie wieder loeswerden. Der Schluessel selbst bleibt
unberuehrt, und das ist keine Vorsicht: in der Tabelle steht nur sein Hash.

HOCHLADEN PER ADRESSE. Der Dienst holt die Datei selbst. Das ist die
gefaehrlichste Funktion in diesem Dienst und steht deshalb in einer eigenen
Datei (holen.ts) -- ein Server, der eine vom Benutzer genannte Adresse abruft,
ist ein Angriff mit eigenem Namen.

Geprueft wird das Schema, der Anschluss und die AUFGELOESTE IP -- nicht der
Name. Ein Namensfilter waere einer fuer den, der ihn nicht umgehen will. Und
jede Umleitung wird SELBST gelaufen (redirect: 'manual') und neu geprueft;
liesse man fetch folgen, waere genau dort die Luecke.

Durchgemessen am laufenden Dienst, fuenfzehn Faelle:

    Erfolgsfall                                     abgelegt, 131956 B
    127.0.0.1 / localhost / [::1] / 0.0.0.0         abgelehnt
    10.x / 172.20.x / 192.168.x / 169.254.169.254   abgelehnt
    127.0.0.1.nip.io  (oeffentlicher Name, private IP)   abgelehnt
    192.168.2.1.nip.io / 169.254.169.254.nip.io         abgelehnt
    file:// und gopher://                           abgelehnt
    Anschluss 8080 / 9101                           abgelehnt

Die nip.io-Faelle sind der eigentliche Beleg: ein oeffentlich aufloesbarer Name,
der auf eine private Adresse zeigt, ist der Standardweg um einen Namensfilter
herum -- und faellt hier durch, weil die IP geprueft wird.

Nicht abschliessend geprueft: eine Umleitung, die ins Private zeigt. Mir fehlt
ein oeffentlicher Umleiter, der das tut (httpbingo lehnt es mit 403 ab). Der
Code laeuft die Kette selbst und ruft je Sprung dieselbe Pruefung -- belegt ist
also die Pruefung, nicht die Kette.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
2026-08-12 01:18:27 +02:00
co-authored by Claude Opus 5
parent 463a821199
commit e8efa2431d
14 changed files with 876 additions and 2 deletions
+198
View File
@@ -0,0 +1,198 @@
// Eine Datei von einer fremden Adresse holen.
//
// WOZU. Ein einzelnes Bild von woanders zu uebernehmen hiess bisher:
// herunterladen, im Dateimanager wiederfinden, ins Fenster ziehen. Drei
// Schritte fuer etwas, das der Dienst selbst tun kann.
//
// UND DAS IST DIE GEFAEHRLICHSTE FUNKTION IN DIESEM DIENST, deshalb steht sie
// in einer eigenen Datei. Ein Server, der eine vom Benutzer genannte Adresse
// abruft, ist ein Angriff mit eigenem Namen: SSRF. Wer ihn dazu bringt,
//
// http://127.0.0.1:9101/api/dash/... die eigene API von innen
// http://169.254.169.254/latest/meta/ die Zugangsdaten der Cloud
// http://192.168.2.1/ den Router im Heimnetz
//
// abzurufen, laesst den Dienst Dinge lesen, die von aussen unerreichbar sind --
// und bekommt sie als Datei zurueck, unter einer oeffentlichen Adresse.
//
// Dass hier eine Anmeldung davorsteht, ist KEIN Ersatz fuer die Pruefung. Es
// gibt zwei Konten, eines davon entsteht ueber eine Discord-Rolle, die jemand
// anders vergibt.
import { lookup } from 'node:dns/promises'
import { isIP } from 'node:net'
/** Wie lange der Dienst auf eine fremde Adresse wartet. */
const FRIST_MS = 20_000
/** Wie oft einer Umleitung gefolgt wird.
*
* Und JEDE davon wird neu geprueft -- das ist der Kern. Ein Server kann mit
* 302 auf http://127.0.0.1 zeigen, und `fetch` folgt von sich aus. Deshalb
* steht unten `redirect: 'manual'`: die Kette wird selbst gelaufen, damit
* jeder Halt durch dieselbe Pruefung geht wie der erste.
*/
const UMLEITUNGEN_MAX = 3
export class HolFehler extends Error {}
/** Adressbereiche, die von hier aus niemanden etwas angehen.
*
* Geprueft wird die AUFGELOESTE IP und nicht der Name: `localtest.me` zeigt
* auf 127.0.0.1, und ein eigener DNS-Eintrag kann auf alles zeigen. Ein
* Namensfilter waere ein Filter fuer den, der ihn nicht umgehen will. */
function istPrivat(ip: string): boolean {
if (isIP(ip) === 6) {
const v = ip.toLowerCase()
if (v === '::1' || v === '::') return true
// fc00::/7 (eindeutig lokal) und fe80::/10 (verbindungslokal)
if (/^f[cd]/.test(v) || /^fe[89ab]/.test(v)) return true
// IPv4 in IPv6-Schreibweise: ::ffff:127.0.0.1
const eingebettet = /^::ffff:(\d+\.\d+\.\d+\.\d+)$/.exec(v)
if (eingebettet?.[1]) return istPrivat(eingebettet[1])
return false
}
const t = ip.split('.').map(Number)
if (t.length !== 4 || t.some((n) => !Number.isInteger(n) || n < 0 || n > 255)) return true
const [a, b] = t as [number, number, number, number]
if (a === 0) return true // dieses Netz
if (a === 10) return true // privat
if (a === 127) return true // Rueckschleife
if (a === 169 && b === 254) return true // verbindungslokal UND Cloud-Metadaten
if (a === 172 && b >= 16 && b <= 31) return true // privat
if (a === 192 && b === 168) return true // privat
if (a === 100 && b >= 64 && b <= 127) return true // Provider-internes NAT
if (a >= 224) return true // Rundruf und reserviert
return false
}
/** Eine Adresse pruefen: Schema, Anschluss, aufgeloeste IP. Wirft HolFehler. */
async function pruefeZiel(roh: string): Promise<URL> {
let u: URL
try {
u = new URL(roh)
} catch {
throw new HolFehler('Das ist keine gueltige Adresse.')
}
if (u.protocol !== 'http:' && u.protocol !== 'https:') {
throw new HolFehler(`Nur http und https — "${u.protocol.replace(':', '')}" nicht.`)
}
// Ein Anschluss, der nicht der uebliche ist, deutet fast immer auf einen
// Dienst im eigenen Netz. Die Ausnahme waere ein oeffentlicher Server auf
// einem krummen Port; die ist selten genug, um sie abzulehnen.
const port = u.port ? Number(u.port) : u.protocol === 'https:' ? 443 : 80
if (port !== 80 && port !== 443) {
throw new HolFehler(`Nur die ueblichen Anschluesse 80 und 443 — nicht ${port}.`)
}
const name = u.hostname.replace(/^\[|\]$/g, '')
let adressen: string[]
if (isIP(name)) {
adressen = [name]
} else {
try {
adressen = (await lookup(name, { all: true })).map((a) => a.address)
} catch {
throw new HolFehler(`"${name}" laesst sich nicht aufloesen.`)
}
}
if (adressen.length === 0) throw new HolFehler(`"${name}" hat keine Adresse.`)
// JEDE aufgeloeste Adresse muss oeffentlich sein. Ein Name mit zwei
// Eintraegen -- einer oeffentlich, einer auf 127.0.0.1 -- waere sonst ein
// Wuerfelspiel darueber, welchen Node nimmt.
for (const ip of adressen) {
if (istPrivat(ip)) {
throw new HolFehler(
`"${name}" zeigt auf ${ip} und damit ins eigene Netz. Das geht von hier aus nicht.`,
)
}
}
return u
}
export type Geholt = { data: Buffer; mime: string; name: string; von: string }
/** Die Datei holen. `hoechstens` ist die Grenze in Bytes.
*
* Der Rumpf wird STUECKWEISE gelesen und abgebrochen, sobald die Grenze
* reisst. `content-length` allein zu glauben waere naiv -- die Angabe kommt
* vom fremden Server und muss nicht stimmen. */
export async function holeVonAdresse(roh: string, hoechstens: number): Promise<Geholt> {
let ziel = await pruefeZiel(roh)
let antwort: Response | null = null
for (let sprung = 0; sprung <= UMLEITUNGEN_MAX; sprung++) {
const res = await fetch(ziel, {
redirect: 'manual',
signal: AbortSignal.timeout(FRIST_MS),
headers: { 'User-Agent': 'd4rk_media/1.0 (+holen)' },
}).catch((err) => {
throw new HolFehler(
`Nicht erreichbar: ${err instanceof Error ? (err.cause as { code?: string })?.code ?? err.name : 'unbekannt'}`,
)
})
if (res.status >= 300 && res.status < 400) {
const wohin = res.headers.get('location')
await res.body?.cancel().catch(() => {})
if (!wohin) throw new HolFehler(`Umleitung ohne Ziel (HTTP ${res.status}).`)
if (sprung === UMLEITUNGEN_MAX) {
throw new HolFehler(`Mehr als ${UMLEITUNGEN_MAX} Umleitungen.`)
}
// Neu pruefen. GENAU HIER waere die Luecke, wenn man `fetch` selbst
// folgen liesse.
ziel = await pruefeZiel(new URL(wohin, ziel).href)
continue
}
if (!res.ok) {
await res.body?.cancel().catch(() => {})
throw new HolFehler(`Die Gegenstelle antwortet mit HTTP ${res.status}.`)
}
antwort = res
break
}
if (!antwort) throw new HolFehler('Keine Antwort.')
const angekuendigt = Number(antwort.headers.get('content-length') ?? NaN)
if (Number.isFinite(angekuendigt) && angekuendigt > hoechstens) {
await antwort.body?.cancel().catch(() => {})
throw new HolFehler(`Zu gross: ${angekuendigt} > ${hoechstens} Bytes.`)
}
const stuecke: Buffer[] = []
let gesamt = 0
const leser = antwort.body?.getReader()
if (!leser) throw new HolFehler('Die Antwort hat keinen Rumpf.')
for (;;) {
const { done, value } = await leser.read()
if (done) break
gesamt += value.byteLength
if (gesamt > hoechstens) {
await leser.cancel().catch(() => {})
throw new HolFehler(
`Zu gross: die Gegenstelle schickt mehr als ${hoechstens} Bytes.`,
)
}
stuecke.push(Buffer.from(value))
}
if (gesamt === 0) throw new HolFehler('Die Gegenstelle hat nichts geschickt.')
// Der Name aus dem Pfad, sonst aus dem Hostnamen. Zurechtgebogen wird er
// erst beim Ablegen -- hier soll stehen, was dort stand.
const ausPfad = decodeURIComponent(ziel.pathname.split('/').pop() ?? '')
return {
data: Buffer.concat(stuecke),
mime: (antwort.headers.get('content-type') ?? '').split(';')[0]?.trim() ?? '',
name: ausPfad || ziel.hostname,
von: ziel.href,
}
}
+4
View File
@@ -324,6 +324,10 @@ function decodeSafely(raw: string): string {
* Stylesheet noch Modul. */
const WEB_MIME: Record<string, string> = {
html: 'text/html; charset=utf-8',
// webmanifest steht hier und nicht in der geteilten Tabelle: es ist eine
// Datei der Oberflaeche. Ohne den Eintrag kaeme sie als
// application/octet-stream, und der Browser wuerde sie verwerfen.
webmanifest: 'application/manifest+json; charset=utf-8',
css: 'text/css; charset=utf-8',
js: 'text/javascript; charset=utf-8',
mjs: 'text/javascript; charset=utf-8',
+221
View File
@@ -52,6 +52,7 @@ import {
} from '../settings.js'
import { discordRoutes, redirectUri } from './discord.js'
import { MELDUNG_KEYS, meldungConfig, testMeldung } from '../meldung.js'
import { holeVonAdresse, HolFehler, type Geholt } from '../holen.js'
import { neuerSchluessel } from './freigabe.js'
import {
absolutePath,
@@ -947,6 +948,103 @@ dashRoutes.post('/upload', async (c) => {
*
* Nur lesen, nie schreiben — geschrieben wird an den Stellen, an denen es
* passiert. Ein Verlauf, den man von aussen fuellen kann, ist keiner. */
/** Hochladen, indem der Dienst die Datei SELBST holt.
*
* Ein einzelnes Bild von woanders zu uebernehmen hiess bisher: herunterladen,
* im Dateimanager wiederfinden, ins Fenster ziehen. Drei Schritte fuer etwas,
* das der Dienst selbst tun kann.
*
* Die Pruefung der Adresse steht in `holen.ts` und ist dort ausfuehrlich
* begruendet -- ein Server, der eine vom Benutzer genannte Adresse abruft, ist
* ein Angriff mit eigenem Namen. Hier bleibt nur das Ablegen, und zwar durch
* DENSELBEN Weg wie jeder andere Upload: gleiche Namensregeln, gleiche
* Vorschau, gleicher Eintrag im Verlauf. */
dashRoutes.post('/upload-url', async (c) => {
const body = await jsonBody<{ url?: string; ordner?: string; name?: string }>(c)
if (!body?.url?.trim()) return c.json({ error: 'Keine Adresse angegeben' }, 400)
let ordner = ''
if (body.ordner?.trim()) {
try {
ordner = checkPath(body.ordner.trim().replace(/^\/+|\/+$/g, ''))
} catch (err) {
if (err instanceof PathError) return c.json({ error: `Ordner: ${err.message}` }, 400)
throw err
}
}
let geholt: Geholt
try {
geholt = await holeVonAdresse(body.url.trim(), config.maxUploadBytes)
} catch (err) {
// HolFehler traegt einen Satz, den man lesen kann -- alles andere
// koennte eine Innenansicht sein und bekommt deshalb einen eigenen.
if (err instanceof HolFehler) return c.json({ error: err.message }, 400)
console.error('[holen]', err)
return c.json({ error: 'Die Adresse liess sich nicht abrufen.' }, 502)
}
const digest = sha256(geholt.data)
// Der gewuenschte Name schlaegt den aus der Adresse. Gezaehmt wird beides.
const roh = body.name?.trim() || geholt.name
const dateiname = safeFilename(roh, digest.slice(0, 8))
let path: string
try {
path = checkPath(ordner ? `${ordner}/${dateiname}` : dateiname)
} catch (err) {
if (err instanceof PathError) return c.json({ error: err.message }, 400)
throw err
}
// Was die Gegenstelle ueber die Art sagt, gilt -- aber nicht ihr
// Achselzucken. Dieselbe Regel wie beim Formular-Upload.
const mime =
geholt.mime && geholt.mime !== 'application/octet-stream' ? geholt.mime : mimeFor(path)
const vorhanden = db.prepare('SELECT * FROM media WHERE path = ?').get(path) as
| Media
| undefined
await writeFileAtomic(path, geholt.data)
if (kannVorschau(mime)) await writeThumb(path, geholt.data)
if (lohntWebp(mime)) await writeWebp(path, geholt.data)
let dauer: number | null = null
if (kannBewegtVorschau(mime)) dauer = (await writeBewegtThumb(path, mime)).dauer
const zeit = now()
if (vorhanden) {
db.prepare(
`UPDATE media SET size = ?, sha256 = ?, mime = ?, dauer = ?, updated_at = ?
WHERE id = ?`,
).run(geholt.data.length, digest, mime, dauer, zeit, vorhanden.id)
} else {
db.prepare(
`INSERT INTO media (path, size, sha256, mime, dauer, token_id, created_at, updated_at)
VALUES (?, ?, ?, ?, ?, NULL, ?, ?)`,
).run(path, geholt.data.length, digest, mime, dauer, zeit, zeit)
}
logEvent(
vorhanden ? 'replace' : 'upload',
path,
geholt.data.length,
'user',
c.get('user').username,
)
return c.json({
path,
url: publicUrlFor(path),
size: geholt.data.length,
mime,
replaced: Boolean(vorhanden),
// Woher sie kam -- nach Umleitungen ist das nicht mehr die Adresse, die
// eingetippt wurde, und das soll man sehen.
von: geholt.von,
})
})
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)
@@ -2247,6 +2345,129 @@ dashRoutes.post('/tokens', async (c) => {
})
})
/** Die Grenzen eines Tokens nachtraeglich aendern.
*
* WARUM ES DAS BRAUCHT: Kontingent, Arten, Ablauf und Praefix liessen sich nur
* beim ANLEGEN setzen. Wer einem bestehenden Token nachtraeglich eine Grenze
* geben wollte, musste ihn neu anlegen -- und damit den Schluessel in jedem
* Skript tauschen, das ihn benutzt. Fuer eine Zahl, die sich in der Datenbank
* aendern laesst, ist das der falsche Preis.
*
* DER SCHLUESSEL SELBST BLEIBT UNBERUEHRT, und zwar nicht aus Vorsicht: in der
* Tabelle steht nur sein Hash. Wir koennten ihn gar nicht aendern, ohne einen
* neuen auszugeben -- und genau das waere dann kein Bearbeiten mehr.
*
* Jedes Feld ist einzeln, und WEGGELASSEN heisst UNVERAENDERT. `null` dagegen
* heisst ausdruecklich "keine Grenze" -- ohne diesen Unterschied liesse sich
* eine einmal gesetzte Grenze nie wieder loeswerden. */
dashRoutes.patch('/tokens/:id', async (c) => {
const id = Number(c.req.param('id'))
const token = db.prepare('SELECT * FROM tokens WHERE id = ?').get(id) as Token | undefined
if (!token) return c.json({ error: 'nicht gefunden' }, 404)
const body = await jsonBody<{
name?: string
prefix?: string
canDelete?: boolean
maxMb?: number | string | null
quotaMb?: number | string | null
arten?: string[] | null
gueltigTage?: number | string | null
}>(c)
if (!body) return c.json(KEIN_JSON, 400)
const feld: Record<string, unknown> = {}
if (body.name !== undefined) {
if (!body.name.trim()) return c.json({ error: 'Der Name darf nicht leer sein' }, 400)
feld.name = body.name.trim()
}
if (body.prefix !== undefined) {
const roh = body.prefix.trim().replace(/^\/+|\/+$/g, '')
if (roh) {
try {
feld.prefix = checkPath(roh)
} catch (err) {
if (err instanceof PathError) return c.json({ error: `Praefix: ${err.message}` }, 400)
throw err
}
} else {
feld.prefix = ''
}
}
if (body.canDelete !== undefined) feld.can_delete = body.canDelete ? 1 : 0
// -- Die Zahlen. null loescht die Grenze, eine Zahl setzt sie. ----------
const zahl = (wert: unknown, was: string, deckel?: number) => {
if (wert === null || String(wert).trim() === '') return null
const n = Number(wert)
if (!Number.isFinite(n) || n <= 0) throw new Error(`${was}: eine Zahl groesser als 0`)
const bytes = Math.round(n * 1024 * 1024)
if (deckel && bytes > deckel) {
throw new Error(
`Der Dienst nimmt hoechstens ${Math.floor(deckel / 1024 / 1024)} MB — mehr kann auch dieser Token nicht.`,
)
}
return bytes
}
try {
if (body.maxMb !== undefined) {
feld.max_bytes = zahl(body.maxMb, 'Groessengrenze', config.maxUploadBytes)
}
if (body.quotaMb !== undefined) feld.quota_bytes = zahl(body.quotaMb, 'Kontingent')
} catch (err) {
return c.json({ error: err instanceof Error ? err.message : 'ungueltige Zahl' }, 400)
}
if (body.arten !== undefined) {
const ERLAUBT = ['bild', 'video', 'ton', 'dokument', 'andere']
const liste = (body.arten ?? []).filter((a) => ERLAUBT.includes(a))
// Alle anzukreuzen ist dasselbe wie keine Einschraenkung -- und als
// leeres Feld gespeichert liest es sich spaeter auch so.
feld.arten = liste.length === 0 || liste.length === ERLAUBT.length ? '' : liste.join(',')
}
if (body.gueltigTage !== undefined) {
if (body.gueltigTage === null || String(body.gueltigTage).trim() === '') {
feld.expires_at = null
} else {
const tage = Number(body.gueltigTage)
if (!Number.isFinite(tage) || tage <= 0) {
return c.json({ error: 'Gueltigkeit: eine Zahl groesser als 0' }, 400)
}
// AB JETZT und nicht ab dem Anlegen: "noch 30 Tage" ist das, was
// jemand meint, der das Feld heute anfasst.
feld.expires_at = now() + tage * 86_400_000
}
}
const spalten = Object.keys(feld)
if (spalten.length === 0) return c.json({ error: 'nichts zu aendern' }, 400)
// Die Spaltennamen kommen aus DIESER Funktion und nie vom Aufrufer -- der
// liefert nur die Werte, und die gehen als Parameter hinein.
db.prepare(`UPDATE tokens SET ${spalten.map((s) => `${s} = ?`).join(', ')} WHERE id = ?`).run(
...spalten.map((s) => feld[s]),
id,
)
const neu = db
.prepare(
`SELECT id, name, prefix, can_delete, created_at, last_used_at,
max_bytes, quota_bytes, arten, expires_at
FROM tokens WHERE id = ?`,
)
.get(id) as Omit<Token, 'hash'>
return c.json({
...neu,
abgelaufen: Boolean(neu.expires_at && neu.expires_at < now()),
geaendert: spalten,
})
})
dashRoutes.delete('/tokens/:id', (c) => {
const info = db.prepare('DELETE FROM tokens WHERE id = ?').run(Number(c.req.param('id')))
if (info.changes === 0) return c.json({ error: 'nicht gefunden' }, 404)