Un jeton par appareil, révocable seul (2.2.0)
Tous les appareils partageaient le jeton du serveur : perdre un téléphone obligeait à changer ce jeton, donc à réappairer tout le monde. Chaque appairage émet désormais le sien. Révoquer devient une opération locale, vérifiée en appairant deux appareils puis en coupant le premier : 401 pour lui, 200 pour l'autre, jeton maître intact. L'interface du serveur les liste avec leur modèle, leur date d'appairage et leur dernière synchronisation — sans jamais afficher les jetons eux-mêmes, ce qui reviendrait à les distribuer. La table s'ajoute sans toucher aux données existantes et le jeton maître reste accepté : rien ne casse pour un appareil appairé avant cette version.
This commit is contained in:
1 parent
51b60cda70
commit
28cb545685
7 files changed
+130
-8
No files matched your search
@@ -18,6 +18,7 @@ from __future__ import annotations
|
||||
|
||||
import asyncio
|
||||
import ipaddress
|
||||
import secrets
|
||||
import time
|
||||
from contextlib import asynccontextmanager
|
||||
from pathlib import Path
|
||||
@@ -41,9 +42,18 @@ watcher = Watcher(config, storage)
|
||||
|
||||
|
||||
def authorised(x_noxely_token: str = Header(default="")) -> None:
|
||||
"""Sans jeton valide, le serveur ne répond rien : il expose des fichiers."""
|
||||
if x_noxely_token != config.token:
|
||||
"""Sans jeton valide, le serveur ne répond rien : il expose des fichiers.
|
||||
|
||||
Deux formes valent : le jeton maître, qui pilote le serveur depuis le navigateur, et celui
|
||||
d'un appareil appairé. Les distinguer est ce qui permet de révoquer un téléphone perdu sans
|
||||
déranger les autres.
|
||||
"""
|
||||
if x_noxely_token and x_noxely_token == config.token:
|
||||
return
|
||||
device = storage.device_by_token(x_noxely_token) if x_noxely_token else None
|
||||
if not device:
|
||||
raise HTTPException(status_code=401, detail="Jeton invalide")
|
||||
storage.touch_device(device["id"])
|
||||
|
||||
|
||||
@asynccontextmanager
|
||||
@@ -221,8 +231,19 @@ def pairing_state() -> dict:
|
||||
return {"open": remaining > 0, "seconds": max(0, remaining)}
|
||||
|
||||
|
||||
@app.get("/api/devices", dependencies=[Depends(authorised)])
|
||||
def devices() -> dict:
|
||||
return {"devices": storage.devices()}
|
||||
|
||||
|
||||
@app.delete("/api/devices/{device_id}", dependencies=[Depends(authorised)])
|
||||
def revoke_device(device_id: str) -> dict:
|
||||
"""Révoque un appareil : son jeton cesse immédiatement d'être accepté."""
|
||||
return {"revoked": storage.delete_device(device_id)}
|
||||
|
||||
|
||||
@app.post("/api/pair")
|
||||
def pair(request: Request) -> dict:
|
||||
def pair(request: Request, payload: dict = Body(default={})) -> dict:
|
||||
"""Livre le jeton à un appareil du réseau local, pendant la fenêtre d'appairage.
|
||||
|
||||
La fenêtre se referme dès qu'un appareil s'est servi : une ouverture, un appairage.
|
||||
@@ -237,7 +258,11 @@ def pair(request: Request) -> dict:
|
||||
if not _is_private(client):
|
||||
raise HTTPException(status_code=403, detail="Appairage réservé au réseau local")
|
||||
_pairing_until = None
|
||||
return {"token": config.token, "name": "Noxely", "version": VERSION}
|
||||
# Chaque appairage reçoit son propre jeton : révoquer l'un n'atteint pas les autres.
|
||||
token = secrets.token_urlsafe(24)
|
||||
name = str(payload.get("name") or "Appareil").strip()[:60]
|
||||
storage.add_device(name, token)
|
||||
return {"token": token, "name": "Noxely", "version": VERSION, "device": name}
|
||||
|
||||
|
||||
@app.get("/api/bootstrap")
|
||||
|
||||
Reference in new issue
Block a user