[BUG]: Proxy startup may report success when retries are exhausted and process does not stabilize
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 45/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- typescript
- Bereich
- ci-cd
Rechercherichtung
Beginne mit start-proxy-action.ts und verfolge die Pfade für Wiederholungen, vorzeitiges Beenden, Stabilisierungsverzögerung und Ausgaben. Bestätige, wie das Ausschöpfen der Wiederholungsversuche und der letzte Exit-Code dargestellt werden, und definiere Tests für endgültiges Fehlschlagen, Prozesslebensfähigkeit und Wiederholungen mit einem neuen ephemeren Port; abgeschlossen ist die Aufgabe, wenn ein instabiler Start explizit fehlschlägt, ohne bestehende Ausgaben, Telemetrie oder Wrapper-Semantik zu ändern.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Problem
The proxy startup logic in start-proxy-action.ts can report successful initialization even when the proxy process fails to stabilize after retry attempts.
The previous implementation:
- Retries on spawn or early exit
- Only throws on immediate spawn errors
- Does not explicitly fail when all retry attempts are exhausted
- May proceed with outputs set despite no stable process remaining alive
This can lead to false-positive startup.
Impact
When startup silently fails:
- Downstream steps may fail with opaque dependency or networking errors
- Failures manifest later in the workflow
- Root cause becomes difficult to diagnose
Instead of failing fast during initialization, workflows continue in a degraded state.
Expected Behavior
If the proxy fails to stabilize after retry attempts, startup should fail explicitly with a clear diagnostic error.
Proposed Solution
- Detect retry exhaustion explicitly
- Throw a clear error including the last exit code
- Verify process remains alive after stabilization delay
- Retry using a new ephemeral port before terminal failure
- Extract startup logic into a dedicated launcher module for testability
This preserves existing outputs, telemetry behavior, and wrapper semantics while preventing silent failure.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 1.6k
- Forks
- 493
- Ø Merge
- 1 T. 13 Std.
- Gemergte PRs (30 T.)
- 44
Beitragsleitfaden
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 github/codeql-action
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
github/codeql-action#4052 · 4 Kommentare ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
github/codeql-action#4078 · 1 Kommentar ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
github/codeql-action#4008 · 9 Kommentare ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 52/100
github/codeql-action#3978 · 4 Kommentare · 1 Reaktion ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
github/codeql-action#3915 · 6 Kommentare · 3 Reaktionen ·
Alle Issues in github/codeql-action
Ähnliche Issues
-
VerificationGate: ATTRIBUTION quote guard never matches a normal quotation (\b around the quote) Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
danielmiessler/LifeOS#2234 ·
-
T: Bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
-
Mend: dependency security vulnerability untriaged
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100