Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG BO] affichage des suivis #2527

Merged
merged 4 commits into from
May 13, 2024
Merged

[BUG BO] affichage des suivis #2527

merged 4 commits into from
May 13, 2024

Conversation

numew
Copy link
Collaborator

@numew numew commented May 3, 2024

Ticket

#2499

Description

Correction de l'affichage des suivis qui pouvant contenir de L’HTML peuvent casser l'affichage

Changements apportés

  • Ajout du composant Symfony HTML Sanitizer
  • Configuration du composant afin qu'il autorise uniquement les balise et attribut HTML attendu lors de l'affichage des suivi / rapport de visite
  • Ne plus convertir les saut de ligne en en entité mais en html simple lors du dépot suivi usager (+ conversion avant affichage des ancien br enregistré en entité html pour un affichage correct)

Pré-requis

make composer

Tests

  • Copier coller du HTML dans un suivi agent / un rapport de visite et vérifier que l'affichage reste correct.
  • Déposer un suivi contenant des saut de ligne en tant qu'usager et vérifier qu'il s'affiche correctement

Notes

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test KO,

J'ai copié/collé un suivi de la prod (suivi du 6 mai de Sophie Martinez https://histologe.beta.gouv.fr/bo/signalements/a7f2dd25-42b8-40b6-acc0-7212377dad63#suivis) qui est un copié-collé de mail, donc plein de tableau, html, signature de mail avec photo etc. Ce sont typiquement le genre de suivis qui reviennent très souvent.
On perd une énorme partie de l'information

@numew
Copy link
Collaborator Author

numew commented May 7, 2024

bien vu @hmeneuvrier
J'ai essayé de trouver un compromis afin que l'on ne perde plus d'information importantes (on perd encore de la mise en forme / image) tout en gardant le coté sécurisation et sauvegarde la mis en page.
Au départ l'objectif était de ne pas permettre de garder du html contenant de style pouvant casser la mis en page (ex https://histologe.beta.gouv.fr/bo/signalements/583ed22a-5b88-4afd-aca4-0be0234d9672)

@numew numew force-pushed the feature/2499-sanitize-html branch from ebfbfbd to 335acbe Compare May 7, 2024 13:12
Copy link

sonarcloud bot commented May 7, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests et relecture OK

Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lecture et test OK

@sfinx13 sfinx13 merged commit ee12842 into develop May 13, 2024
3 checks passed
@sfinx13 sfinx13 deleted the feature/2499-sanitize-html branch May 13, 2024 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants