lovele0107 / signatures-conformance-checker

7 stars 0 forks source link

CompleteCertificateRefsV2 is not correctly supported #6

Open bsanchezb opened 3 years ago

bsanchezb commented 3 years ago

Hello,

According to the EN 319 132-1 and the schema, the element CompleteCertificateRefsV2 shall contain the element CertRefs inside:

<xsd:element name="CompleteCertificateRefsV2" type="CompleteCertificateRefsTypeV2"/>
<xsd:complexType name="CompleteCertificateRefsTypeV2">
    <xsd:sequence>
        <xsd:element name="CertRefs" type="xades:CertIDListV2Type"/>
    </xsd:sequence>
    <xsd:attribute name="Id" type="xsd:ID" use="optional"/>
</xsd:complexType> 

However, when I validate a file with the elements, the XAdES Conformance Checked reports me the error:

Location-{CodeTest}:UnsignedSignatureProperties/CompleteCertificateRefsV2[2]-{CheckSchemaForChildren}
--
Children order and number DO NOT MATCH specification 
Specification: xadesv132:Cert+
Elements found: xadesv141:CertRefs
Error indication (^ appears at the end of the last correct child): ^xadesv141:CertRefs

It seems like the element 'xadesv141:CertRefs' is skipped in the checker. Could you check it, please?

The test file is in the attachment (archived, because GitHub does not support XML attachments).

Best regards, Aleksandr.


Edited: updated the attachment

sample-signed-xades-x.zip

jccruellas commented 3 years ago

Dear Alexander, Sorry for the delay in reacting. I have experienced problems during some time as the notifications did not arrive to may email and also heavy work in the university. Indeed the behaviour is not correct. I will check and modify XAdESCC accordingly

I will keep you posted Best regards Juan Carlos.

jccruellas commented 3 years ago

Hi again Alexander, I have fixed this issue in my local copy of XAdESCC. I attach a screenshot of my screen where you will be able to see that this version now works fine

20210414_XAdESCCScreenShot
jccruellas commented 3 years ago

I plan to update the deployed XAdESCC at the ETSI portal during this week Regards Juan Carlos.