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

[FRONT] Migrer la gestion des CSP en Record #3472

Open
Mintoo200 opened this issue Nov 26, 2024 · 0 comments
Open

[FRONT] Migrer la gestion des CSP en Record #3472

Mintoo200 opened this issue Nov 26, 2024 · 0 comments
Assignees

Comments

@Mintoo200
Copy link
Contributor

Mintoo200 commented Nov 26, 2024

Les CSP sont aujourd'hui sous la forme d'une string avec des retours à la ligne qu'il faut ensuite supprimer dans la config (contentSecurityPolicy.replace(/\s{2,}/g, ' ')).
Passer sur un Record<CSP, AllowedPattern[]> nous permettrait d'y voir un peu plus clair et d'éviter les typos du style ajouter une valeur après le séparateur, oublier le séparateur, espace en trop ou en moins...

Et en profiter pour faire un peu de ménage dans ce qui devrait être feature flippé ou configuré dans le .env et pour trouver un moyen d'activer les CSP en local (le problème actuellement est que si on connecte l'env local à la recette ou la prod, les images ne chargeront pas puisque self représentera localhost et pas 1jeune1solution.gouv.fr)

@Mintoo200 Mintoo200 self-assigned this Nov 26, 2024
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

No branches or pull requests

1 participant