[utilities COMMIT] /java-support/trunk/src/main/java/net/shibboleth/utilities/java/support/xml/SchemaBuilder.java

[noreply at shibboleth.net]

Author: scantor
Date: Fri Jan 24 18:18:07 2014
New Revision: 526

URL: http://svn.shibboleth.net/view/utilities?rev=526&view=rev
Log:
Rethink using external limiting props, since LSResourceResolver defaults to no access anyway.

Modified:
    java-support/trunk/src/main/java/net/shibboleth/utilities/java/support/xml/SchemaBuilder.java

Modified: java-support/trunk/src/main/java/net/shibboleth/utilities/java/support/xml/SchemaBuilder.java
URL: http://svn.shibboleth.net/view/utilities/java-support/trunk/src/main/java/net/shibboleth/utilities/java/support/xml/SchemaBuilder.java?rev=526&r1=525&r2=526&view=diff
==============================================================================
--- java-support/trunk/src/main/java/net/shibboleth/utilities/java/support/xml/SchemaBuilder.java (original)
+++ java-support/trunk/src/main/java/net/shibboleth/utilities/java/support/xml/SchemaBuilder.java Fri Jan 24 18:18:07 2014
@@ -259,18 +259,8 @@
             }
         }
 
-        if (properties.isEmpty()) {
-            try {
-                schemaFactory.setProperty("http://javax.xml.XMLConstants/property/accessExternalDTD", "");
-                schemaFactory.setProperty("http://javax.xml.XMLConstants/property/accessExternalSchema", "");
-            } catch (SAXNotRecognizedException e) {
-                log.warn("Default properties limiting external schema/DTD access not supported by JAXP implementation",
-                        e);
-            }
-        } else {
-            for (final Map.Entry<String, Object> entry : properties.entrySet()) {
-                schemaFactory.setProperty(entry.getKey(), entry.getValue());
-            }
+        for (final Map.Entry<String, Object> entry : properties.entrySet()) {
+            schemaFactory.setProperty(entry.getKey(), entry.getValue());
         }
         
         schemaFactory.setErrorHandler(errorHandler);