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

Tweaked code for Cpix signature support #55

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ap13istream
Copy link

@ap13istream ap13istream commented Mar 9, 2020

Changes:

  • Make id element optional
  • The transform element is also optional, and the caconicalization is implied and declared outside of the transform element.

trying to make it work for the CPIX xml document signature

<CPIX
	xmlns="urn:dashif:org:cpix">
<!-- Body of the Document -->
	<ds:Signature>
		<ds:SignedInfo>
			<ds:CanonicalizationMethod Algorithm="http://www.w3.org/TR/2001/REC-xml-c14n-20010315"/>
			<ds:SignatureMethod Algorithm="http://www.w3.org/2001/04/xmldsig-more#rsa-sha512"/>
			<ds:Reference URI="">
				<ds:Transforms>
					<ds:Transform Algorithm="http://www.w3.org/2000/09/xmldsig#enveloped-signature"/>
				</ds:Transforms>
				<ds:DigestMethod Algorithm="http://www.w3.org/2001/04/xmlenc#sha512"/>
				<ds:DigestValue>YzsBBIZCWia4+DeVdFho6AgomXbsCIiQ0lovzl+u09/ccWF/
+EybgMDHUeecu6SqDqdfIEXVkoo1W+CQHnoLOQ==</ds:DigestValue>
			</ds:Reference>
		</ds:SignedInfo>
		<ds:SignatureValue>Rso3WHnj9bp9oXKdCWFNwtcjucdS9iR8x6xUlduAlhxxIpSeSZrxcnR8ZpDzLU
DuVnuFTPI6hGCD4qAnG8GgefhZIdMZ2SQWIhmaDVsRlfQjytC29HtRy4V/
Jp95XYigNHtlJFakh8kiWjDbTrXmAFBAAzMP5bn1LHEMY6QmIDQxUtTgWD8lO8lpgcjdJptST2WbUcAUx
+vXQPdhYebZSt91fOx99n+jt619HfMGd5H8NZDjdY+Eu3937MAxj4bKv+Lg+ZKz4FMz7bFj0bfyWkap9/
nbaFPW1wT1zQlA3oSz690jjKMcsYL32ZxLqpNQI2Nqmkek7jGaBLLW01yQOkvvvUNiJgVzoWkE7KgzRTy
vMtcTd9dlxLBsRPl6hxUSnFx55WgJaF6w8PBkrvqSv4kGhZ193H2rcSzcMcJAt3a1dTDFF7xd5fhMP1fJ
FZChVf3TsChQ0GMWPQrObajuvjpCljl/
p5FiSyJeLl6/2wvytoaNUtAiLv+DSxi7k8FOr24BuPBOeuRYnyV9WeA96mkfxhKVKzh3u9bwO9+UpZzb8
T2fJN8zwF6k34nfDbC8ccCdSPllLrHesKLTeL4xWvKZQMP2vWWOyJobFxL97dercnOVFEPN2tddzypyhR
47rUfHTY/b13gMjHX2b7ph6hV4uC/Rg1OaSAGB5kzURBA=</ds:SignatureValue>
		<ds:KeyInfo>
			<ds:X509Data>
				<ds:X509Certificate>... X509 certificate ... </ds:X509Certificate>
			</ds:X509Data>
		</ds:KeyInfo>
	</ds:Signature>
</CPIX>

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.

1 participant