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

Utiliser l'attribut crossorigin #1275

Open
maxime-carbonneau opened this issue Sep 16, 2024 · 6 comments
Open

Utiliser l'attribut crossorigin #1275

maxime-carbonneau opened this issue Sep 16, 2024 · 6 comments
Labels

Comments

@maxime-carbonneau
Copy link

Quand TAC est chargé via un CDN, il y a des enjeux de CORS.

Est-ce que c'est possible d'ajouter l'attribut crossorigin=anonymous pour les addInternalScript et pour link_element dans Step 1: load css

@AmauriC
Copy link
Owner

AmauriC commented Sep 16, 2024

Je ne suis pas fan d'ajouter des attributs pour contrer une mesure de sécurité.

Si la politique CORS d'un site est strict, il faut ajouter dedans le CDN et les différents domaines.

@maxime-carbonneau
Copy link
Author

Je comprends, moi non plus je ne suis pas fan.

En même temps, ce n'est pas l'interprétation que je fais de l'attribut. D'après ma compréhension, comme le serveur va retourner l'en-tête Access-Control-Allow-Origin: *, quand mon browser demande le fichier https://cdn.jsdelivr.net/npm/[email protected]/css/tarteaucitron.min.css, l'attribut permet de confirmer qu'il n'y aura pas d'échange de cookie.

Par contre, peut-être que c'est seulement pour le tag link que c'est pertinent.

Voici le en-tête que retourne cdn.jsdelivr.net
Access-Control-Allow-Origin: *
Access-Control-Expose-Headers: *
Cross-Origin-Resource-Policy: cross-origin

@AmauriC
Copy link
Owner

AmauriC commented Sep 17, 2024

Ok. Pour bien comprendre, actuellement les fichiers CDN ne sont pas chargés par le navigateur?
En ajoutant crossorigin=anonymous sur la balise tarteaucitron, idem?

@maxime-carbonneau
Copy link
Author

Les fichiers sont bien chargés par le navigateur. Cependant, la consultation des règles CSS en javascript document.styleSheets pour TAC ne permet pas la consultation de clés cssRules et/ou rules. Cela génère l'erreur « SecurityError: Not allowed to access cross-origin stylesheet ».

C'est en cherchant à comprendre l'origine de l'erreur que j'ai découvert l'attribut crossorigin.

@AmauriC
Copy link
Owner

AmauriC commented Sep 17, 2024

J'aurai aussi appris quelque chose 😃
Techniquement c'est facile à ajouter mais j'aimerai bien avoir l'avis d'utilisateur calé en CORS pour valider ou non le changement.

Copy link

stale bot commented Oct 30, 2024

This issue has been automatically marked as inactive because it has not had activity for 20 days. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the inactive label Oct 30, 2024
@AmauriC AmauriC removed the inactive label Oct 30, 2024
@AmauriC AmauriC added the active label Nov 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants