Skip to content

Advertise NIP-FI discovery in relay NIP-11 - #7716

Draft
bradseiler wants to merge 2 commits into
mainfrom
coder3/nip-fi-enterprise-auth-ad
Draft

bradseiler wants to merge 2 commits into
mainfrom
coder3/nip-fi-enterprise-auth-ad

Conversation

@bradseiler

Copy link
Copy Markdown
Contributor

Summary

This PR lets the relay advertise a NIP-FI federated identity discovery signal in NIP-11 without enabling relay-side admission enforcement. It adds a relay-local off / shadow config mode, emits the privacy-safe discovery shape only in shadow, and rejects enforce at startup until validation and protected-ingress checks exist.

The advertised wire shape contains only:

{
  "limitation": { "federated_identity": true },
  "federated_identity": {
    "core": "client-attached",
    "assertion_freshness": {
      "class": "offline-jwt",
      "maximum_residual_upstream_revocation_seconds": null
    }
  }
}

No issuer, audience, JWKS, identity provider, login, tenant, assertion, or deployment-local details are included.

Related issue

N/A

Testing

  • cargo fmt --check — passed
  • cargo test -p buzz-relay --lib nip_fi — passed, 8 tests
  • cargo test -p buzz-relay --lib -- --skip api::mesh_demo::tests::demo_join_forwarded_arm_round_trips_echo — passed, 1063 tests; skipped one existing relay mesh demo failure reproduced on main
  • cargo clippy -p buzz-relay --lib -- -D warnings — passed
  • /opt/homebrew/Cellar/helm/4.1.4/bin/helm lint deploy/charts/buzz — passed
  • HELM_DATA_HOME=/tmp/helm-data-buzz /opt/homebrew/Cellar/helm/4.1.4/bin/helm unittest deploy/charts/buzz — passed, 10 suites and 52 tests

Pre-push hook ran with default repository lanes and failed in unrelated buzz-acp timing tests. I pushed with --no-verify after the relay and chart checks above passed.

Generated with Codex

coder 3 added 2 commits September 17, 2026 16:37
Signed-off-by: coder 3 <3bc2c73cf6228485ecee3c37d374aa143eab158665389f5168c4cbbf82ef7018@buzz.block.builderlab.xyz>
Signed-off-by: coder 3 <3bc2c73cf6228485ecee3c37d374aa143eab158665389f5168c4cbbf82ef7018@buzz.block.builderlab.xyz>
@github-actions

Copy link
Copy Markdown

🔐 Codex Security Review

Status: review required for the current range.

The current range is 01b6174a1cbad249e93f31df97d4b2ed1d0e8638...5252c876ba01b7f9d5ab1b640a87d04d383e90c2.
A new review must complete for this exact range. When manual authorization
is required, a Block organization member must comment exactly
@buzz-security-review 5252c876ba01b7f9d5ab1b640a87d04d383e90c2 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.

1 participant