Clarification regarding OP metadata: documents_methods_supported, documents_validation_methods_supported, documents_verification_methods_supported

[Vladimir Dzhuvinov]

A question came up when devising OP metadata constraints logic for these 3 fields - do they apply to evidences of type document (and the deprecated id_document), or to all types of evidence? The documents_ prefix seems to imply the former.

‌

https://openid.bitbucket.io/eKYC-IDA/openid-connect-4-identity-assurance-1_0-master.html#section-9

documents_methods_supported: OPTIONAL. JSON array containing the validation & verification process the OP supports (see @!predefined_values).

documents_validation_methods_supported: OPTIONAL. JSON array containing the document validation methods the OP supports (see @!predefined_values).

documents_verification_methods_supported: OPTIONAL. JSON array containing the verification methods the OP supports (see @!predefined_values).

‌

[Mark Haine]

I think that these OP Metadata entries apply to both document and id_document types

[Vladimir Dzhuvinov]

Thank you Mark, I wanted to confirm this so that the software logic works correctly.

[Julian White]

Mark is correct, it is supposed to apply to document and id_document types

[Mark Haine]

Some clarifying text may be appropriate given that the question was asked… perhaps something like:

“in all places that we refer to document it should include id_document if the deprecated version is being used”

[Vladimir Dzhuvinov]

That’s a good idea Mark, to put this id_document note in the section where document is defined:

https://openid.bitbucket.io/ekyc/openid-connect-4-identity-assurance.html#section-5.1.1.1

All references to id_document in section 9 describing the OP metadata can then be removed.

[Vladimir Dzhuvinov]

I noticed that in Julian’s PR the mention of id_document is moved up to the parent section where the evidence types are outlined:

https://bitbucket.org/openid/ekyc-ida/pull-requests/87#Lopenid-connect-4-identity-assurance.mdT256

[Mark Haine]

Julians PR being PR #87 - if that is merged are you happy that this issue is resolved @Vladimir Dzhuvinov

[Vladimir Dzhuvinov]

I just submitted a PR so we can close this:

https://bitbucket.org/openid/ekyc-ida/pull-requests/111/clarifies-that-documents_methods_supported

[Takahiko Kawasaki]

Resolved by PR#111