Skip to content

Conversation

@Andrurachi
Copy link

Issue Addressed

Closes #8569

Proposed Changes

Updates the HTTP API error when the node cannot reconstruct blobs due to "Insufficient data columns".

Changes the response from 500 Internal Server Error to 400 Bad Request and adds a hint to run with --supernode or --semi-supernode.

@Andrurachi Andrurachi force-pushed the fix-improve-error-blob-reconstruction-8569 branch from 92f12db to 7888f89 Compare December 16, 2025 20:06
Updates the HTTP API error when the node cannot reconstruct blobs due to
"Insufficient data columns".

Changes the response from 500 Internal Server Error to 400 Bad Request
and adds a hint to run with --supernode or --semi-supernode. Also updates
the corresponding test.

Closes sigp#8569
@Andrurachi Andrurachi force-pushed the fix-improve-error-blob-reconstruction-8569 branch from 7888f89 to 4ddb3c5 Compare December 16, 2025 20:07
@chong-he chong-he added the ready-for-review The code is ready for review label Dec 17, 2025
Copy link
Member

@chong-he chong-he left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, looks good

@jimmygchen jimmygchen added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Jan 5, 2026
@mergify mergify bot added the queued label Jan 5, 2026
@mergify
Copy link

mergify bot commented Jan 5, 2026

Merge Queue Status

✅ The pull request has been merged at 4ddb3c5

This pull request spent 39 minutes 49 seconds in the queue, including 38 minutes 37 seconds running CI.
The checks were run on draft #8622.

Required conditions to merge
  • check-success=local-testnet-success
  • check-success=test-suite-success

mergify bot added a commit that referenced this pull request Jan 5, 2026
@mergify mergify bot merged commit ea3a3da into sigp:unstable Jan 5, 2026
36 checks passed
@mergify mergify bot removed the queued label Jan 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge This PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants