New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WsdlXsdSchema NullPointerException for valid WSDLs #241

Closed
gucce opened this Issue May 18, 2017 · 0 comments

Comments

Projects
None yet
2 participants
@gucce
Contributor

gucce commented May 18, 2017

Background

Citrus version 2.7.1
SOAP 1.1

Problem

I have a WSDL which does not contain a <wsdl:types> tag but it imports other WSDLs (via <wsdl:import>) which have a <wsdl:types> tag and reference XSDs. So this is a transitive XSD dependency.
However, Citrus cannot load the schemas due to a NullPointerException in class WsdlXsdSchema because the code expects a <wsdl:types> tags to be present (which is a false assumption).

See line:
https://github.com/christophd/citrus/blob/master/modules/citrus-core/src/main/java/com/consol/citrus/xml/schema/WsdlXsdSchema.java#L94

private Resource loadSchemas(Definition definition) throws WSDLException, IOException, TransformerException, TransformerFactoryConfigurationError {
        Types types = definition.getTypes();
        List<?> schemaTypes = types.getExtensibilityElements();

types is null in my case which provokes the NullPointerException on types.getExtensibilityElements().

Solution

Null check for types and process imported WSDLs.

@gucce gucce added the Type: Bug label May 18, 2017

@christophd christophd added the READY label May 28, 2017

@christophd christophd added this to the v2.7.2 milestone May 28, 2017

@christophd christophd added IN PROGRESS and removed READY labels Jun 29, 2017

@christophd christophd self-assigned this Jun 29, 2017

@christophd christophd removed the IN PROGRESS label Jun 29, 2017

christophd added a commit that referenced this issue Jun 29, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment