Messagerie dossier en lecture #83
Conversation
DavidBruant
force-pushed
the
lecture-messagerie
branch
from
acbc245
to
b26d836
Compare
DavidBruant
force-pushed
the
lecture-messagerie
branch
from
ea9b637
to
d7224cb
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
Normalement, on n'est pas mal en l'état Cette PR contient tout ce qu'il faut pour avoir la messagerie en lecture chez Pitchou :
|
| Actuellement, les contenus viennent de Démarches Simplifiées et on | ||
| leur fait confiance pour assainir le HTML, mais | ||
| --> | ||
| {@html contenu} |
There was a problem hiding this comment.
il y a un risque ouvert ici de XSS
There was a problem hiding this comment.
Je vais voir avec l'équipe de DS s'iels protègent bien contre ça
|
Ah... je viens de comprendre un truc De ce qu'on récupère de l'API DS, les messages automatisés sont en HTML (et donc copiables comme tels) et ceux écris par instructeurs/usagers sont en texte brut (notamment avec sauts de lignes en |
There was a problem hiding this comment.
La PR est ok pour moi, 1-2 mini commentaires.
Par contre, je n'ai pas réussi à la tester en local. La synchronisation des dossiers pète avec le dernier dossier qui a été crée et qui a l'air vide ? Je suis allée regarder en prod et le script lève également une erreur : https://dashboard.scalingo.com/apps/osc-fr1/especes-protegees/logs
| */ | ||
| export function getDossierIdsFromDS_Ids(DS_ids, databaseConnection = directDatabaseConnection){ | ||
| return databaseConnection('dossier') | ||
| .select(['id', 'id_demarches_simplifiées', 'number_demarches_simplifiées']) |
There was a problem hiding this comment.
Question pour moi : c'est quoi la différence entre id_démarches_simplifiées et number_demarches_simplifiées ?
There was a problem hiding this comment.
ouais, c'est une bizarrerie de DS
le number_demarches_simplifiées, c'est l'identifiant qu'on voit dans le dossier et dans l'URL dans DS (qu'on utilise nous pour créer les liens vers DS notamment)
id_démarches_simplifiées, c'est un autre identifiant (qui vaut à peu près base64("dossier-<numéroDossier>")) qui est utilisé à d'autres endroits de l'API DS, notamment pour la mutation d'annotations privées
| * @param {any} _request | ||
| * @param {any} reply | ||
| */ | ||
| function sendIndexHTMLFile(_request, reply){ |
|
Oui, le script de sync était cassé parce que la question de la liste des espèces est devenue conditionnelle |
Co-authored-by: Fanny Cheung <[email protected]>
…ateToProps de DossierMessagerie
|
C'est bon de mon côté |
No description provided.