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.

Sign up for GitHub

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

Jump to bottom

Externaliser le code de l'analyseur #19

Open
jycr opened this issue Feb 15, 2022 · 1 comment · May be fixed by #21
Open

Externaliser le code de l'analyseur #19

jycr opened this issue Feb 15, 2022 · 1 comment · May be fixed by #21

Comments

@jycr
Copy link

jycr commented Feb 15, 2022

J'ai le même problème de CSP (Content Security Policy) que celui mentionné sur : #16 (comment)
Que créé l'issue ici afin de tracer le problème.
@jycr jycr mentioned this issue Feb 15, 2022
Closed
jycr pushed a commit to jycr/GreenIT-Analysis-cli that referenced this issue Feb 15, 2022
@hayaofr @jycr
feat: Changement de la méthode d'analyse en recréant le DOM de la pag…
3c345ef 
…e en mémoire.

Cela évite de charge le code de l'analyseur dans la page elle même.
Cela permet d'analyser des sites ayant des Content Security Policy de configurés.

Ces modifications sont issues de la proposition de @hayaofr (via cnumr#16), j'ai simplement ici réduit au maximum le nombre de modification pour faciliter la relecture de la PR

Fix: cnumr#19
@jycr jycr linked a pull request Feb 15, 2022 that will close this issue
Open
jycr pushed a commit to jycr/GreenIT-Analysis-cli that referenced this issue Feb 15, 2022
@hayaofr @jycr
feat: Changement de la méthode d'analyse en recréant le DOM de la pag…
ae9b82d 
…e en mémoire.

Cela évite de charge le code de l'analyseur dans la page elle même.
Cela permet d'analyser des sites ayant des Content Security Policy de configurés.

Ces modifications sont issues de la proposition de @hayaofr (via cnumr#16), j'ai simplement ici réduit au maximum le nombre de modification pour faciliter la relecture de la PR

Fix: cnumr#19
@jpreisner
Copy link
Collaborator

Merci à vous 2 pour la proposition et la PR. J'essaye de prendre le temps de tester tout ça et je vous fais un retour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@jycr @jpreisner