maximus/Simpl-Resultat

Fork 0

fix: carte Licence — erreurs nommées, clé révoquée affichée, édition indépendante du jeton d'activation, machines n/limite, dates #347

New issue

Closed

opened 2026-09-14 00:19:31 +00:00 by maximus · 1 comment

maximus commented

2026-09-14 00:19:31 +00:00

Owner

Après #270 et #346, la carte Licence garde cinq défauts :

les erreurs de saisie de clé et l'erreur de chargement s'affichent en anglais brut ;
une clé révoquée n'est signalée qu'au moment du refus, puis le message disparaît ;
la limite de machines n'est jamais affichée ;
les dates suivent la langue du système, pas celle de l'app, et une clé de 100 ans afficherait une échéance en 2126 ;
le jeton d'activation peut faire retomber une clé valide en Gratuite, sans issue dans l'interface : au bout de 2 ans, parce que le jeton expire avant la clé, ou après une réinstallation ou une migration de poste avec restauration des données (#360).

C'est l'écran qu'un acheteur voit quand sa clé pose problème.

Décisions

D12 (analyse du 2026-09-29, ferme #360) : le jeton d'activation ne fait plus jamais retomber en Gratuite. L'édition et les features[] suivent la seule clé ; le jeton sert à l'état « Activée » de la carte et aux cours du Bilan, que le serveur vérifie. D12 remplace D10 (jeton expiré de cette machine = absent), qui en devient un cas particulier. Motif : une license.key copiée seule garde déjà son édition (état pré-activation toléré), donc la règle ne protégeait rien et ne pénalisait que la restauration légitime des données.
D13 (analyse du 2026-09-29) : clé révoquée = drapeau de session dans useLicenseMachines, avec le message existant license.errors.licenseRevoked posé par #270. Aucune clé i18n nouvelle.
D5, révisée en revue le 09-24 : quatre clés orphelines branchées (count, activating, deactivating, notActivated), deux retirées (removeLicense, activateError). Pas de commande de suppression de licence.
D9 : clés de 100 ans (maximus/maximus-api#63, mergée par la PR maximus-api#72). L'état « clé expirée le {date} » est abandonné. Une clé déjà expirée qu'on saisit reste nommée (license_expired), et « Sans échéance » s'affiche au-delà de 50 ans.
L'état révoqué ne dure que la session : rien n'est persisté.

Contexte vérifié (2026-09-29)

Relevé sur main 5d549d6 et, côté serveur, sur maximus-api main 3673d09. Se repérer aux noms de fonctions.

Client, src-tauri/src/commands/license_commands.rs et la carte :

strip_prefix, validate_with_key (« Invalid license: {erreur jsonwebtoken} », « Unknown edition »), store_license et read_license renvoient des chaînes anglaises. LicenseCard.tsx affiche validationError et l'erreur de chargement (licenseStatus === "error") telles quelles ; LicenseContext garde la chaîne brute.
Aucun appelant ne lit le détail de l'erreur de validate_with_key : validate_license_key et store_license la renvoient, read_license fait .ok(), resolve_license_entitlements fait let Ok(..) else.
LicenseError (posé par #270) porte déjà LicenseExpired, Storage et Internal ; son commentaire annonce l'extension de #347.
resolve_license_entitlements retombe en Gratuite dès qu'un jeton existe et ne valide pas (strict_validation(), leeway = 0). Un jeton absent est toléré : le test missing_activation_token_keeps_edition_and_features prouve qu'une clé copiée seule garde son édition. La contrainte 2 de l'ADR 0017 (« une license.key copiée sur une autre machine est déclassée en free ») est donc inexacte.
get_activation_status valide strictement : un jeton expiré ou d'une autre machine s'affiche « non activé », et le bouton « Activer » apparaît. Mais la section Machines est masquée en Gratuite (hasLicense), d'où l'impasse de #360.
LicenseInfo.machine_limit est un claim signé, disponible localement.
i18n : license.machines.count, .activating, .deactivating et .notActivated existent sans être utilisées ; license.removeLicense et license.machines.activateError sont mortes.
toLocaleDateString() est appelé sans locale (formatExpiry et la date d'activation d'une machine). Convention du dépôt : i18n.language === "fr" ? "fr-CA" : "en-CA", écrit en ligne.
jsonwebtoken 9.3.1 vérifie la présence des claims exigés indépendamment de validate_exp.
Un license_revoked affiche déjà « Cette clé de licence a été révoquée. » là où le refus arrive (#270) : sous l'en-tête Machines pour activate, dans la liste pour verify. Le message disparaît quand on referme la liste, l'édition garde sa coche verte, et « Activer » reste cliquable alors que le serveur répondra 403 à chaque essai.
Une fois le jeton expiré, le Bilan affiche balance.priceFetching.errors.authFailed (« Échec d'authentification — vérifie ta licence »), sans indiquer qu'il suffit de réactiver la machine.

Serveur, maximus-api :

Le jeton d'activation est signé pour 2 ans (signActivationJwt, src/crypto/ed25519.ts) ; #63 n'y a pas touché. Claims : product, machine_id, jti, sub (UUID de la licence), iat, exp.
Clés de 100 ans : LICENSE_DURATION_DAYS = 36525 pour /generate et pour Stripe, soit un exp vers 2126, au-delà de 2^32.
Activate contrôle dans cet ordre : clé introuvable, révoquée, expirée, machine déjà active, limite. Une machine déjà active reçoit un jeton neuf sans consommer de place. Une licence révoquée répond 403 license_revoked, même pour une machine active.
Verify répond toujours 200, avec {valid:false, reason: "revoked" | "not_found" | "expired"}.
Deactivate sur une licence révoquée répond 404 no_active_activation (la révocation désactive toutes les machines), que le client traite comme un succès.
/v1/prices valide le jeton strictement (exp, jti révoqué, licence révoquée) : il reste la seule barrière dure.
Un seul quota de 5 requêtes par minute et par IP est partagé par les routes /v1/licenses/*.

Tâches

Erreurs locales nommées (Rust). Étendre LicenseError avec InvalidPrefix, InvalidSignature, Malformed et UnknownEdition, et réutiliser LicenseExpired, Storage et Internal. validate_with_key renvoie Result<LicenseInfo, LicenseError>, à sémantique inchangée : mêmes clés acceptées, mêmes refus. Correspondance :
- strip_prefix → InvalidPrefix ;
- ErrorKind::ExpiredSignature → LicenseExpired ;
- ErrorKind::InvalidSignature → InvalidSignature ;
- tout autre échec de décodage (JWT illisible, claim manquant, algorithme inattendu) → Malformed ;
- édition ni base ni premium → UnknownEdition ;
- clé publique embarquée illisible → Internal.
validate_license_key, store_license et read_license renvoient le JSON via license_error_to_string ; leurs erreurs de fichier deviennent Storage.
Côté TS. Ajouter les codes à LicenseErrorCode et à LICENSE_ERROR_I18N_MAP, et les clés FR/EN license.errors.invalidPrefix, invalidSignature, malformed et unknownEdition, au vouvoiement. LicenseCard.tsx passe validationError et l'erreur de chargement par parseLicenseError, puis par t(...) ; le contexte garde la chaîne brute.
L'édition ne dépend plus du jeton (D12, ferme #360). L'édition et les features[] se résolvent depuis la seule clé : une clé valide et non expirée donne son édition, quel que soit le jeton (absent, expiré, d'une autre machine, illisible) ; une clé invalide ou expirée donne ("free", []). resolve_license_entitlements ne prend plus le jeton ni l'identifiant machine, et current_entitlements ne les lit plus. Pour tester au point de décision, extraire la lecture de fichiers de current_entitlements dans une fonction qui prend le dossier de données, testée avec std::env::temp_dir() comme dans token_store.rs.

Le jeton garde une validation stricte là où il décide de quelque chose : get_activation_status (« Activée » / « Non activée ») et persist_activation_token (refus d'un jeton serveur qui ne vaut pas pour cette machine). Mettre à jour les commentaires qui décrivent l'ancienne règle : en-tête du module, get_edition, current_entitlements, machine_id_internal.

Conséquence voulue : après une réinstallation avec restauration des données, la clé garde son édition, et la section Machines reste visible avec « Non activée » et « Activer ». L'activation écrase l'ancien jeton ; elle consomme une place, et la limite ouvre la liste comme aujourd'hui. Avec un jeton expiré de cette machine, « Activer » renouvelle le jeton sans consommer de place.
Clé révoquée (D13). Ajouter revoked: boolean à LicenseMachinesState, posé par tout license_revoked (ACTIVATE_ERROR, LIST_LOAD_ERROR, DEACTIVATE_ERROR), jamais remis à zéro pendant la session, jamais persisté. La carte affiche license.errors.licenseRevoked à son niveau, visible liste fermée ; une icône d'alerte remplace la coche verte de l'édition, et selectCanActivate renvoie false. L'édition suit toujours la validation locale (ADR 0017).
Machines n/limite. Une fois la liste chargée sans erreur, afficher license.machines.count avec {count: machines.length, limit: info.machine_limit}.
Clés orphelines (D5). license.machines.activating et deactivating servent de libellés aux boutons pendant le chargement ; license.machines.notActivated s'affiche quand la machine n'est pas activée ; license.removeLicense et license.machines.activateError sont retirées des deux langues ; license.errors.unknown reste le seul repli générique.
Échéance (D9). Fonction pure dans src/shared/licenseExpiry.ts : au-delà de 50 ans, « Sans échéance » / « No expiry date » (clé license.noExpiry) ; sinon, la date dans la langue de l'app.
Dates. formatExpiry et la date d'activation d'une machine : toLocaleDateString(i18n.language === "fr" ? "fr-CA" : "en-CA").
Cours du Bilan. Reformuler balance.priceFetching.errors.authFailed (FR/EN) pour indiquer d'activer la machine dans Paramètres → Utilisateurs, dans le registre des messages voisins. Le même 401 couvre un jeton absent, expiré ou invalide.
Tests.
- Rust : chaque variante locale nommée (préfixe, signature d'une autre clé, JWT corrompu, clé expirée, édition inconnue). Les rejects_* existants vérifient la variante, plus seulement is_err() ; rejects_expired_license reste vert et vérifie LicenseExpired.
- Rust, au point de décision (dossier temporaire avec license.key) : clé expirée → Gratuite sans features ; clé valide avec un activation.token expiré de cette machine, d'une autre machine ou illisible → édition et features conservées. Ces tests remplacent machine_match_keeps_edition_and_features, machine_mismatch_downgrades_to_free_and_drops_features et missing_activation_token_keeps_edition_and_features, dont le sujet disparaît.
- Rust, la validation stricte du jeton reste : validate_activation_with_key refuse un jeton expiré et un jeton d'une autre machine.
- Rust, tests de #363 : une clé avec exp = now + 36525 * 86400 est acceptée, et expires_at vaut cette valeur ; la même clé avec un exp passé est refusée.
- vitest : nouveaux codes → clés ; toutes les clés license.errors.* présentes en FR et en EN (test existant) ; seuil « Sans échéance », dont une clé de 100 ans ; transition revoked du reducer et selectCanActivate.
- vitest, rendu de LicenseCard (renderToStaticMarkup, contexte et hook remplacés par des faux, comme UpsellCtas.render.test.tsx) : aucune erreur anglaise brute sur la carte en français, message révoqué visible liste fermée, « n/limite », « Sans échéance ».
Documentation.
- CHANGELOG.md et CHANGELOG.fr.md : Fixed / Corrigé (l'édition ne dépend plus du jeton d'activation, qu'il soit expiré ou issu d'une réinstallation ou d'une migration de poste ; erreurs de saisie nommées ; dates dans la langue de l'app) et Changed / Modifié (clé révoquée affichée, compteur de machines, « Sans échéance »).
- docs/guide-utilisateur.md §12 : nouvelle sous-section sur la carte Licence (activation et réactivation pour les cours, clé révoquée, compteur de machines, « Sans échéance »), avec son pendant dans les docs intégrées (docs.editions.*, FR/EN).
- docs/architecture.md, §license_commands.rs : règle de résolution (la clé seule) et rôle du jeton (état de la carte, serveur).
- ADR 0017 : amendement (#347) qui corrige la contrainte 2 (la licence n'est liée à la machine que côté serveur) et reformule la règle CWE-863 locale : fail-closed en Gratuite pour une clé invalide ou expirée, features[] signés qui suivent la clé.

Fichiers concernés

src-tauri/src/commands/license_commands.rs
src/services/licenseService.ts, src/services/licenseService.test.ts
src/hooks/useLicenseMachines.ts, src/hooks/useLicenseMachines.test.ts
src/shared/licenseExpiry.ts, src/shared/licenseExpiry.test.ts (nouveaux)
src/components/settings/LicenseCard.tsx, src/components/settings/LicenseCard.render.test.tsx (nouveau)
src/i18n/locales/fr.json, src/i18n/locales/en.json
docs/adr/0017-feature-gating-par-tier.md, docs/guide-utilisateur.md, docs/architecture.md
CHANGELOG.md, CHANGELOG.fr.md

Critères d'acceptation

Aucune chaîne anglaise en dur n'atteint la carte Licence en français : saisie, chargement et refus serveur passent par license.errors.*.
Une clé valide garde son édition et ses features quel que soit le jeton (absent, expiré, d'une autre machine, illisible), prouvé par des tests Rust au point de décision. Une clé invalide ou expirée résout toujours ("free", []).
Une machine dont le jeton est expiré ou appartient à une autre machine affiche « Non activée » et peut (ré)activer depuis la carte.
Un license_revoked reçu du serveur reste affiché au niveau de la carte pendant la session, et « Activer » est désactivé.
La liste des machines affiche « {n}/{limite} machines activées ».
Une clé dont l'échéance est à plus de 50 ans affiche « Sans échéance » (clé de 100 ans testée) ; les autres dates suivent la langue de l'app (fr-CA / en-CA).
license.removeLicense et license.machines.activateError n'existent plus ; activating, deactivating, notActivated et count sont utilisées.
cargo test, npm test et npm run build verts ; CHANGELOG FR/EN ; guide §12, docs intégrées et ADR 0017 à jour.

Points de vigilance

La clé de licence reste validée strictement (exp et iat exigés, leeway = 0, CWE-613). Ne jamais lui appliquer une validation qui n'impose pas exp.
D12 ne touche que le poids du jeton sur l'édition locale. get_activation_status et persist_activation_token gardent la validation stricte (signature, machine_id, exp) : ne pas les assouplir.
Prouver la règle au point de décision : tester la résolution à partir du dossier de données, pas seulement validate_with_key.
license.errors.unknown reste le seul repli générique (#270) ; activateError affichait « Activation failed » en anglais.
fr.json, en.json, les deux CHANGELOG, le guide §12 et l'ADR 0017 ont été modifiés par #346, mergée : partir de main.

Hors périmètre

Avertissement avant l'échéance : les clés durent désormais 100 ans.
Alignement côté serveur de la durée du jeton d'activation (2 ans).
Hachage de l'identifiant machine → #350. D12 retire le risque de sa transition pour l'édition locale ; la réactivation pour les cours reste nécessaire.
Côté serveur, /v1/prices ne vérifie pas que la machine est encore active : une machine désactivée garde les cours jusqu'à l'expiration de son jeton. À traiter dans maximus-api.
Reste de #363 : plafond de /issue-license à 36525 jours une fois maximus-api#63 déployée, et essai d'une vraie clé émise après #63.

Liens

Dépend de #270 (PR #359) et de #346 (PR #362), mergées.
Ferme #360 (D12).
Absorbe les tests de #363.

Spec source

spec-plan-paiements.md (séance /plan-run du 2026-09-24, fichier de travail ignoré par git), puis analyse /analyze du 2026-09-29. Le corps du 09-24 est archivé en commentaire. Ce corps se suffit à lui-même.

Après #270 et #346, la carte Licence garde cinq défauts : - les erreurs de saisie de clé et l'erreur de chargement s'affichent en anglais brut ; - une clé révoquée n'est signalée qu'au moment du refus, puis le message disparaît ; - la limite de machines n'est jamais affichée ; - les dates suivent la langue du système, pas celle de l'app, et une clé de 100 ans afficherait une échéance en 2126 ; - le jeton d'activation peut faire retomber une clé valide en Gratuite, sans issue dans l'interface : au bout de 2 ans, parce que le jeton expire avant la clé, ou après une réinstallation ou une migration de poste avec restauration des données (#360). C'est l'écran qu'un acheteur voit quand sa clé pose problème. ## Décisions - **D12 (analyse du 2026-09-29, ferme #360)** : le jeton d'activation ne fait plus jamais retomber en Gratuite. L'édition et les `features[]` suivent la seule clé ; le jeton sert à l'état « Activée » de la carte et aux cours du Bilan, que le serveur vérifie. D12 remplace D10 (jeton expiré de cette machine = absent), qui en devient un cas particulier. Motif : une `license.key` copiée seule garde déjà son édition (état pré-activation toléré), donc la règle ne protégeait rien et ne pénalisait que la restauration légitime des données. - **D13 (analyse du 2026-09-29)** : clé révoquée = drapeau de session dans `useLicenseMachines`, avec le message existant `license.errors.licenseRevoked` posé par #270. Aucune clé i18n nouvelle. - D5, révisée en revue le 09-24 : quatre clés orphelines branchées (`count`, `activating`, `deactivating`, `notActivated`), deux retirées (`removeLicense`, `activateError`). Pas de commande de suppression de licence. - D9 : clés de 100 ans (maximus/maximus-api#63, mergée par la PR maximus-api#72). L'état « clé expirée le {date} » est abandonné. Une clé déjà expirée qu'on saisit reste nommée (`license_expired`), et « Sans échéance » s'affiche au-delà de 50 ans. - L'état révoqué ne dure que la session : rien n'est persisté. ## Contexte vérifié (2026-09-29) Relevé sur `main` `5d549d6` et, côté serveur, sur `maximus-api` `main` `3673d09`. Se repérer aux noms de fonctions. Client, `src-tauri/src/commands/license_commands.rs` et la carte : - `strip_prefix`, `validate_with_key` (« Invalid license: {erreur jsonwebtoken} », « Unknown edition »), `store_license` et `read_license` renvoient des chaînes anglaises. `LicenseCard.tsx` affiche `validationError` et l'erreur de chargement (`licenseStatus === "error"`) telles quelles ; `LicenseContext` garde la chaîne brute. - Aucun appelant ne lit le détail de l'erreur de `validate_with_key` : `validate_license_key` et `store_license` la renvoient, `read_license` fait `.ok()`, `resolve_license_entitlements` fait `let Ok(..) else`. - `LicenseError` (posé par #270) porte déjà `LicenseExpired`, `Storage` et `Internal` ; son commentaire annonce l'extension de #347. - `resolve_license_entitlements` retombe en Gratuite dès qu'un jeton existe et ne valide pas (`strict_validation()`, `leeway = 0`). Un jeton absent est toléré : le test `missing_activation_token_keeps_edition_and_features` prouve qu'une clé copiée seule garde son édition. La contrainte 2 de l'ADR 0017 (« une `license.key` copiée sur une autre machine est déclassée en `free` ») est donc inexacte. - `get_activation_status` valide strictement : un jeton expiré ou d'une autre machine s'affiche « non activé », et le bouton « Activer » apparaît. Mais la section Machines est masquée en Gratuite (`hasLicense`), d'où l'impasse de #360. - `LicenseInfo.machine_limit` est un claim signé, disponible localement. - i18n : `license.machines.count`, `.activating`, `.deactivating` et `.notActivated` existent sans être utilisées ; `license.removeLicense` et `license.machines.activateError` sont mortes. - `toLocaleDateString()` est appelé sans locale (`formatExpiry` et la date d'activation d'une machine). Convention du dépôt : `i18n.language === "fr" ? "fr-CA" : "en-CA"`, écrit en ligne. - `jsonwebtoken` 9.3.1 vérifie la présence des claims exigés indépendamment de `validate_exp`. - Un `license_revoked` affiche déjà « Cette clé de licence a été révoquée. » là où le refus arrive (#270) : sous l'en-tête Machines pour activate, dans la liste pour verify. Le message disparaît quand on referme la liste, l'édition garde sa coche verte, et « Activer » reste cliquable alors que le serveur répondra 403 à chaque essai. - Une fois le jeton expiré, le Bilan affiche `balance.priceFetching.errors.authFailed` (« Échec d'authentification — vérifie ta licence »), sans indiquer qu'il suffit de réactiver la machine. Serveur, `maximus-api` : - Le jeton d'activation est signé pour 2 ans (`signActivationJwt`, `src/crypto/ed25519.ts`) ; #63 n'y a pas touché. Claims : `product`, `machine_id`, `jti`, `sub` (UUID de la licence), `iat`, `exp`. - Clés de 100 ans : `LICENSE_DURATION_DAYS = 36525` pour `/generate` et pour Stripe, soit un `exp` vers 2126, au-delà de 2^32. - Activate contrôle dans cet ordre : clé introuvable, révoquée, expirée, machine déjà active, limite. Une machine déjà active reçoit un jeton neuf sans consommer de place. Une licence révoquée répond 403 `license_revoked`, même pour une machine active. - Verify répond toujours 200, avec `{valid:false, reason: "revoked" | "not_found" | "expired"}`. - Deactivate sur une licence révoquée répond 404 `no_active_activation` (la révocation désactive toutes les machines), que le client traite comme un succès. - `/v1/prices` valide le jeton strictement (`exp`, `jti` révoqué, licence révoquée) : il reste la seule barrière dure. - Un seul quota de 5 requêtes par minute et par IP est partagé par les routes `/v1/licenses/*`. ## Tâches 1. **Erreurs locales nommées (Rust).** Étendre `LicenseError` avec `InvalidPrefix`, `InvalidSignature`, `Malformed` et `UnknownEdition`, et réutiliser `LicenseExpired`, `Storage` et `Internal`. `validate_with_key` renvoie `Result<LicenseInfo, LicenseError>`, à sémantique inchangée : mêmes clés acceptées, mêmes refus. Correspondance : - `strip_prefix` → `InvalidPrefix` ; - `ErrorKind::ExpiredSignature` → `LicenseExpired` ; - `ErrorKind::InvalidSignature` → `InvalidSignature` ; - tout autre échec de décodage (JWT illisible, claim manquant, algorithme inattendu) → `Malformed` ; - édition ni `base` ni `premium` → `UnknownEdition` ; - clé publique embarquée illisible → `Internal`. `validate_license_key`, `store_license` et `read_license` renvoient le JSON via `license_error_to_string` ; leurs erreurs de fichier deviennent `Storage`. 2. **Côté TS.** Ajouter les codes à `LicenseErrorCode` et à `LICENSE_ERROR_I18N_MAP`, et les clés FR/EN `license.errors.invalidPrefix`, `invalidSignature`, `malformed` et `unknownEdition`, au vouvoiement. `LicenseCard.tsx` passe `validationError` et l'erreur de chargement par `parseLicenseError`, puis par `t(...)` ; le contexte garde la chaîne brute. 3. **L'édition ne dépend plus du jeton (D12, ferme #360).** L'édition et les `features[]` se résolvent depuis la seule clé : une clé valide et non expirée donne son édition, quel que soit le jeton (absent, expiré, d'une autre machine, illisible) ; une clé invalide ou expirée donne `("free", [])`. `resolve_license_entitlements` ne prend plus le jeton ni l'identifiant machine, et `current_entitlements` ne les lit plus. Pour tester au point de décision, extraire la lecture de fichiers de `current_entitlements` dans une fonction qui prend le dossier de données, testée avec `std::env::temp_dir()` comme dans `token_store.rs`. Le jeton garde une validation stricte là où il décide de quelque chose : `get_activation_status` (« Activée » / « Non activée ») et `persist_activation_token` (refus d'un jeton serveur qui ne vaut pas pour cette machine). Mettre à jour les commentaires qui décrivent l'ancienne règle : en-tête du module, `get_edition`, `current_entitlements`, `machine_id_internal`. Conséquence voulue : après une réinstallation avec restauration des données, la clé garde son édition, et la section Machines reste visible avec « Non activée » et « Activer ». L'activation écrase l'ancien jeton ; elle consomme une place, et la limite ouvre la liste comme aujourd'hui. Avec un jeton expiré de cette machine, « Activer » renouvelle le jeton sans consommer de place. 4. **Clé révoquée (D13).** Ajouter `revoked: boolean` à `LicenseMachinesState`, posé par tout `license_revoked` (`ACTIVATE_ERROR`, `LIST_LOAD_ERROR`, `DEACTIVATE_ERROR`), jamais remis à zéro pendant la session, jamais persisté. La carte affiche `license.errors.licenseRevoked` à son niveau, visible liste fermée ; une icône d'alerte remplace la coche verte de l'édition, et `selectCanActivate` renvoie `false`. L'édition suit toujours la validation locale (ADR 0017). 5. **Machines n/limite.** Une fois la liste chargée sans erreur, afficher `license.machines.count` avec `{count: machines.length, limit: info.machine_limit}`. 6. **Clés orphelines (D5).** `license.machines.activating` et `deactivating` servent de libellés aux boutons pendant le chargement ; `license.machines.notActivated` s'affiche quand la machine n'est pas activée ; `license.removeLicense` et `license.machines.activateError` sont retirées des deux langues ; `license.errors.unknown` reste le seul repli générique. 7. **Échéance (D9).** Fonction pure dans `src/shared/licenseExpiry.ts` : au-delà de 50 ans, « Sans échéance » / « No expiry date » (clé `license.noExpiry`) ; sinon, la date dans la langue de l'app. 8. **Dates.** `formatExpiry` et la date d'activation d'une machine : `toLocaleDateString(i18n.language === "fr" ? "fr-CA" : "en-CA")`. 9. **Cours du Bilan.** Reformuler `balance.priceFetching.errors.authFailed` (FR/EN) pour indiquer d'activer la machine dans Paramètres → Utilisateurs, dans le registre des messages voisins. Le même 401 couvre un jeton absent, expiré ou invalide. 10. **Tests.** - Rust : chaque variante locale nommée (préfixe, signature d'une autre clé, JWT corrompu, clé expirée, édition inconnue). Les `rejects_*` existants vérifient la variante, plus seulement `is_err()` ; `rejects_expired_license` reste vert et vérifie `LicenseExpired`. - Rust, au point de décision (dossier temporaire avec `license.key`) : clé expirée → Gratuite sans features ; clé valide avec un `activation.token` expiré de cette machine, d'une autre machine ou illisible → édition et features conservées. Ces tests remplacent `machine_match_keeps_edition_and_features`, `machine_mismatch_downgrades_to_free_and_drops_features` et `missing_activation_token_keeps_edition_and_features`, dont le sujet disparaît. - Rust, la validation stricte du jeton reste : `validate_activation_with_key` refuse un jeton expiré et un jeton d'une autre machine. - Rust, tests de #363 : une clé avec `exp = now + 36525 * 86400` est acceptée, et `expires_at` vaut cette valeur ; la même clé avec un `exp` passé est refusée. - vitest : nouveaux codes → clés ; toutes les clés `license.errors.*` présentes en FR et en EN (test existant) ; seuil « Sans échéance », dont une clé de 100 ans ; transition `revoked` du reducer et `selectCanActivate`. - vitest, rendu de `LicenseCard` (`renderToStaticMarkup`, contexte et hook remplacés par des faux, comme `UpsellCtas.render.test.tsx`) : aucune erreur anglaise brute sur la carte en français, message révoqué visible liste fermée, « n/limite », « Sans échéance ». 11. **Documentation.** - `CHANGELOG.md` et `CHANGELOG.fr.md` : Fixed / Corrigé (l'édition ne dépend plus du jeton d'activation, qu'il soit expiré ou issu d'une réinstallation ou d'une migration de poste ; erreurs de saisie nommées ; dates dans la langue de l'app) et Changed / Modifié (clé révoquée affichée, compteur de machines, « Sans échéance »). - `docs/guide-utilisateur.md` §12 : nouvelle sous-section sur la carte Licence (activation et réactivation pour les cours, clé révoquée, compteur de machines, « Sans échéance »), avec son pendant dans les docs intégrées (`docs.editions.*`, FR/EN). - `docs/architecture.md`, §`license_commands.rs` : règle de résolution (la clé seule) et rôle du jeton (état de la carte, serveur). - ADR 0017 : amendement (#347) qui corrige la contrainte 2 (la licence n'est liée à la machine que côté serveur) et reformule la règle CWE-863 locale : fail-closed en Gratuite pour une clé invalide ou expirée, `features[]` signés qui suivent la clé. ## Fichiers concernés - `src-tauri/src/commands/license_commands.rs` - `src/services/licenseService.ts`, `src/services/licenseService.test.ts` - `src/hooks/useLicenseMachines.ts`, `src/hooks/useLicenseMachines.test.ts` - `src/shared/licenseExpiry.ts`, `src/shared/licenseExpiry.test.ts` (nouveaux) - `src/components/settings/LicenseCard.tsx`, `src/components/settings/LicenseCard.render.test.tsx` (nouveau) - `src/i18n/locales/fr.json`, `src/i18n/locales/en.json` - `docs/adr/0017-feature-gating-par-tier.md`, `docs/guide-utilisateur.md`, `docs/architecture.md` - `CHANGELOG.md`, `CHANGELOG.fr.md` ## Critères d'acceptation - [ ] Aucune chaîne anglaise en dur n'atteint la carte Licence en français : saisie, chargement et refus serveur passent par `license.errors.*`. - [ ] Une clé valide garde son édition et ses features quel que soit le jeton (absent, expiré, d'une autre machine, illisible), prouvé par des tests Rust au point de décision. Une clé invalide ou expirée résout toujours `("free", [])`. - [ ] Une machine dont le jeton est expiré ou appartient à une autre machine affiche « Non activée » et peut (ré)activer depuis la carte. - [ ] Un `license_revoked` reçu du serveur reste affiché au niveau de la carte pendant la session, et « Activer » est désactivé. - [ ] La liste des machines affiche « {n}/{limite} machines activées ». - [ ] Une clé dont l'échéance est à plus de 50 ans affiche « Sans échéance » (clé de 100 ans testée) ; les autres dates suivent la langue de l'app (`fr-CA` / `en-CA`). - [ ] `license.removeLicense` et `license.machines.activateError` n'existent plus ; `activating`, `deactivating`, `notActivated` et `count` sont utilisées. - [ ] `cargo test`, `npm test` et `npm run build` verts ; CHANGELOG FR/EN ; guide §12, docs intégrées et ADR 0017 à jour. ## Points de vigilance - La clé de licence reste validée strictement (`exp` et `iat` exigés, `leeway = 0`, CWE-613). Ne jamais lui appliquer une validation qui n'impose pas `exp`. - D12 ne touche que le poids du jeton sur l'édition locale. `get_activation_status` et `persist_activation_token` gardent la validation stricte (signature, `machine_id`, `exp`) : ne pas les assouplir. - Prouver la règle au point de décision : tester la résolution à partir du dossier de données, pas seulement `validate_with_key`. - `license.errors.unknown` reste le seul repli générique (#270) ; `activateError` affichait « Activation failed » en anglais. - `fr.json`, `en.json`, les deux CHANGELOG, le guide §12 et l'ADR 0017 ont été modifiés par #346, mergée : partir de `main`. ## Hors périmètre - Avertissement avant l'échéance : les clés durent désormais 100 ans. - Alignement côté serveur de la durée du jeton d'activation (2 ans). - Hachage de l'identifiant machine → #350. D12 retire le risque de sa transition pour l'édition locale ; la réactivation pour les cours reste nécessaire. - Côté serveur, `/v1/prices` ne vérifie pas que la machine est encore active : une machine désactivée garde les cours jusqu'à l'expiration de son jeton. À traiter dans maximus-api. - Reste de #363 : plafond de `/issue-license` à 36525 jours une fois maximus-api#63 déployée, et essai d'une vraie clé émise après #63. ## Liens - Dépend de #270 (PR #359) et de #346 (PR #362), mergées. - Ferme #360 (D12). - Absorbe les tests de #363. ## Spec source `spec-plan-paiements.md` (séance `/plan-run` du 2026-09-24, fichier de travail ignoré par git), puis analyse `/analyze` du 2026-09-29. Le corps du 09-24 est archivé en commentaire. Ce corps se suffit à lui-même.

maximus added this to the spec-paiements milestone 2026-09-14 00:19:31 +00:00

maximus added the

status:ready

type:bug

source:analyste

labels 2026-09-14 00:19:31 +00:00

fix: carte Licence — erreurs nommées, clé révoquée affichée, édition indépendante du jeton d'activation, machines n/limite, dates #347

Décisions

Contexte vérifié (2026-09-29)

Tâches

Fichiers concernés

Critères d'acceptation

Points de vigilance

Hors périmètre

Liens

Spec source

Contexte vérifié (2026-09-24)

Tâches

Fichiers concernés

Depends on

Critères d'acceptation

Review caveats

Décisions prises en planification

Hors périmètre

Spec source