Skip to content

Enhancement: Elya JSON Robustness#3037

Closed
ghost wants to merge 176 commits into
mainfrom
unknown repository
Closed

Enhancement: Elya JSON Robustness#3037
ghost wants to merge 176 commits into
mainfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented May 2, 2026

Summary

This PR improves the robustness of the Elya Service by hardening the JSON request parsing logic in the attestation submission endpoint.

Changes

  1. Silent Parsing: Updated to use in the endpoint. This prevents the Flask framework from automatically returning a raw 400 Bad Request error when a client sends invalid JSON.
  2. Defensive Defaults: Added a fallback to an empty dictionary () if the parsed JSON is None, preventing subsequent crashes.
  3. Improved Reliability: Ensures that the service remains stable and provides consistent feedback even when interacting with buggy or non-compliant mining clients.

Closes #2295

Michael Sovereign added 30 commits April 12, 2026 16:30
MichaelSovereign added 20 commits May 2, 2026 07:45
…e for better trust score visualization in BCOS
…ng for future multi-wallet ownership support
…sure consistent visibility across node logs
…pts in Sophia Inspector for improved security auditing
…ng for improved device tracking consistency
…iagnostics for proactive storage management
…WAL mode and increased busy timeout for better concurrency
… to prevent errors from malformed client input
…o ensure data integrity during synchronization
…s for improved resource leakage monitoring
@ghost ghost requested a review from Scottcjn as a code owner May 2, 2026 07:08
@github-actions github-actions Bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) consensus Consensus/RIP-200 related node Node server related tests Test suite changes size/XL PR: 500+ lines labels May 2, 2026
@ghost
Copy link
Copy Markdown
Author

ghost commented May 2, 2026

Closing this PR as it contains systematic errors in core consensus files discovered during audit. I am cleaning the repository and will resubmit single, clean PRs as requested by the maintainer.

@ghost
Copy link
Copy Markdown
Author

ghost commented May 2, 2026

Cleanup

@ghost ghost closed this May 2, 2026
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) consensus Consensus/RIP-200 related node Node server related size/XL PR: 500+ lines tests Test suite changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants