Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Separate core into a standalone crate #798

Merged
merged 38 commits into from
Apr 13, 2023
Merged

Conversation

mirgee
Copy link
Contributor

@mirgee mirgee commented Apr 11, 2023

Relocates BaseLedger, BaseAnonCreds, BaseWallet trait definitions and their corresponding implementations (including the indy module and related utils) into a distinct crate called aries_vcx_core. As a result, aries-vcx now depends on indy-vdr, indy-credx, and libvdrtools indirectly through this new crate, which doesn't reexport any of them.

The primary objective of this pull request is the separation of these components, and no additional efforts were made to enhance the codebase beyond this specific aim.

mirgee added 22 commits April 8, 2023 20:23
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
@mirgee mirgee added the skip-ci label Apr 11, 2023
mirgee added 2 commits April 12, 2023 08:49
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

# Conflicts:
#	aries_vcx/Cargo.toml
#	aries_vcx/src/common/primitives/credential_definition.rs
#	aries_vcx/src/errors/mod.rs
#	aries_vcx/src/lib.rs
#	aries_vcx/src/utils/mockdata/profile/mock_anoncreds.rs
#	aries_vcx/src/utils/mockdata/profile/mock_ledger.rs
#	aries_vcx/src/utils/mockdata/profile/mock_wallet.rs
#	aries_vcx/src/utils/mod.rs
#	aries_vcx_core/src/indy/credentials/issuer/mod.rs
#	aries_vcx_core/src/ledger/indy_ledger.rs
#	libvcx_core/src/api_vcx/api_handle/issuer_credential.rs
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
@mirgee mirgee force-pushed the refactor/seperate-plugins branch from bed3d69 to e0b9e3a Compare April 12, 2023 11:08
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

# Conflicts:
#	Cargo.lock
#	aries_vcx/Cargo.toml
#	libvdrtools/Cargo.toml
@mirgee mirgee marked this pull request as ready for review April 12, 2023 11:49
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
@mirgee mirgee force-pushed the refactor/seperate-plugins branch from 1364873 to 944236a Compare April 12, 2023 12:00
bobozaur added a commit that referenced this pull request Apr 13, 2023
Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>
gmulhearn
gmulhearn previously approved these changes Apr 13, 2023
Copy link
Contributor

@gmulhearn gmulhearn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I probably won't have time to give this an in-depth, but I definitely approve of the idea. Good improvement for vcx. Will put my review as approval assuming others have had a deeper look at the implementation

@gmulhearn gmulhearn self-requested a review April 13, 2023 08:19
Copy link
Contributor

@gmulhearn gmulhearn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see my review above ^. But i would like to see skip-ci removed before any merging. just as a confirmation

@Patrik-Stas
Copy link
Contributor

@gmulhearn good catch! Rerunning

Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
@Patrik-Stas Patrik-Stas dismissed stale reviews from gmulhearn, bobozaur, and themself via a699292 April 13, 2023 10:42
@codecov-commenter
Copy link

codecov-commenter commented Apr 13, 2023

Codecov Report

Merging #798 (b99839d) into main (f477a95) will decrease coverage by 25.05%.
The diff coverage is 12.36%.

@@             Coverage Diff             @@
##             main     #798       +/-   ##
===========================================
- Coverage   54.18%   29.13%   -25.05%     
===========================================
  Files         381      390        +9     
  Lines       36787    30694     -6093     
  Branches     8300     6380     -1920     
===========================================
- Hits        19932     8944    -10988     
- Misses      10702    18993     +8291     
+ Partials     6153     2757     -3396     
Flag Coverage Δ
unittests-aries-vcx 29.13% <12.36%> (-24.94%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
aries_vcx/src/common/keys.rs 0.00% <ø> (-55.72%) ⬇️
aries_vcx/src/common/ledger/transactions.rs 39.39% <ø> (-17.51%) ⬇️
...vcx/src/common/primitives/credential_definition.rs 0.00% <0.00%> (-49.75%) ⬇️
...s_vcx/src/common/primitives/revocation_registry.rs 0.00% <ø> (-47.91%) ⬇️
...cx/src/common/proofs/verifier/verifier_internal.rs 57.58% <0.00%> (-12.06%) ⬇️
aries_vcx/src/common/test_utils.rs 34.88% <ø> (-58.99%) ⬇️
aries_vcx/src/core/profile/modular_libs_profile.rs 0.00% <ø> (-78.95%) ⬇️
aries_vcx/src/core/profile/vdrtools_profile.rs 83.33% <ø> (ø)
aries_vcx/src/handlers/connection/cloud_agent.rs 30.50% <ø> (-40.41%) ⬇️
aries_vcx/src/handlers/discovery/mod.rs 0.00% <ø> (-93.75%) ⬇️
... and 72 more

... and 163 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
@Patrik-Stas Patrik-Stas force-pushed the refactor/seperate-plugins branch from bc966c6 to b99839d Compare April 13, 2023 16:08
@Patrik-Stas
Copy link
Contributor

Running full CI revealed few issues (compile and CI issues), go it fixed, not a big changes so I am merging per previous approvals.

@Patrik-Stas Patrik-Stas merged commit e7375a1 into main Apr 13, 2023
@Patrik-Stas Patrik-Stas deleted the refactor/seperate-plugins branch April 13, 2023 17:40
bobozaur added a commit that referenced this pull request Apr 19, 2023
Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>
bobozaur added a commit that referenced this pull request Apr 26, 2023
* Separate plugins into a crate

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use BaseAnoncreds from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use BaseWallet from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use IndyWallet from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use IndyAnoncreds, CredxAnoncreds from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use BaseLedger from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use IndyLedger, IndyVdrLedger from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Prune indy aries_vcx module

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Remove holder and wallet from indy module

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Remove remainder of indy dir from aries_vcx

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* aries_vcx, aries_vcx core builds with all features

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* libvcx_core + tests compile

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Export newtype wallet handle from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* libvcx_core + tests compile

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Decide to reexport aries_vcx_core from aries_vcx

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Workspace compiles

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* workspace compiles, no reexports

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Replaced messages in mediated connection handler

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* All handlers use new messages

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced old messages with new ones in Connection invitee state machine

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced old messages with new ones in Connection state machine

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced old messages with new ones in Holder state machine

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced more of the old messages

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced old messages to new ones in MediatedConnection

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced messages in Prover

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced messages in Verifier

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced old messages with new ones in aries_vcx

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Finished messages replacement

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Added more usage for AttachmentId and retrofitted deserialization of mediated_connection states

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Integrated new messages into aries-vcx-agent

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Workspace compiling (except for uniffi wrapper due to lack of Url type definition in the UDL file)

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed https links

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed dummy invitation

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Updated request building

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Restored index.js and index.d.ts from main

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed oob to_string() impls

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed mediated connection thread ID

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed forward message wrapping

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced Vec<u8> with Value in Forward msg for backwards compatibility

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed deserialization errors when the deserializer owned the data

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed response building in MediatedConnection

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* More fixes to Response building

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* DID doc state fixes

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Finally fixed Response building

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Added early return and removed ? in Protocol parsing

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Extended TS HandshakeProtocol enum for backwards compatibility

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed MimeType and renamed Attribute to PresentationAttr

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Corrected AttachmentId handling

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Corrected impl of connection signature to be its own message type

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Added SignatureTypeV1 to ProtocolRegistry

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* With teary eyes added #[serde(default)] to PleaseAck decorator for backwards compatibility

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Corrected OOB invitation impl

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced Vec<u8> with String in Base64 attachment

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed attachment handling

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Further fixed on attachment handling

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed OOB invitation attachment serialization

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed con unit tests

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed some thread_id issues

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Rebased from main

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed uniffi by accepting AriesDidDoc as String

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Restored messages/test_utils in aries_vcx Cargo.toml

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced more Strings with Urls

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Agency_client tests working

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* DidDoc crate tests fixed

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Libvcx_core tests compile

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Rebased from #798

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* aries_vcx integration tests compile

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Removed feature flags from shared_vcx

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Removed feature flags from aries_vcx_core

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Removed feature flags from diddoc

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Post feature flag removal in aries-vcx fix

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* More fixes on aries_vcx tests

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Aries-vcx tests work (sometimes)

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Adapted CI to removed feature flags from aries_vcx

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Rebased from main

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* libvcx_core tests all run

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Removed feature flag traces from aries_vcx

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Parsing URLs in node wrappers as URLs and not strings

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed CI workflow

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* CI fixes and ignoring tails in gitignore

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed feature flags mismatch for CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* More CI fixes

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fix for wrapper tests

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Updated rust toolchain version in all places

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Bumped vcxagency docker container version in CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed connection_send_generic_message in napi wrapper

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fix issue with *_proposal_* tests

Signed-off-by: Patrik Stas <patrik.stas@absa.africa>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Tentative fix for the iOS CI builds

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Adding package-lock.json back

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Removed unnecessary vdrtools handle wrappers

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Test fixes after handle wrapper removal

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Attempt to fix linking issues in iOS CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Trying to trigger CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Dummy commit to make CI publish a different napi package version

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Dummy commit to trigger CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Another dummy commit to trigger CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Uncommented test

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Adjusted tests as per PR comments

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed obsolete dependency_flag check and default_features issue in aries_vcx

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Hopefully fixing the mess we call tests

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

---------

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Co-authored-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Co-authored-by: Patrik Stas <patrik.stas@absa.africa>
tech-bash pushed a commit to tech-bash/aries-vcx that referenced this pull request Apr 30, 2023
* Separate plugins into a crate

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use BaseAnoncreds from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use BaseWallet from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use IndyWallet from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use IndyAnoncreds, CredxAnoncreds from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use BaseLedger from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Use IndyLedger, IndyVdrLedger from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Prune indy aries_vcx module

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Remove holder and wallet from indy module

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Remove remainder of indy dir from aries_vcx

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* aries_vcx, aries_vcx core builds with all features

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* libvcx_core + tests compile

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Export newtype wallet handle from aries_vcx_core

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* libvcx_core + tests compile

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Decide to reexport aries_vcx_core from aries_vcx

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Workspace compiles

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* workspace compiles, no reexports

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>

* Replaced messages in mediated connection handler

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* All handlers use new messages

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced old messages with new ones in Connection invitee state machine

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced old messages with new ones in Connection state machine

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced old messages with new ones in Holder state machine

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced more of the old messages

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced old messages to new ones in MediatedConnection

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced messages in Prover

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced messages in Verifier

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced old messages with new ones in aries_vcx

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Finished messages replacement

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Added more usage for AttachmentId and retrofitted deserialization of mediated_connection states

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Integrated new messages into aries-vcx-agent

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Workspace compiling (except for uniffi wrapper due to lack of Url type definition in the UDL file)

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed https links

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed dummy invitation

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Updated request building

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Restored index.js and index.d.ts from main

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed oob to_string() impls

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed mediated connection thread ID

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed forward message wrapping

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced Vec<u8> with Value in Forward msg for backwards compatibility

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed deserialization errors when the deserializer owned the data

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed response building in MediatedConnection

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* More fixes to Response building

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* DID doc state fixes

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Finally fixed Response building

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Added early return and removed ? in Protocol parsing

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Extended TS HandshakeProtocol enum for backwards compatibility

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed MimeType and renamed Attribute to PresentationAttr

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Corrected AttachmentId handling

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Corrected impl of connection signature to be its own message type

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Added SignatureTypeV1 to ProtocolRegistry

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* With teary eyes added #[serde(default)] to PleaseAck decorator for backwards compatibility

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Corrected OOB invitation impl

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced Vec<u8> with String in Base64 attachment

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed attachment handling

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Further fixed on attachment handling

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed OOB invitation attachment serialization

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed con unit tests

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed some thread_id issues

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Rebased from main

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed uniffi by accepting AriesDidDoc as String

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Restored messages/test_utils in aries_vcx Cargo.toml

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Replaced more Strings with Urls

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Agency_client tests working

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* DidDoc crate tests fixed

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Libvcx_core tests compile

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Rebased from hyperledger#798

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* aries_vcx integration tests compile

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Removed feature flags from shared_vcx

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Removed feature flags from aries_vcx_core

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Removed feature flags from diddoc

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Post feature flag removal in aries-vcx fix

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* More fixes on aries_vcx tests

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Aries-vcx tests work (sometimes)

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Adapted CI to removed feature flags from aries_vcx

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Rebased from main

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* libvcx_core tests all run

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Removed feature flag traces from aries_vcx

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Parsing URLs in node wrappers as URLs and not strings

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed CI workflow

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* CI fixes and ignoring tails in gitignore

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed feature flags mismatch for CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* More CI fixes

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fix for wrapper tests

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Updated rust toolchain version in all places

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Bumped vcxagency docker container version in CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed connection_send_generic_message in napi wrapper

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fix issue with *_proposal_* tests

Signed-off-by: Patrik Stas <patrik.stas@absa.africa>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Tentative fix for the iOS CI builds

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Adding package-lock.json back

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Removed unnecessary vdrtools handle wrappers

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Test fixes after handle wrapper removal

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Attempt to fix linking issues in iOS CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Trying to trigger CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Dummy commit to make CI publish a different napi package version

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Dummy commit to trigger CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Another dummy commit to trigger CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Uncommented test

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Adjusted tests as per PR comments

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed obsolete dependency_flag check and default_features issue in aries_vcx

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Hopefully fixing the mess we call tests

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

---------

Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Co-authored-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Co-authored-by: Patrik Stas <patrik.stas@absa.africa>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants