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

Already on GitHub? Sign in to your account

WSDL port location parsing issue fixed #62

Merged
merged 2 commits into from May 23, 2012

Conversation

Projects
None yet
2 participants

Consider the following part of a WSDL port section:

<port binding="binding" name="service">
    <soap:address location="http://serviceaddress"/>
    <other:address random="service"/>
</port>

Before, location was at first set to "http://serviceaddress", which is correct, but later reset to "undefined".
This fix checks if address tag has location or not. In case of two address locations, the latter one is chosen.

mikk soone added some commits May 23, 2012

Fixed WSDL port multiple addresses issue. When WSDL port has multiple…
… addresses (only one of them contains location) then should not overwrite correct location with undefined value
Collaborator

milewise commented May 23, 2012

thanks for the fix! merging.

milewise added a commit that referenced this pull request May 23, 2012

Merge pull request #62 from mikksoone/master
WSDL port location parsing issue fixed

@milewise milewise merged commit 8cf215b into vpulim:master May 23, 2012

diarmaidm pushed a commit to diarmaidm/node-soap that referenced this pull request Feb 3, 2016

Merge pull request #62 from mikksoone/master
WSDL port location parsing issue fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment