bseddon / xml-signer

Provides signing and verification of XML documents including support for XAdES
BSD 3-Clause "New" or "Revised" License
19 stars 8 forks source link

Class '.\Transform' not found #8

Closed sangar82 closed 3 years ago

sangar82 commented 3 years ago

When try to test the new methods to get LTA format #7 I get the following error:

Class '.\Transform' not found

[2021-10-19 12:12:16] develop.ERROR: Class '.\Transform' not found {"exception":"[object] (Error(code: 0): Class '.\\Transform' not found at /home/vagrant/laravel/vendor/lyquidity/xml-signer/src/xml/Transforms.php:131)
[stacktrace]
#0 /home/vagrant/laravel/vendor/lyquidity/xml-signer/src/xml/Reference.php(167): lyquidity\\xmldsig\\xml\\Transforms->loadInnerXml()
#1 /home/vagrant/laravel/vendor/lyquidity/xml-signer/src/xml/SignedInfo.php(138): lyquidity\\xmldsig\\xml\\Reference->loadInnerXml()
#2 /home/vagrant/laravel/vendor/lyquidity/xml-signer/src/xml/Signature.php(139): lyquidity\\xmldsig\\xml\\SignedInfo->loadInnerXml()
#3 /home/vagrant/laravel/vendor/lyquidity/xml-signer/src/xml/Generic.php(67): lyquidity\\xmldsig\\xml\\Signature->loadInnerXml()
#4 /home/vagrant/laravel/vendor/lyquidity/xml-signer/src/XAdES.php(1551): lyquidity\\xmldsig\\xml\\Generic::fromNode()
#5 /home/vagrant/laravel/vendor/lyquidity/xml-signer/src/XAdES.php(336): lyquidity\\xmldsig\\XAdES->addArchiveTimestamp()
#6 /home/vagrant/laravel/vendor/lyquidity/xml-signer/src/XAdES.php(252): lyquidity\\xmldsig\\XAdES::internalTimestamp()

I think is something #1 and #3

Thank you!

bseddon commented 3 years ago

Yes, it is very similar to the earlier ones. It's fixed and the update is available via composer. I was able to reproduce on Linux and show the issue seems to be resolved on Linux. There was another example of the same problem in KeyInfo.php. This has been fixed as well.

Thanks for letting me know.

sangar82 commented 3 years ago

Fixed! Thank you!

You've forgotten some echo's in the code, like this output: lyquidity\xmldsig\xml\SignatureTimeStamp

bseddon commented 3 years ago

Thanks. I've removed them and pushed the changes. Since its not a exception I'll update composer later.