[BUGFIX] Vérification des CGUs acceptées avant acceptation (PIX-16377)

[bpetetot]

🥞 Problème

L’appel /api/users/{id}/pix-orga-terms-of-service-acceptance provoque des erreurs 500 quand les CGUs ont déjà été acceptées.

DatabaseError: /* path: /api/users/{id}/pix-orga-terms-of-service-acceptance */ insert into "legal-document-version-user-acceptances" ("legalDocumentVersionId", "userId") values ($1, $2) - duplicate key value violates unique constraint "legal_document_version_user_acceptances_userid_legaldocumentver"

Cela peut survenir quand l’utilisateur a plusieurs pages d’ouvertes sur lesquelles la validation des CGUs est demandée.

🥓 Proposition

Ajouter une vérification au moment de l'acceptation d'un document pour s'assurer qu'elle n'a pas été déjà réalisée.

🧃 Remarques

Correction du script de conversion des CGUs des utilisateurs, pour garantir un ordre dans la requête de récupération des utilisateurs paginés.

😋 Pour tester

Pré-requis
Exécuter le script depuis le dossier API :

node src/legal-documents/scripts/add-new-legal-document-version.js --type="TOS" --service="pix-orga" --versionAt="2022-11-30"

Scénario

1. Se connecter à Pix Orga avec le compte allorga@example.net
2. Récupérer l'access token dans le local storage (pour le renseigner dans le CURL suivant)

Exécuter la commande cURL suivante en remplaçant ACCESS_TOKEN par le token utilisateur, vérifier que le user id (10001) est celui utilisé, et que l'URL (http://localhost:4201) correspond à votre environnement:

curl -X 'PATCH' 'http://localhost:4201/api/users/10001/pix-orga-terms-of-service-acceptance' -H 'Authorization: Bearer ACCESS_TOKEN'

_

Cette commande NE DOIT PAS retourner une erreur 500, si elle est exécuter plusieurs fois de suite:

{"statusCode":500,"error":"Internal Server Error","message":"An internal server error occurred"}

[pix-bot-github]

Une fois les applications déployées, elles seront accessibles via les liens suivants :

• App (.fr)
• App (.org)
• Orga (.fr)
• Orga (.org)
• Certif (.fr)
• Certif (.org)
• Junior
• Admin
• API
• Audit Logger

Les variables d'environnement seront accessibles via les liens suivants :

• scalingo front
• scalingo api
• scalingo audit-logger

[P-Jeremy]

Test func OK ✅

[theotime2005]

Si on a un code 204 c'est ok?

[bpetetot]

Si on a un code 204 c'est ok?

@theotime2005 Oui c'est bon. Le code 204 signifie un succès sans données retournées par l'appel (doc: https://developer.mozilla.org/fr/docs/Web/HTTP/Status/204)