(feat) Implement AnoncredsLedgerRead for cheqd #1306

[gmulhearn]

Closes #1306

Additions

• Adds a Cheqd implementation of the AnoncredsLedgerRead trait to the aries_vcx_ledger crate (hidden behind a feature flag) CheqdAnoncredsLedgerRead. This implementation uses cheqd resource resolution, and is largely inspired by credo-ts https://github.com/openwallet-foundation/credo-ts/blob/main/packages/cheqd/src/anoncreds/services/CheqdAnonCredsRegistry.ts
• Adds a resolve_resource method to the cheqd DID resolver, allowing DLRs to be resolved directly or via a name+type+time? DLR query. The implementation is largely inspired by credo-ts https://github.com/openwallet-foundation/credo-ts/blob/main/packages/cheqd/src/dids/CheqdDidResolver.ts#L69 . Note that the implementation for resolving DLR by name+type+time? query is not very efficient - it involves resolves the full set of DLR metadatas for a DID (collection) and then filtering in memory. This is a current limitation of the cheqd gRPC as pagination is not functioning as expected - this will need to be fixed once the gRPC is updated, however for now we ignore pagination (like credo)
• Adds a MultiLedgerAnoncredsRead implementation of the AnoncredsLedgerRead trait. This is similar to the DID resolver registry, it allows you to add multiple AnoncredsLedgerRead implementations and automatically switch between them depending on the requested schema/cred-def/rev-reg ID.
• add new attachment types for anoncreds attachments in issue-cred-v2 & present-proof-v2

Refactors

• add a method to the anoncredsledgerread trait, get_rev_status_list, which directly queries for the revocation status list - this is now the preferred method, instead of get_rev_reg_delta_json and then leaving the delta -> status list conversion elsewhere.
• All holder logic has been refactored to use get_rev_status_list instead of get_rev_reg_delta_json, however issuer & verifier is still a bit tied to it (issue moved to (refactor) Transition from revocation registry deltas to revocation status lists #1309 )
• (holder) parts of the Anoncreds trait refactored to take in a statuslist instead of delta, this allows some junk & conversions to be cut from the implementation: https://github.com/hyperledger/aries-vcx/pull/1308/files#diff-0c817a2ecd6c1ff01c6fc49b9c82add42df79bdbebc03a864fdf6d2566fc5aa1R936
• Adjusted DidUrl parsing to support/understand percent encoded characters, and to return the queries in their decoded form (matches behaviour of crates like url).

Testing

• manual testing against credo-ts is successful (including revocation) (after this PR fix: Revocation Index 0 anoncreds-rs support openwallet-foundation/credo-ts#2126)
• manual testing against acapy is successful (including revocation) ( on this branch https://github.com/cheqd/aries-cloudagent-python/tree/acapy-cheqd )
• Integration tests for cheqd resources & anoncreds which query known resources/vectors
• unit testing for edge cases & multiledgeranoncreds

[gmulhearn]

@JamesKEbert pls forgive me for some rust sins i committed in latest commit: 06f9afd

particularly in aries/aries_vcx/src/protocols/issuance/holder/state_machine.rs, there is a several lack of typing, so everything is JSON strings and passed around in tuples. This needs a good re-evaluation, as it's way behind the rest of the VCX codebase in terms of type safety

[JamesKEbert]

@JamesKEbert pls forgive me for some rust sins i committed in latest commit: 06f9afd

particularly in aries/aries_vcx/src/protocols/issuance/holder/state_machine.rs, there is a several lack of typing, so everything is JSON strings and passed around in tuples. This needs a good re-evaluation, as it's way behind the rest of the VCX codebase in terms of type safety

Oh yeah that is gross, but I approve of your approach in this commit given it's already not good and IMO should be addressed in a separate PR entirely anyways.

[JamesKEbert]

Really great stuff--LOTS in here! Great to see further implementation on the Cheqd integration. Love the addition of the multi ledger read and the cleanup to use status lists.

Thanks for your hard work here @gmulhearn!

[JamesKEbert]

LGTM