Some scripts not removed #33

New Issue

koalp · 2021-05-10T18:21:59+02:00

koalp commented

2021-05-10 18:21:59 +02:00

Description

Some scripts are not removed, wether in header or in body

For css, another issue have been : #34

Steps to reproduce

Download an article from le monde diplomatique (ex: https://www.monde-diplomatique.fr/2021/05/MALET/63059)
Open source and search for <script, there are occurences

Expected behavior

All <script> should be removed

**Description** Some scripts are not removed, wether in header or in body For css, another issue have been : #34 **Steps to reproduce** 1. Download an article from le monde diplomatique (ex: https://www.monde-diplomatique.fr/2021/05/MALET/63059) 2. Open source and search for `<script`, there are occurences **Expected behavior** All `<script>` should be removed

koalp added the

labels 2021-05-10 18:21:59 +02:00

koalp added this to the v0.1.1 - multiple bug resolution and enhancements milestone 2021-05-10 18:24:27 +02:00

koalp commented

2021-05-10 18:40:40 +02:00

Seems that it is linked to the fact that CSS are injected without any sanitizing and one css links to a 404.

It will be solved locally by refusing 404, but the issue #34 about css sanitizing should solve this issue globally.

It seem that the issue #35 is also caused by this css injection

Seems that it is linked to the fact that CSS are injected without any sanitizing and one css links to a 404. It will be solved locally by refusing 404, but the issue #34 about css sanitizing should solve this issue globally. It seem that the issue #35 is also caused by this css injection

koalp referenced a pull request that will close this issue

2021-05-10 19:31:20 +02:00

add several newspapers #36

koalp added this to the 10 may - 21 may 2021 project 2021-05-13 20:43:21 +02:00

koalp added

and removed

labels 2021-05-13 20:46:55 +02:00

koalp closed this issue