Skip to content

security: enforce /relay/ping pubkey binding and nonce replay rejection (Issue #48)#70

Closed
AdnanMehr8 wants to merge 8 commits intoScottcjn:mainfrom
AdnanMehr8:fix/issue-48-security-harden
Closed

security: enforce /relay/ping pubkey binding and nonce replay rejection (Issue #48)#70
AdnanMehr8 wants to merge 8 commits intoScottcjn:mainfrom
AdnanMehr8:fix/issue-48-security-harden

Conversation

@AdnanMehr8
Copy link
Contributor

This PR provides a clean, conflict-free implementation of the security fixes requested in Issue #48:

  1. Nonce replay protection: Implemented using a SQLite-backed window.
  2. Pubkey identity binding: Enforces that heartbeats match the registered key for bcn_ agents.
  3. SwarmHub Compatibility: Includes the fix to avoid rejecting SwarmHub agents.
  4. Input validation: Hardened hex/length checks on registration paths.

Replaces dirty PR #65.

@AdnanMehr8 AdnanMehr8 requested a review from Scottcjn as a code owner February 27, 2026 20:36
@github-actions github-actions bot added the size/XL PR: 500+ lines label Feb 27, 2026
@Scottcjn
Copy link
Owner

Closing as duplicate — PR #65 by autonomy414941 addresses the same issue (#48) and was submitted first. Thanks for the contribution.

@Scottcjn Scottcjn closed this Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XL PR: 500+ lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants