Skip to content

paths: reject incomplete remote PoRep vanilla responses - #1539

Open
hyunmoon wants to merge 1 commit into
filecoin-project:mainfrom
hyunmoon:codex/ten-pr-q9-20260913
Open

hyunmoon wants to merge 1 commit into
filecoin-project:mainfrom
hyunmoon:codex/ten-pr-q9-20260913

Conversation

@hyunmoon

@hyunmoon hyunmoon commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Do not accept incomplete PoRep vanilla input merely because the HTTP status is 200. If reading the response body fails, record the location's error and try another location. Scheduler, failure-budget and GPU/backend policies are unchanged.

Validation and limits

Production-path HTTP fixtures cover partial bodies, fallback, success and error propagation; normal/race, build, vet, pinned lint and formatting checks passed. Two unrelated tests with fallback database connections were excluded. These transport fixtures do not exercise native PoRep or reproduce a production partial-response incident.

Series overview. Independently reviewable; no dependency on the foundation PR.

@FilOzzy FilOzzy added this to FOC Sep 14, 2026
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FOC Sep 14, 2026
@hyunmoon
hyunmoon marked this pull request as ready for review September 14, 2026 10:37
@hyunmoon
hyunmoon requested review from a team and ZenGround0 as code owners September 14, 2026 10:37
@FilOzzy FilOzzy moved this from 📌 Triage to 🔎 Awaiting review in FOC Sep 14, 2026
@github-project-automation github-project-automation Bot moved this from 🔎 Awaiting review to ✔️ Approved by reviewer in FOC Sep 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants