-
Notifications
You must be signed in to change notification settings - Fork 0
Add bounded timeouts and safe retry rules to release HTTP operations #51
Copy link
Copy link
Open
Labels
P4Low: minor issue, maintenance work, or non-runtime improvementLow: minor issue, maintenance work, or non-runtime improvementarea:compatibilityCross-mod, version, or platform compatibilityCross-mod, version, or platform compatibilityloader:commonShared Common implementation or assetsShared Common implementation or assetsstatus:backlogValid work that is not currently scheduledValid work that is not currently scheduledtype:maintenanceTests, dependencies, build tooling, or repository upkeepTests, dependencies, build tooling, or repository upkeep
Description
Metadata
Metadata
Assignees
Labels
P4Low: minor issue, maintenance work, or non-runtime improvementLow: minor issue, maintenance work, or non-runtime improvementarea:compatibilityCross-mod, version, or platform compatibilityCross-mod, version, or platform compatibilityloader:commonShared Common implementation or assetsShared Common implementation or assetsstatus:backlogValid work that is not currently scheduledValid work that is not currently scheduledtype:maintenanceTests, dependencies, build tooling, or repository upkeepTests, dependencies, build tooling, or repository upkeep
Summary
GitHub, Modrinth, and CurseForge publication requests use urllib.request.urlopen(...) without explicit timeouts.
Evidence
Impact
A stalled TLS connection or response can occupy a release runner until the much larger workflow timeout and leave publication state unclear.
Reproduction / verification
Proposed direction
Add bounded connection/response deadlines, retry only safe idempotent reads automatically, and reconcile uncertain mutation outcomes before retrying.
Acceptance criteria
Audit provenance
Candidate #30 from the read-only repository audit performed on 2026-07-20. This is separate from existing issues #3, #4, #25, and #27 unless explicitly linked later.