fix(licence): refus de la 2e couche indiscernable d'une route inconnue
Le 404 a corps vide etait remplace par la page HTML d'Indy, distincte du 404 JSON d'une route inexistante : on pouvait lister les routes reelles. La 2e couche renvoie maintenant exactement le meme JSON. Ajoute un exe d'essai qui saute l'ecran d'activation, pour eprouver la 2e couche depuis l'application. Define de compilation LICENCE_TEST_SKIP_SCREEN, jamais un reglage : un interrupteur dans config.txt ferait sauter la 1re couche sans rapiecage. Build dedie, .dcu a part ; verifie que seul l'exe d'essai porte le define. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -476,6 +476,24 @@ l'ecran ne donne plus acces aux donnees.
|
|||||||
atteignent ce point ; les fichiers de l'ecran d'activation passent par
|
atteignent ce point ; les fichiers de l'ecran d'activation passent par
|
||||||
`TryServeEmbedded`, non garde — sinon l'ecran qui permet de s'activer ne se
|
`TryServeEmbedded`, non garde — sinon l'ecran qui permet de s'activer ne se
|
||||||
chargerait plus. **404**, pas 403, comme le reste du verrouillage.
|
chargerait plus. **404**, pas 403, comme le reste du verrouillage.
|
||||||
|
- **Le refus est le MEME 404 qu'une route inexistante** (`TJSONHelper.SendError(404,
|
||||||
|
'Not found')`). Un corps vide ne suffisait pas : Indy le remplace par sa page
|
||||||
|
HTML, distincte du JSON d'une route inconnue — ce qui revelait les routes
|
||||||
|
existantes (vu a l'essai F de LICENCE-TESTS.md).
|
||||||
|
- **Exe d'essai qui saute l'ecran** : define `LICENCE_TEST_SKIP_SCREEN`, qui fait
|
||||||
|
repondre `licence/status` ok. **A la compilation et jamais par un reglage** —
|
||||||
|
un interrupteur dans `config.txt`, que n'importe qui cree a cote de l'exe,
|
||||||
|
ferait sauter la 1re couche sans rapiecage. Build dedie, dossiers exe ET
|
||||||
|
`.dcu` a part (des unites compilees avec ce define ne doivent jamais etre
|
||||||
|
liees dans un exe normal) ; `;` des defines echappe en `%3B` pour MSBuild :
|
||||||
|
```
|
||||||
|
msbuild PMServer.dproj /p:Config=Release /p:Platform=Win32
|
||||||
|
"/p:DCC_Define=RELEASE%3BLICENCE_TEST_SKIP_SCREEN"
|
||||||
|
"/p:DCC_ExeOutput=E:\password manager\Win32-TEST-licence"
|
||||||
|
"/p:DCC_DcuOutput=E:\password manager\dcu\Win32-TEST-licence"
|
||||||
|
```
|
||||||
|
Verifier apres build : la chaine `LICENCE TEST BUILD` (UTF-16) ne doit
|
||||||
|
exister que dans l'exe d'essai.
|
||||||
- Un **oui** est retenu pour le processus (le test est sur le chemin de chaque
|
- Un **oui** est retenu pour le processus (le test est sur le chemin de chaque
|
||||||
requete) ; un **non** jamais, pour qu'une activation faite juste avant soit
|
requete) ; un **non** jamais, pour qu'une activation faite juste avant soit
|
||||||
vue a la requete suivante.
|
vue a la requete suivante.
|
||||||
|
|||||||
@@ -45,7 +45,7 @@ var
|
|||||||
implementation
|
implementation
|
||||||
|
|
||||||
uses
|
uses
|
||||||
PM.LicenceGate;
|
PM.LicenceGate, PM.JSON;
|
||||||
|
|
||||||
constructor TPMRouter.Create;
|
constructor TPMRouter.Create;
|
||||||
begin
|
begin
|
||||||
@@ -92,12 +92,15 @@ begin
|
|||||||
// DEUXIEME COUCHE DE LICENCE : ici, et seulement ici. Seules les routes
|
// DEUXIEME COUCHE DE LICENCE : ici, et seulement ici. Seules les routes
|
||||||
// de l'API arrivent a cette ligne — les fichiers de l'ecran d'activation
|
// de l'API arrivent a cette ligne — les fichiers de l'ecran d'activation
|
||||||
// passent par TryServeEmbedded, qui n'est pas garde, sinon l'ecran qui
|
// passent par TryServeEmbedded, qui n'est pas garde, sinon l'ecran qui
|
||||||
// permet de s'activer ne pourrait plus se charger. 404 et non 403, comme
|
// permet de s'activer ne pourrait plus se charger.
|
||||||
// le reste du verrouillage : ne pas dire ce qui manque.
|
//
|
||||||
|
// LE MEME 404 QU'UNE ROUTE INEXISTANTE, AU CARACTERE PRES. Un corps vide
|
||||||
|
// ne suffisait pas : Indy le remplace par sa propre page, qui se
|
||||||
|
// distinguait du JSON d'une route inconnue — et revelait donc quelles
|
||||||
|
// routes existent. Refuser doit etre indiscernable de ne rien avoir.
|
||||||
if not ApiLicenceOk then
|
if not ApiLicenceOk then
|
||||||
begin
|
begin
|
||||||
AResponse.ResponseNo := 404;
|
TJSONHelper.SendError(AResponse, 404, 'Not found');
|
||||||
AResponse.ContentText := '';
|
|
||||||
Exit(True);
|
Exit(True);
|
||||||
end;
|
end;
|
||||||
SetLength(LParams, LMatch.Groups.Count - 1);
|
SetLength(LParams, LMatch.Groups.Count - 1);
|
||||||
|
|||||||
@@ -1362,6 +1362,15 @@ begin
|
|||||||
begin
|
begin
|
||||||
LLicOk := PM.Licence.IsLicensed;
|
LLicOk := PM.Licence.IsLicensed;
|
||||||
LLicInfo := PM.Licence.LicenceInfo;
|
LLicInfo := PM.Licence.LicenceInfo;
|
||||||
|
{$IFDEF LICENCE_TEST_SKIP_SCREEN}
|
||||||
|
// EXE D'ESSAI UNIQUEMENT : simule une 1re couche rapiecee, pour eprouver
|
||||||
|
// la 2e depuis l'application elle-meme. A la COMPILATION et jamais par un
|
||||||
|
// reglage — un interrupteur dans config.txt, que n'importe qui peut
|
||||||
|
// creer a cote de l'exe, ferait sauter la 1re couche sans rapiecage.
|
||||||
|
// La 2e couche, elle, verifie toujours la vraie licence : c'est l'essai.
|
||||||
|
LLicOk := True;
|
||||||
|
LogLine('LICENCE TEST BUILD: ecran d''activation saute (1re couche simulee rapiecee)');
|
||||||
|
{$ENDIF}
|
||||||
end;
|
end;
|
||||||
// Le nom, le courriel et la date viennent d'une charge utile QUE NOUS AVONS
|
// Le nom, le courriel et la date viennent d'une charge utile QUE NOUS AVONS
|
||||||
// SIGNEE, mais ils sont interpoles dans du JavaScript : on les echappe
|
// SIGNEE, mais ils sont interpoles dans du JavaScript : on les echappe
|
||||||
|
|||||||
Reference in New Issue
Block a user