slovensko-digital / autogram-extension

European Union Public License 1.2
10 stars 4 forks source link

Nesprávne nastavený XDC pri XadesBp2Xml #57

Closed celuchmarek closed 1 year ago

celuchmarek commented 1 year ago

Môže vôobec nastať situácia, že payload pri XadesBp2Xml (viacnásobný podpis XML) nie je XDC? Podľa mňa skôr nie, keďže pri pridávaní ďalšieho podpisu sa už podpisovaný content nemôže meniť a UPVS ponúka podpísnie XDC iba ako XDC. Ibažeby nejaký iný portál robil aj podpis XML bez XDC.

Tuto som našiel kód, ktorý ale nefunguje. Totiž, Ditec a aj nový Autogram tam dávajú <xdc:XMLDataContainer namiesto <XMLDataContainer. Ale ak by sme prehlásili, že to vždy bude XDC, tak by sme ani nepotrebovali tento check vôbec.

https://github.com/slovensko-digital/signer-switcher-extension/blob/a3fd7d7c53cdf3ee94bcaf8fd93d0f99f490978a/src/dbridge_js/ditecx/filetype-strategy/xml-bp-strategy.ts#L60-L70

cc @jsuchal @pomali

pomali commented 1 year ago

fixed