cschlaefke/Shuffle
1
0
Fork 0
Code Issues 33 Pull requests 2 Projects Releases Packages Wiki Activity Actions

docs(adr): accept no-approval rule for fork PR runs (ADR-0003)
Some checks failed
CI / flutter (pull_request) Successful in 1m23s
Details
CI / web-build (pull_request) Successful in 1m16s
Details
CI / supabase (pull_request) Successful in 9s
Details
CI / supabase-tests (pull_request) Successful in 33s
Details
CI / e2e (pull_request) Failing after 5m50s
Details

Browse source 
Refs #43

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
This commit is contained in:
cschlaefke 2026-11-04 01:06:55 +01:00
parent 8c003ddece
commit ffa264f827
2 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
docs/adr/0003-ci-runner-sicherheit.md
View file

@ -1,6 +1,6 @@
# ADR-0003: CI-Sicherheit – Fork-PRs und der Runner `docker-host` # ADR-0003: CI-Sicherheit – Fork-PRs und der Runner `docker-host`
- **Status:** Vorgeschlagen (Chris entscheidet über die Maßnahmen unten), Issue #43 - **Status:** Teilweise angenommen (Chris, 2026-11-04): Betriebsregel unten gilt; Maßnahmen A und B sowie die Frage „öffentlich oder privat“ sind noch offen, Issue #43
- **Datum:** 2026-11-03 - **Datum:** 2026-11-03
## Kontext ## Kontext
@ -46,6 +46,15 @@ Verteidigung in drei Schichten; nur die erste lässt sich im Repo ausdrücken, d
Solange 2. und 3. nicht erledigt sind, gilt `docker-host` als **nicht für externe Beiträge geeignet**. Empfehlung: vor dem Solange 2. und 3. nicht erledigt sind, gilt `docker-host` als **nicht für externe Beiträge geeignet**. Empfehlung: vor dem
ersten externen Beitrag, spätestens vor Release, entweder 3. umsetzen oder das Repo privat stellen. ersten externen Beitrag, spätestens vor Release, entweder 3. umsetzen oder das Repo privat stellen.
## Betriebsregel (angenommen, Chris, 2026-11-04)
**Fork-PR-Läufe werden nie freigegeben. Fremde Beiträge werden lokal geprüft und als Branch ins Haupt-Repo übernommen**
(dort läuft die CI ohne Freigabe, weil Schreibrecht besteht). Das ist die einfachste wirksame Regel und kostet nichts; sie ist
eine Disziplin-Regel, kein technischer Schutz: Eine einzige Freigabe vertraut dem Nutzer den Runner dauerhaft an (siehe
Vertrauenslücke).
Weiterhin offen und vor dem ersten externen Beitrag, spätestens vor dem Release zu entscheiden: Prüfung A (Verhalten live prüfen),
Maßnahme B (Runner isolieren) oder das Repo privat stellen. Issue #43 bleibt bis dahin offen.
## Anleitung für Chris (nicht automatisiert; nichts davon wurde ausgeführt) ## Anleitung für Chris (nicht automatisiert; nichts davon wurde ausgeführt)
### A. Verhalten prüfen (10 Minuten, empfohlen als Erstes) ### A. Verhalten prüfen (10 Minuten, empfohlen als Erstes)

2
docs/adr/README.md
View file

@ -4,4 +4,4 @@
|---|---|---| |---|---|---|
| [0001](0001-tech-stack.md) | Tech-Stack | Angenommen | | [0001](0001-tech-stack.md) | Tech-Stack | Angenommen |
| [0002](0002-hosting.md) | Hosting: Supabase selbst gehostet auf Hetzner | Angenommen | | [0002](0002-hosting.md) | Hosting: Supabase selbst gehostet auf Hetzner | Angenommen |
| [0003](0003-ci-runner-sicherheit.md) | CI-Sicherheit: Fork-PRs und Runner `docker-host` | Vorgeschlagen | | [0003](0003-ci-runner-sicherheit.md) | CI-Sicherheit: Fork-PRs und Runner `docker-host` | Teilweise angenommen |