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

Add CORS permission in header via netlify.toml configuration file #19

Closed
wants to merge 1 commit into from

Conversation

phe-sto
Copy link
Contributor

@phe-sto phe-sto commented Apr 11, 2021

According to this documentation https://docs.netlify.com/configure-builds/file-based-configuration/ it should be enough to enable CORS. Even tough I was not able to test it, it should do the trick.

@JulienRobberechts
Copy link

Je ne suis bien sûr mais pour moi il n'y a pas besoin de header CORS pour afficher une iframe. CORS pour moi c'est uniquement pour les appels XHR (appel js vers un backend d'un autre domaine). Par exemple j'ai fait un test d'insertion d'une iframe dans un portail: https://www.enthic.tk/entreprises.html
Et ça fonctionne, donc j'ai peut être loupé une problématique de sécu mais pour moi il n'y a rien à faire.

@JulienRobberechts
Copy link

Est-ce que le but est juste d'afficher une Iframe ou bien d'intéragir avec le contenu de l'iFrame ?

@phe-sto
Copy link
Contributor Author

phe-sto commented Apr 12, 2021

En effet ça marche bien je me suis emellé les pinceaux avec la protection contre le clickjacking avec le header X-Frame-Options. Je crois qu'on peut oublier la PR.

@JulienRobberechts JulienRobberechts linked an issue Apr 12, 2021 that may be closed by this pull request
@JulienRobberechts
Copy link

En effet ça marche bien je me suis emellé les pinceaux avec la protection contre le clickjacking avec le header X-Frame-Options. Je crois qu'on peut oublier la PR.

Oui je viens de découvrir X-Frame-Options dans mes recherches :-)
Je ferme la PR.

@JulienRobberechts JulienRobberechts deleted the cors-configuration branch April 12, 2021 17:35
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.

Intégrer le site enthic-dataviz dans enthic.fr
2 participants