Skip to content

fix(relay): advertise NIP-45 COUNT in SUPPORTED_NIPS (#7432) - #7434

Open
bounce12340 wants to merge 1 commit into
block:mainfrom
bounce12340:fix/7432-advertise-nip45
Open

fix(relay): advertise NIP-45 COUNT in SUPPORTED_NIPS (#7432)#7434
bounce12340 wants to merge 1 commit into
block:mainfrom
bounce12340:fix/7432-advertise-nip45

Conversation

@bounce12340

Copy link
Copy Markdown

Summary

  • Add 45 to SUPPORTED_NIPS in crates/buzz-relay/src/nip11.rs so NIP-11 advertises COUNT.
  • COUNT is already implemented (handlers/count.rs); clients checking NIP-11 were incorrectly skipping it.
  • Add unit test supported_nips_includes_nip45.

Closes #7432

Test plan

  • cargo test -p buzz-relay supported_nips_includes_nip45
  • Confirm NIP-11 document lists 45 alongside existing NIPs

Signed-off-by: Josh Tsai <bounce12340@users.noreply.github.com>
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

🔐 Codex Security Review

Status: review required for the current range.

The current range is 3c7f288c60d67df78577b237e27c3dfc8831aaa1...82de18ebc1fe4b71e884c5d3b2ba214acd24375c.
A new review must complete for this exact range. When manual authorization
is required, a Block organization member must comment exactly
@buzz-security-review 82de18ebc1fe4b71e884c5d3b2ba214acd24375c to authorize a new review.
Any previous review applies only to its recorded range.

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.

relay: SUPPORTED_NIPS omits NIP-45 (COUNT) despite an implemented handler

1 participant