diff --git a/spring-xml/src/main/java/org/springframework/xml/transform/TransformerFactoryUtils.java b/spring-xml/src/main/java/org/springframework/xml/transform/TransformerFactoryUtils.java index b365b371e..b7693707a 100644 --- a/spring-xml/src/main/java/org/springframework/xml/transform/TransformerFactoryUtils.java +++ b/spring-xml/src/main/java/org/springframework/xml/transform/TransformerFactoryUtils.java @@ -19,12 +19,17 @@ import javax.xml.transform.TransformerFactory; import javax.xml.transform.TransformerFactoryConfigurationError; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; + /** * @author Greg Turnquist * @since 3.0.5 */ public class TransformerFactoryUtils { + private static final Log log = LogFactory.getLog(TransformerFactoryUtils.class); + /** * Build a new {@link TransformerFactory} using the default constructor. */ @@ -50,7 +55,13 @@ public static TransformerFactory newInstance(Class * Prevent external entities from accessing. */ private static TransformerFactory defaultSettings(TransformerFactory factory) { - factory.setAttribute(XMLConstants.ACCESS_EXTERNAL_DTD, ""); + try { + factory.setAttribute(XMLConstants.ACCESS_EXTERNAL_DTD, ""); + } catch (IllegalArgumentException e) { + if (log.isWarnEnabled()) { + log.warn(XMLConstants.ACCESS_EXTERNAL_DTD + " property not supported by " + factory.getClass().getCanonicalName()); + } + } factory.setAttribute(XMLConstants.ACCESS_EXTERNAL_STYLESHEET, ""); return factory; }