search

italia / eudi-wallet-it-python

Python toolchain for building an OpenID4VP RP with a SATOSA backend compliant with the Italian Wallet implementation profile
Apache License 2.0
16 stars 8 forks source link

`./satosa/backend.py:659`: # TODO: validate wia scheme using pydantic #141

Closed salvatorelaiso closed 8 months ago

salvatorelaiso commented 8 months ago

Nella WIA, quale dei due identificatori è corretto per il campo attested_security_context?

In questi due test, vengono utilizzate due chiavi diverse:

https://github.com/italia/eudi-wallet-it-python/blob/9a32ad139ea1cd05aac2d9aee737896b8a4a04a1/pyeudiw/tests/oauth2/test_dpop.py#L23

https://github.com/italia/eudi-wallet-it-python/blob/9a32ad139ea1cd05aac2d9aee737896b8a4a04a1/pyeudiw/tests/openid4vp/schemas/test_wallet_instance_attestation.py#L26

La documentazione riporta la dicitura estesa, ci atteniamo a questa, @peppelinux?

salvatorelaiso commented 8 months ago

Preciso che lo schema Pydantic attuale prevede la dicitura estesa.

https://github.com/italia/eudi-wallet-it-python/blob/9a32ad139ea1cd05aac2d9aee737896b8a4a04a1/pyeudiw/openid4vp/schemas/wallet_instance_attestation.py#L49

peppelinux commented 8 months ago

attualmente è attested_security_context ma stiamo migrando a aal qui https://github.com/italia/eudi-wallet-it-docs/pull/155

dato che con lo sviluppo siamo in una milestone avanzata che guarda alla prossima milestone della documentazione, ti chiedo di allineare aal