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.

Sign up for GitHub

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

Jump to bottom

wsse does not sign parts required by WSDL #497

Open
andreixk opened this issue Jul 13, 2017 · 2 comments · May be fixed by #747 or #1428
Open

wsse does not sign parts required by WSDL #497

andreixk opened this issue Jul 13, 2017 · 2 comments · May be fixed by #747 or #1428
Labels
enhancement

Comments

@andreixk
Copy link

If the WSDL file specifies the document parts that need to be signed, as in example below, the signature.py module doesn't sign any of them except for Body and Timestamp (which are hard coded for some reason)

<sp:SignedParts xmlns="http://schemas.xmlsoap.org/wsdl/" xmlns:ebs="http://ebs.health.ontario.ca/" xmlns:hcv="http://hcv.health.ontario.ca/" xmlns:idp="http://idp.ebs.health.ontario.ca/" xmlns:msa="http://msa.ebs.health.ontario.ca/" xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:sp="http://docs.oasis-open.org/ws-sx/ws-securitypolicy/200512" xmlns:wsdlsoap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:wsse="http://docs.oasis-open.org/wss/2004/01/oasis-200401-wss-wssecurity-secext-1.0.xsd" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
	<sp:Header Name="EBS" Namespace="http://ebs.health.ontario.ca/"/>
	<sp:Header Name="MSA" Namespace="http://msa.ebs.health.ontario.ca/"/>
	<sp:Header Name="Timestamp" Namespace="http://docs.oasis-open.org/wss/2004/01/oasis-200401-wss-wssecurity-utility-1.0.xsd"/>
	<sp:Header Name="UsernameToken" Namespace="http://docs.oasis-open.org/wss/2004/01/oasis-200401-wss-wssecurity-utility-1.0.xsd"/>
	<sp:Body/>
</sp:SignedParts>
@mvantellingen
Copy link
Owner

Ah interesting, it is indeed hardcoded currently. Are you able to open a PR to implement this?

@tpazderka tpazderka linked a pull request May 7, 2018 that will close this issue
Open
@ba1dr
Copy link

ba1dr commented Oct 12, 2021

What a pity that this feature has not been merged yet. Any chance to have it done soon?

@blaggacao blaggacao linked a pull request Sep 21, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Signed parts tpazderka/python-zeep
signed parts blaggacao/python-zeep
3 participants
@mvantellingen @ba1dr @andreixk