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

Ajout de CSP #1490

Merged
merged 4 commits into from
Dec 8, 2023
Merged

Ajout de CSP #1490

merged 4 commits into from
Dec 8, 2023

Conversation

pabois
Copy link
Member

@pabois pabois commented Dec 8, 2023

  • Ajout de CSP explicites
  • Passage de leaflet en package interne plutôt qu'un chargement par cdn
  • début de simplication des styles inlines

A noter :

  • ce sont des CSP souples. Comme on a des styles inline dynamiques (avec calculs liés à des itérations rails) on est un peu obligés de laisser ce style inline, et donc de ne pas forcer l'utilisation de nonce pour les styles (si on active les nonce on ne peut plus autoriser le unsafe_inline). Idem pour les scripts. L'utlisation de Vue.js oblige à autoriser le unsafe_eval.
  • le CSP autorisent explicitement une version définie (v7) de Bugsnag. En cas de màj de Bugsnag il faudra changer cette directive (le numéro de version est en dur donc il faudra de toute façon le changer dans la vue, mais aussi dans le CSP)

@pabois pabois merged commit 05de495 into main Dec 8, 2023
3 checks passed
@pabois pabois deleted the csp branch December 8, 2023 10:22
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

2 participants