carlosame

I found this issue while browsing Batik's JIRA, although I'm not related to the Batik project (nor the ASF) in any way. I do not think that there is anything...

Incidentally, I see that you are checking against old CSS 2.1 units. ```xml ``` But today's websites use a wider range of values and units, see [CSS Values and Units...

> I don't know why some of the units were left out really. We could expand the list even more with absolute and relative units. I haven't seen that updated...

Do you have the `org.xml.sax.driver` system property set to anything? Because you are probably using some version of the _Xerces-j_ SAX parser under the hood, and therefore this would not...

> For making it easier, I give you the Junit test here. Thank you, although the line: ` if (nowNode instanceof Comment) { ` should be: ` if (nowNode instanceof...

I tested the patch, and gives the desired result in `MANIFEST.MF`: ``` Manifest-Version: 1.0 Automatic-Module-Name: org.dom4j ```

The most convenient way to set those limits is by: ```java SAXReader builder = new SAXReader(); builder.setFeature(javax.xml.XMLConstants.FEATURE_SECURE_PROCESSING, true); ``` However, to fully use the protections other (internal) changes in dom4j...

Duplicate of #67 (see also PR #84).

To summarize what's at my post in the css4j forum: With dom4j 2.1.1: 1) Removes entities that are in "ENTITIES Latin 1" but not entities in e.g. "ENTITIES Special". If...

I was trying to look at this issue more closely and came up to [commit 53f923a](https://github.com/dom4j/dom4j/commit/53f923ad459caa70f16f43bd3cb3d474db9818b4#diff-337df36ef9955490d72e90cbc8881003), where at lines 111 and 112 of the `SAXHelper` file you can read this:...