search

cnumr / GreenIT-Analysis-cli

Wrapper de l'extension GreenIT Analysis
GNU Affero General Public License v3.0
46 stars 19 forks source link

Externaliser le code de l'analyseur #21

Open jycr opened 2 years ago

jycr commented 2 years ago

Changement de la méthode d'analyse en recréant le DOM de la page 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 https://github.com/cnumr/GreenIT-Analysis-cli/pull/16), j'ai simplement ici réduit au maximum le nombre de modification pour faciliter la relecture de la PR

Fix: https://github.com/cnumr/GreenIT-Analysis-cli/issues/19

jpreisner commented 2 years ago

Hello @jycr, désolé de n'avoir pu regarder ta PR que maintenant. Je te laisse prendre connaissance de mes commentaires, je pense que nous avons besoin d'échanger davantage sur le problème et voir ce qu'on peut faire pour éviter au maximum de modifier le coeur de GreenIT-Analysis dans la version cli. Si tu as aussi des problèmes avec le plugin, peut-être qu'il faudrait créer une PR côté GreenIT-Analysis ?