fix(pr-pipeline): scope read-runner caller secrets per caller in check-read-caller-keys
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 74/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- github-actions, javascript
Rechercherichtung
Beginne mit scripts/check-read-caller-keys.mjs und untersuche, wie das Skript die Workflows der Aufrufer einliest und ALLOWED_SECRETS validiert; verwende die bestehende Behandlung des Automations-Tokens als Referenz. Füge Tests für das Skript hinzu oder finde sie und decke den eigenen Secret jedes Aufrufers, das Secret eines anderen Lanes und einen Aufrufer ab, der in der Zuordnung fehlt. Fertig ist die Arbeit, wenn sich alle drei Fälle wie angegeben verhalten und die Prüfungen bestehen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Problem
scripts/check-read-caller-keys.mjs holds the allowed secret names for read-runner callers in one global set, ALLOWED_SECRETS (line 30 after #8691). So any caller of pr-run-activity-read.yml may pass any allowed lane credential, for example issue_lane_claude_code_oauth_token from a caller that is not the issue lane, and the check stays green. The automations token already works this way.
Abusing this needs a workflow change that is already on main, so it is a hardening item, not a live gap.
Fix
Replace the global set with a per-caller map from caller workflow file to the secret names it may pass, for example work-resolve-issue.yml → the ISSUE_LANE_ names and pr-unblock.yml → the AUTOMATIONS_ names. A caller that passes a secret outside its own entry fails the check. Add tests:
- each caller's own secret passes;
- another lane's secret fails;
- a caller missing from the map fails.
Refs: #8691
- Vorherrschende Sprache
- Shell
- Sterne
- 22
- Forks
- 2
- Ø Merge
- 6 Std. 11 Min.
- Gemergte PRs (30 T.)
- 955
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus melodic-software/claude-code-plugins
-
needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
melodic-software/claude-code-plugins#9619 ·
Maintainer antworten meist innerhalb von 1 Tag
-
priority: medium status: ready
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
melodic-software/claude-code-plugins#9617 ·
Maintainer antworten meist innerhalb von 1 Tag
-
evals: run-validity calls runs VALID when every Bash call failed to start the sandbox (long TMPDIR)Offenpriority: medium status: needs-decision work-class: scoped
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
melodic-software/claude-code-plugins#9611 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
fix(pr-pipeline): apply-verdict readFields reads a malformed field-values answer as no fields setOffenpriority: low status: confirmed
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 83/100
melodic-software/claude-code-plugins#9598 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
priority: medium status: needs-decision work-class: scoped
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
melodic-software/claude-code-plugins#9578 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in melodic-software/claude-code-plugins
Ähnliche Issues
-
[Request] Add BBportOffenenhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Leonxlnx/taste-skill#164 ·
Maintainer antworten meist innerhalb von 1 Tag
-
patch: moving an executable script removes its executable bitEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offenreproduced
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
anomalyco/opencode#58651 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
使用脚本 DD 腾讯云 报错Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100