diff options
| author | Christophe Besson <cbesson@gmail.com> | 2026-08-15 02:34:19 +0200 |
|---|---|---|
| committer | Christophe Besson <cbesson@gmail.com> | 2026-08-15 02:34:19 +0200 |
| commit | 84b032c65e17267d41e04605e79eea82a6f5a59f (patch) | |
| tree | eb7708a72944bb15540e103b4319242199af6fbb /docs/invite-pairing-v1.md | |
| parent | 5338894f7fec9e1a60affb0e2ff3b9797bcbc968 (diff) | |
| download | meshbay-84b032c65e17267d41e04605e79eea82a6f5a59f.tar.gz | |
feat(groups): editable description, and one source of operator authority
A description could only be set the moment a group was created, so every
group made before anyone thought of one stayed blank for good. The owner
can now edit it from the group's page, and PATCH /v1/groups/{id} takes it.
That endpoint takes the description and nothing else, deliberately. The
name, the visibility and the join policy are the terms members joined on;
a private group that can quietly become public is not the group they
agreed to be in. Changing those needs a decision about who gets told, not
a field on a form — there is a test saying so.
Separately, the legacy operator key is gone. `admin_pk_ed25519` in
node.toml named the operator before the roster existed and was kept so
that an existing deployment would keep working; nothing uses it, and a
second source of node authority is not something to carry around out of
politeness. Authority is the roster, read fresh on every check.
It is removed rather than ignored: a config that still names the key gets
a warning at startup pointing at the file. Dropping it in silence would
refuse invites and file deletion with a signature error that looks like a
bug somewhere else — which is exactly how finding M3 presented.
Two tests were verifying admin operations by naming a key in the context,
which was the legacy path. They now pair an operator into a roster, the
way an operator does. The authority test anchored on the deleted function
and passed vacuously once it disappeared; it states the invariant against
the verifier and the daemon instead.
Also defined .btn-secondary, used in four places and styled in none.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Diffstat (limited to 'docs/invite-pairing-v1.md')
| -rw-r--r-- | docs/invite-pairing-v1.md | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/docs/invite-pairing-v1.md b/docs/invite-pairing-v1.md index 290c756..ad36a1a 100644 --- a/docs/invite-pairing-v1.md +++ b/docs/invite-pairing-v1.md @@ -96,8 +96,11 @@ node writes the pin to its roster DB, prints it in `status` The operator types 8 characters into their own browser. Nothing is pasted, nothing is copied out of a terminal, no browser is needed on the node host, and the hub is -not involved at any point. `admin_pk_ed25519` in `node.toml` becomes a legacy -fallback, still read, no longer required. +not involved at any point. `admin_pk_ed25519` in `node.toml` became a legacy +fallback, and was removed on 2026-08-15: the roster is the only source of operator +authority. A config that still names the key is warned about at startup, so a +deployment relying on it learns why its invites stopped rather than discovering a +signature error. ### 3.2 Invite (one click, operator or delegate) @@ -357,7 +360,7 @@ is a roster row and a CLI command — no protocol change, no migration. | Two people race one code | Single-use row, `used_at` set under a transaction; the loser gets a plain refusal | | Invite created while the node is offline | Not possible — invites are created on the node. The SPA must say "node offline, cannot invite" instead of failing obscurely | | Member connects while the group has no active GEK | `join_result {ok: false, reason: "no_gek"}`; the operator runs `gek-init` | -| Legacy deployment with `admin_pk_ed25519` set | Read at startup and inserted as an `identities` row with `pinned_via = 'legacy-config'`; no migration needed for the current demo | +| Legacy deployment with `admin_pk_ed25519` set | Ignored since 2026-08-15, with a warning at startup naming the config file. The operator pairs a browser; there is no second path | --- @@ -393,7 +396,7 @@ coherent with each other rather than individually demo-able. | `identities` / `members` / `invites`, codes, single-use redemption | `meshbay_node/roster.py` | | `join_transcript` — both public keys signed together | `meshbay_common/join.py` | | `join_request` / `join_result` handler, valid pre-proof and post-handshake | `transport/webrtc_server.py` | -| Admin authority read from the roster on every check, `admin_pk_ed25519` kept as legacy | `webrtc_server._verify_admin_sig` | +| Admin authority read from the roster on every check, and from nowhere else | `webrtc_server._verify_admin_sig` | | **Auto-pin of the keystore key deleted** (M3) | `daemon._legacy_admin_pk` | | `meshbay-node operator pair`, roster in `status` | `daemon.main`, `ui/app.py` | | Pairing form in the group's Members tab | `app.js`, `transport.js` | |