muchl / owaspantisamy

Automatically exported from code.google.com/p/owaspantisamy
0 stars 0 forks source link

out of date dependancy on nekohtml (breaks Intelij's parsing of schemas) #118

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
The current NekoHTML dependency is:

        <dependency>
            <groupId>net.sourceforge.nekohtml</groupId>
            <artifactId>nekohtml</artifactId>
            <version>1.9.12</version>
        </dependency>

This version is out of date.  Having it on your classpath breaks Intellij as it 
can no longer parse a schema.

 The version of NekoHTML that in this code is blowing up Intellij when you try to parse any schema with antisamy on your classpath.  

Versions 1.9.13 and 1.9.15 work just fine but versions 1.9.12 & 1.9.14 do not.  

I could use dependency management to force my build to use a later version, 
which solves the parsing errors in Intellij, but when I try to run the build 
for antisamy with one of these versions, I get test failures so using 
dependency management to upgrade the version might be silently breaking 
antisamy functionality in my project.

Original issue reported on code.google.com by jacob.co...@gmail.com on 12 Sep 2011 at 4:52

GoogleCodeExporter commented 8 years ago
No point in not upgrading this - we'll do so.

Original comment by arshan.d...@gmail.com on 14 Sep 2011 at 6:31

GoogleCodeExporter commented 8 years ago
Unfortunately, there is a point. More recent versions of Neko cause some of our 
test cases to fail. Specifically, it seems to be having trouble allowing sparse 
HTML fragments like "<td>test</td>", which is sanitized to "test".

Original comment by arshan.d...@gmail.com on 15 Sep 2011 at 2:06

GoogleCodeExporter commented 8 years ago
When Neko fixes their bug, we can consider upgrading again.

Original comment by arshan.d...@gmail.com on 15 Sep 2011 at 8:13