GH-2723: Handle unsupported XML properties

Fixes #2723

Not all XML components support all the configuration properties.
For example Saxon HE doesn't support `XMLConstants.ACCESS_EXTERNAL_DTD`
and end up with an exception like:
`IllegalArgumentException: Unknown configuration property http://javax.xml.XMLConstants/property/accessExternalDTD`

* Change `XsltPayloadTransformer` to re-use `TransformerFactoryUtils`
from spring-ws as a centralized source of `TransformerFactory`
configuration.
* Wrap `XMLConstants.ACCESS_EXTERNAL_STYLESHEET` to the `try..catch`
and log INFO about not supported property

**Cherry-pick to 5.0.x & 4.3.x**

Author: artembilan

spring-integration-xml/src/main/java/org/springframework/integration/xml/transformer/XsltPayloadTransformer.java

@@ -240,25 +240,39 @@ protected void onInit() throws Exception {
 		super.onInit();
 		this.evaluationContext = ExpressionUtils.createStandardEvaluationContext(getBeanFactory());
 		if (this.templates == null) {
-			TransformerFactory transformerFactory;
-			if (this.transformerFactoryClassName != null) {
-				transformerFactory = TransformerFactory.newInstance(this.transformerFactoryClassName,
-						this.classLoader);
-			}
-			else {
-				transformerFactory = TransformerFactoryUtils.newInstance();
-			}
-			transformerFactory.setAttribute(XMLConstants.ACCESS_EXTERNAL_DTD, "");
-			transformerFactory.setAttribute(XMLConstants.ACCESS_EXTERNAL_STYLESHEET, "file,jar:file");
 			try {
+			TransformerFactory transformerFactory = createTransformerFactory();
 				this.templates = transformerFactory.newTemplates(createStreamSourceOnResource(this.xslResource));
 			}
-			catch (TransformerConfigurationException | IOException e) {
+			catch (ClassNotFoundException | TransformerConfigurationException | IOException e) {
 				throw new IllegalStateException(e);
 			}
 		}
 	}
 
+	private TransformerFactory createTransformerFactory() throws ClassNotFoundException {
+		TransformerFactory transformerFactory;
+		if (this.transformerFactoryClassName != null) {
+			@SuppressWarnings("unchecked")
+			Class<TransformerFactory> transformerFactoryClass =
+					(Class<TransformerFactory>) ClassUtils.forName(this.transformerFactoryClassName, this.classLoader);
+			transformerFactory = TransformerFactoryUtils.newInstance(transformerFactoryClass);
+		}
+		else {
+			transformerFactory = TransformerFactoryUtils.newInstance();
+		}
+		try {
+			transformerFactory.setAttribute(XMLConstants.ACCESS_EXTERNAL_STYLESHEET, "file,jar:file");
+		}
+		catch (IllegalArgumentException ex) {
+			if (logger.isInfoEnabled()) {
+				logger.info("The '" + XMLConstants.ACCESS_EXTERNAL_STYLESHEET + "' property is not supported by "
+						+ transformerFactory.getClass().getCanonicalName());
+			}
+		}
+		return transformerFactory;
+	}
+
 	@Override
 	protected Object doTransform(Message<?> message) throws Exception {
 		Transformer transformer = buildTransformer(message);

spring-integration-xml/src/test/java/org/springframework/integration/xml/transformer/XsltPayloadTransformerTests.java

@@ -27,7 +27,6 @@
 import javax.xml.transform.Templates;
 import javax.xml.transform.TransformerException;
 import javax.xml.transform.TransformerFactory;
-import javax.xml.transform.TransformerFactoryConfigurationError;
 import javax.xml.transform.dom.DOMResult;
 
 import org.junit.Before;
@@ -148,7 +147,7 @@ public void testXsltPayloadWithTransformerFactoryClassName() throws Exception {
 				transformed);
 	}
 
-	@Test(expected = TransformerFactoryConfigurationError.class)
+	@Test(expected = IllegalStateException.class)
 	public void testXsltPayloadWithBadTransformerFactoryClassname() throws Exception {
 		transformer = new XsltPayloadTransformer(getXslResourceThatOutputsText(), "foo.bar.Baz");
 		transformer.setBeanFactory(Mockito.mock(BeanFactory.class));
@@ -249,6 +248,7 @@ public StubResultTransformer(Object objectToReturn) {
 			this.objectToReturn = objectToReturn;
 		}
 
+		@Override
 		public Object transformResult(Result result) {
 			return objectToReturn;
 		}