You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Mar 10, 2022. It is now read-only.
After transforming scripts, you could end up with an HTML file that has one or more instances of <script></script>.
For example, since description("Test description"); calls are removed, if you had HTML that looked like: <script>description("Test description");</script> the transformed HTML would be left with <script></script>.
It would be nice (and likely not very difficult) to remove these empty tags in injectScripts.js.
Although, some subtleties include: make sure that the scriptCount is incremented and the scripts array is handled correctly after deletion, that traversal continues correctly after deletion, and that the <script> tag is truly empty including no attributes within the tag.
After transforming scripts, you could end up with an HTML file that has one or more instances of
<script></script>.For example, since
description("Test description");calls are removed, if you had HTML that looked like:<script>description("Test description");</script>the transformed HTML would be left with<script></script>.It would be nice (and likely not very difficult) to remove these empty tags in
injectScripts.js.Although, some subtleties include: make sure that the
scriptCountis incremented and thescriptsarray is handled correctly after deletion, that traversal continues correctly after deletion, and that the<script>tag is truly empty including no attributes within the tag.