Transforms element is optionnal
Opened this issue · 2 comments
Hello,
It's my first use of Xml signature , giving the good implementation of your library i used it and i think you for your work.
Actually, the API that i consume didn't accept the transforms element and breaks if it's present. I read the official documentation of xmlDSig and i found that the transforms element is optionnal. The implemenation here does not permit it ufortunately.
I suggest to surround the transforms node creation and it's traitement after in if condition.
https://github.com/robrichards/xmlseclibs/blob/master/src/XMLSecurityDSig.php#L665-L666
exp :
if (!empty($options['transforms'])) {
$transNodes = $this->createNewSignNode('Transforms');
$refNode->appendChild($transNodes);
if (is_array($arTransforms)) {
..........................
} elseif (! empty($this->canonicalMethod)) {
................................
}
}
Regards.
Raouf.
Hello,
Please can you consider/review my request.
Thanks.
Raouf.
Hello @robrichards can you please make a review for my issue/PR ?
Thanks,
Raouf.