[ 
https://issues.apache.org/jira/browse/HADOOP-18575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17648771#comment-17648771
 ] 

ASF GitHub Bot commented on HADOOP-18575:
-----------------------------------------

pjfanning commented on code in PR #5224:
URL: https://github.com/apache/hadoop/pull/5224#discussion_r1051079320


##########
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/XMLUtils.java:
##########
@@ -156,8 +161,19 @@ public static SAXTransformerFactory 
newSecureSAXTransformerFactory()
           throws TransformerConfigurationException {
     SAXTransformerFactory trfactory = (SAXTransformerFactory) 
SAXTransformerFactory.newInstance();
     trfactory.setFeature(XMLConstants.FEATURE_SECURE_PROCESSING, true);
-    trfactory.setAttribute(XMLConstants.ACCESS_EXTERNAL_DTD, "");
-    trfactory.setAttribute(XMLConstants.ACCESS_EXTERNAL_STYLESHEET, "");
+    bestEffortSet(trfactory, XMLConstants.ACCESS_EXTERNAL_DTD, "");
+    bestEffortSet(trfactory, XMLConstants.ACCESS_EXTERNAL_STYLESHEET, "");
     return trfactory;
   }
+
+  private static boolean bestEffortSet(TransformerFactory transformerFactory,

Review Comment:
   added javadoc





> make transformer factory creation more lenient
> ----------------------------------------------
>
>                 Key: HADOOP-18575
>                 URL: https://issues.apache.org/jira/browse/HADOOP-18575
>             Project: Hadoop Common
>          Issue Type: Task
>          Components: common
>    Affects Versions: 3.4.0, 3.3.5
>            Reporter: PJ Fanning
>            Priority: Major
>              Labels: pull-request-available
>
> Issues raised today in HADOOP-18469 about particular attributes that are not 
> supported by Saxon transformer.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to