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
If you try to use yahp on an html document that looks like this:
<html>
<script>
var foo = "</foo>"
</script>
</html>
then it fails with a null pointer exception. It's because jtidy fails on those documents and yahp tries to use the original html, but its tidiness assumptions get violated.
See this jtidy bug report: http://sourceforge.net/p/jtidy/discussion/41437/thread/408cffe8/
Right now yahp is stripping out <script> elements, but it does it after passing the html through jtidy. I think a reasonable workaround would be to strip out the scripts before calling jtidy, so jtidy won't fail. I am working on coding this fix.
Is this project still active enough for a pull request that fixes this to get merged?
The text was updated successfully, but these errors were encountered:
If you try to use yahp on an html document that looks like this:
then it fails with a null pointer exception. It's because jtidy fails on those documents and yahp tries to use the original html, but its tidiness assumptions get violated.
See this jtidy bug report: http://sourceforge.net/p/jtidy/discussion/41437/thread/408cffe8/
Right now yahp is stripping out
<script>elements, but it does it after passing the html through jtidy. I think a reasonable workaround would be to strip out the scripts before calling jtidy, so jtidy won't fail. I am working on coding this fix.Is this project still active enough for a pull request that fixes this to get merged?
The text was updated successfully, but these errors were encountered: