Skip to content

Enhancement: WebSocket Payload Validation#2974

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

Enhancement: WebSocket Payload Validation#2974
ghost wants to merge 114 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 WebSocket event handlers by adding basic input validation.

Changes

  1. Payload Type Checking: Added a check to the event to ensure that if any data is provided by the client, it must be a dictionary. This prevents potential errors or unexpected behavior if a client sends a malformed or non-standard ping payload.
  2. Standardization: Follows the pattern of defensive programming used in other API endpoints.

Closes #2295

Michael Sovereign added 30 commits April 12, 2026 16:30
MichaelSovereign added 20 commits May 2, 2026 07:08
…event connectivity issues with outdated certs
…o prevent rewards from known abusive addresses
@ghost ghost requested a review from Scottcjn as a code owner May 2, 2026 06:19
@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
@Scottcjn
Copy link
Copy Markdown
Owner

Scottcjn commented May 3, 2026

Closing as part of Tier 0 hard-ban cleanup — see #3074 / #3104 / #3169 for the documented incident chain. All MichaelSovereign PRs are closed unread per the Tier 0 contract. No review path; no future PRs from this account will be processed. (See feedback_michaelsovereign_tier0_2026-05-02.md.)

@Scottcjn Scottcjn closed this May 3, 2026
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.

1 participant