Skip to content

docs: refresh rust-nostr NIP-46 compat note + link filed upstream issue#56

Open
DocNR wants to merge 1 commit into
mainfrom
docs/nip46-compat-rust-nostr-issue
Open

docs: refresh rust-nostr NIP-46 compat note + link filed upstream issue#56
DocNR wants to merge 1 commit into
mainfrom
docs/nip46-compat-rust-nostr-issue

Conversation

@DocNR
Copy link
Copy Markdown
Owner

@DocNR DocNR commented May 14, 2026

Summary

  • Refreshes the rust-nostr family note in docs/nip46-compatibility.md with the verified state of rust-nostr master 0a030e1c7 — the response parser and the nostrconnect:// path now accept the echoed-secret connect response, but the bunker:// connect() path still rejects it via res.to_ack()?.
  • Documents the secondary swapped-label bug in Error::UnexpectedResponse's Display impl.
  • Filed the bug upstream as rust-nostr/nostr#1352 and linked it in the open-upstream-issues table; bumped the Last updated date + changelog.

Context

A Flare iOS user reported a bunker:// pairing failure that looked like a Clave spec violation. It's actually a rust-nostr bug (Flare uses nostr-sdk-kmp, which fully delegates NIP-46 to rust-nostr): rust-nostr's bunker:// connect path rejects the spec-allowed echoed-secret response. Clave's behavior is NIP-46 compliant and unchanged — this PR is docs-only.

Test plan

🤖 Generated with Claude Code

The rust-nostr family note was stale (dated 2026-04-22, "not yet filed").
Updated it with the verified state of rust-nostr master 0a030e1c7: the
response parser and the nostrconnect:// path now accept the echoed-secret
connect response, but the bunker:// path (connect() -> to_ack()) still
rejects it. Also documents the swapped expected/received labels in the
UnexpectedResponse Display impl. Filed upstream as rust-nostr/nostr#1352
and linked it in the open-upstream-issues table.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

1 participant