• Type: Bug
• Resolution: Done
• Priority: Major
• Fix Version/s: jbossws-cxf-5.0.0.Beta2
• Affects Version/s: jbossws-cxf-4.3
• Component/s: jbossws-cxf
• Labels:

  None

[JBWS-3851] WSDL publisher incorrectly attempt to publish external HTTPS schema import

Collapse comment: r searls added a comment - 2014/10/28 10:58 AM

r searls added a comment - 2014/10/28 10:58 AM

Code change checked in.
https://svn.jboss.org/repos/jbossws/common/trunk (version 3.0.0-SNAPSHOT)
Committed revision 19034.

— diff
— src/main/java/org/jboss/ws/common/utils/AbstractWSDLFilePublisher.java (revision 19033)
+++ src/main/java/org/jboss/ws/common/utils/AbstractWSDLFilePublisher.java (working copy)
@@ -207,7 +207,7 @@
final String ln = childElement.getLocalName();
if ("import".equals(ln) || "include".equals(ln)) {
String schemaLocation = childElement.getAttribute("schemaLocation");

• if (schemaLocation.length() > 0 && schemaLocation.startsWith("http://") == false)
  + if (schemaLocation.length() > 0 && schemaLocation.startsWith("http://") == false && schemaLocat
  ion.startsWith("https://") == false)
  {
  // infinity loops prevention
  if (!published.contains(schemaLocation))

Expand comment: r searls added a comment - 2014/10/28 10:58 AM

r searls added a comment - 2014/10/28 10:58 AM Code change checked in. https://svn.jboss.org/repos/jbossws/common/trunk (version 3.0.0-SNAPSHOT) Committed revision 19034. — diff — src/main/java/org/jboss/ws/common/utils/AbstractWSDLFilePublisher.java (revision 19033) +++ src/main/java/org/jboss/ws/common/utils/AbstractWSDLFilePublisher.java (working copy) @@ -207,7 +207,7 @@ final String ln = childElement.getLocalName(); if ("import".equals(ln) || "include".equals(ln)) { String schemaLocation = childElement.getAttribute("schemaLocation"); if (schemaLocation.length() > 0 && schemaLocation.startsWith("http://") == false) + if (schemaLocation.length() > 0 && schemaLocation.startsWith("http://") == false && schemaLocat ion.startsWith("https://") == false) { // infinity loops prevention if (!published.contains(schemaLocation))

Collapse comment: r searls added a comment - 2014/10/28 8:10 AM

r searls added a comment - 2014/10/28 8:10 AM

This issue is the same as for https://issues.jboss.org/browse/JBWS-3827 except it is for a
schema import and not a wsdl import.

Expand comment: r searls added a comment - 2014/10/28 8:10 AM

r searls added a comment - 2014/10/28 8:10 AM This issue is the same as for https://issues.jboss.org/browse/JBWS-3827 except it is for a schema import and not a wsdl import.