Skip to content
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

feat(wsdl-meta): add soap and soap12 namespaces if WSDL contains both #1047

Merged
merged 2 commits into from
May 31, 2024

Conversation

Yozer
Copy link
Contributor

@Yozer Yozer commented Mar 29, 2024

When a service has bindings with mixed soap11/soap12 versions the produced WSDL was correct but every "soap11" element had extra namespace.
This PR adds soap namespace to the wsdl definition.

Copy link
Collaborator

@andersjonsson andersjonsson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unless I'm missing something there is a duplicate check in one of your if-statements.
Other than that it looks really good. Thanks!

src/SoapCore/Meta/MetaMessage.cs Outdated Show resolved Hide resolved
@Yozer
Copy link
Contributor Author

Yozer commented May 30, 2024

Unless I'm missing something there is a duplicate check in one of your if-statements. Other than that it looks really good. Thanks!

Yep, that was a typo, thanks!

@andersjonsson
Copy link
Collaborator

Thanks!

@andersjonsson andersjonsson merged commit 0c709cd into DigDes:develop May 31, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants