vrk-kpa / xroad-joint-development

Unmaintained repository. Development moved to: https://github.com/nordic-institute/X-Road-development
19 stars 8 forks source link

As a Product Owner I want that the Wsdlvalidator is not anymore checking the SOAP faults and Wsdlvalidator is also included to RHEL packaging so that it's usability is improved #147

Closed hanhaka closed 7 years ago

hanhaka commented 7 years ago

Affected components: Wsdlvalidator, packaging Affected documentation: - Estimated delivery: Q2/2017 External reference: https://jira.csc.fi/browse/PVAYLADEV-730

Problem Wsdlvalidator checks for the some reason SOAP faults when importing the WSDL. It seems that the checkForSoapFault() is relict from X-Road v5 migration, where service mediator was developed. So it's safe to remove this function.

In practice when the WSDL is imported in the security server UI it logs an exception to jetty.log as follows: 2017-04-21 16:47:31,152 [qtp972765878-12] INFO ee.ria.xroad.proxyui.WSDLParser - Exception while parsing: {} ee.ria.xroad.common.CodedException: InvalidSoap: Malformed SOAP message: envelope missing at ee.ria.xroad.common.message.SaxSoapParserImpl$XRoadSoapHandler.handleRootElement(SaxSoapParserImpl.java:368) ~[common-util-1.0.jar:na] at ee.ria.xroad.common.message.SaxSoapParserImpl$XRoadSoapHandler.startElement(SaxSoapParserImpl.java:343) ~[common-util-1.0.jar:na] at org.apache.xerces.parsers.AbstractSAXParser.startElement(AbstractSAXParser.java:498) ~[xercesImpl-2.11.0.SP5.jar:2.11.0.SP5]

Wsdlvalidator must be included also to RHEL packaging as it is already included to Ubuntu packaging.

Acceptance criteria

hanhaka commented 7 years ago

Fixed in version 6.15.0