-
Notifications
You must be signed in to change notification settings - Fork 66
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
Include Timestamp certificates and crls in TimeStampValidationData when extending to XAdES X-L #115
Comments
I think this feature falls into a broader set of features/refactorings needed to better support extended forms. The library has some well-known design limitations (see older issues) that probably make those refactorings almost a rewrite of many parts. This is not something I'll tackle right away. |
Yes I agree the modification isn't easy, unfortunately I inherited some code that use your library extensively and i was asked to upgrade it to support XAdES X-L... At this stage adding the support will be less expensive than replacing it. We're keeping our modifications open source, as per LGPL requirement. Maybe they won't be beautiful but we must complete the feature somehow. |
I understand. It will be nice to see your mods anyway and we may discuss some aspects here if you want/need. Development has been kind of stalled, but I hope to bring this back on track (maybe with some help?) soon enough. I'm not sure if you'd also need to support X-L on verification. Anyway, here are some things I remember from previous discussions with other folks:
|
How can I sign with XAdES X-L format? Is there any approach? I tried to patch libraries with patches found in the web but many errors occured in my project. |
@sayat2002 there's no supported way to directly sign to X-L. However, it is possible to extend a XAdES-C signature to XAdES -X-L upon verification, given the limitations described in this issue. https://github.com/luisgoncalves/xades4j/wiki/SignatureEnrichment Regarding the existing patches, they are scattered among multiple issues and some override others... It would require some work to find yout which ones should be applied and on which order. |
ETSI TS 101 903 V1.4.2 section 8.1 :
Xades4j doesn't include TimeStampValidationData when extending a XAdES document to XAdES X-L
The text was updated successfully, but these errors were encountered: