From d12ccc782d8e6eab46eb6e0976b747dca1013b36 Mon Sep 17 00:00:00 2001
From: tv <tv@krebsco.de>
Date: Mon, 9 Nov 2026 12:25:09 +0000
Subject: access: ending a single session

Logging out of the dashboard (`DELETE /auth/session`) only cleared the cookie: the
session's tokens stayed taken until they expired, by every node. Ending them took
`POST /auth/logout`, which ends all of a user's sessions.

Now `DELETE /auth/session` ends the session of the token it comes with (the cookie, or
a bearer token): where the swarm has a registry, it is recorded as
`_continuum/sessions/<user>/<session>` (`registry::session_path`), its value until when
the session would have lasted, and every node refuses the session's tokens once the
change reaches it (`token::Refused::Ended`): the page's own, its narrow tokens, and one
handed to another member's dashboard, all refreshed from it and keeping its ID. The
rules take the entry from the user itself or one who manages users, as a logout. A
remake's snapshot leaves out entries past that time, a user's map where none is left:
their tokens are expired anyway.

Without a registry the document can't record it: the node ending it refuses the
session's tokens itself (`AppState::ended_sessions`, until it restarts), others take
them until they expire, as a logout there is refused.
---
 src/access/token.rs | 21 ++++++++++++++++++++-
 1 file changed, 20 insertions(+), 1 deletion(-)

(limited to 'src/access/token.rs')

diff --git a/src/access/token.rs b/src/access/token.rs
index d33a3fe..e7aff42 100644
--- a/src/access/token.rs
+++ b/src/access/token.rs
@@ -89,13 +89,16 @@ pub(crate) enum Refused {
     Disabled,
     #[error("revoked: issued before its user's tokens were")]
     Revoked,
+    #[error("its session was ended (logged out of)")]
+    Ended,
 }
 
 /// The claims of `token` if this node takes it: well formed, for `swarm`, signed by a
 /// node that is a member in `doc` and not banned, not expired at `now` (Unix ms), nor issued later
 /// than a clock off by `SKEW_MS` would; and in a swarm with a registry, for a
 /// registered user that is enabled, issued after its tokens were last revoked
-/// (`registry::User::tokens_not_before`).
+/// (`registry::User::tokens_not_before`), of a session not ended
+/// (`registry::session_ended`).
 pub(crate) fn verify(
     doc: &impl ReadDoc,
     token: &str,
@@ -150,6 +153,9 @@ pub(crate) fn verify(
         if claims.iat < user.tokens_not_before {
             return Err(Refused::Revoked);
         }
+        if registry::session_ended(doc, &claims.user, &claims.session) {
+            return Err(Refused::Ended);
+        }
     }
     Ok(claims)
 }
@@ -290,6 +296,19 @@ mod tests {
             store::write_path(tx, &registry::logouts_path("alice"), &json!(1_001)).unwrap()
         });
         assert_eq!(verify(&doc, &token, "s", 1_500), Err(Refused::Revoked));
+        // One session ended: its tokens refused, another's taken.
+        let session = claims_of(&token).session;
+        change(&mut doc, |tx| {
+            store::write_path(tx, &registry::logouts_path("alice"), &json!(0)).unwrap();
+            store::write_path(
+                tx,
+                &registry::session_path("alice", &session),
+                &json!(10_000),
+            )
+            .unwrap()
+        });
+        assert_eq!(verify(&doc, &token, "s", 1_500), Err(Refused::Ended));
+        assert!(verify(&doc, &issue(&key, &claims(&key)), "s", 1_500).is_ok());
         alice["disabled"] = json!(true);
         change(&mut doc, |tx| {
             store::write_path(tx, &registry::logouts_path("alice"), &json!(0)).unwrap();
-- 
cgit v1.3.1