Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

refactor: set DIDcomm V1 VS V2 Endpoint support #3239

Merged
merged 1 commit into from
May 26, 2022

Conversation

baha-ai
Copy link
Contributor

@baha-ai baha-ai commented May 9, 2022

This change sets back the original DIDComm V1's Endpoint logic that was supperceded with the Endpoint struct update for DIDComm V2.
With this change, both ServiceEndpoint structures for V1 and V2 are not supported.

closes #3219

Signed-off-by: Baha Shaaban [email protected]

@baha-ai baha-ai force-pushed the endpoint_refactor branch 6 times, most recently from 7c61b6c to 94b339f Compare May 13, 2022 13:42
@baha-ai baha-ai changed the title [WIP- do not merge] refactor: set DIDcomm V1 VS V2 Endpoint support refactor: set DIDcomm V1 VS V2 Endpoint support May 13, 2022
@baha-ai baha-ai force-pushed the endpoint_refactor branch 2 times, most recently from 085da23 to 60322cb Compare May 13, 2022 14:03
@codecov
Copy link

codecov bot commented May 13, 2022

Codecov Report

Merging #3239 (967f08f) into main (2d8ae9d) will decrease coverage by 0.40%.
The diff coverage is 63.73%.

@@            Coverage Diff             @@
##             main    #3239      +/-   ##
==========================================
- Coverage   88.75%   88.34%   -0.41%     
==========================================
  Files         309      310       +1     
  Lines       41766    42257     +491     
==========================================
+ Hits        37068    37334     +266     
- Misses       3422     3615     +193     
- Partials     1276     1308      +32     
Impacted Files Coverage Δ
pkg/didcomm/common/service/destination.go 100.00% <ø> (ø)
pkg/store/connection/connection_lookup.go 94.25% <ø> (ø)
pkg/client/outofband/client.go 64.88% <9.67%> (-5.41%) ⬇️
pkg/vdr/peer/creator.go 69.94% <20.37%> (-19.02%) ⬇️
pkg/doc/did/doc.go 83.69% <34.42%> (-3.36%) ⬇️
pkg/didcomm/transport/http/outbound.go 81.94% <40.00%> (-3.35%) ⬇️
pkg/client/connection/client.go 81.90% <47.05%> (-7.35%) ⬇️
pkg/didcomm/transport/ws/outbound.go 81.92% <53.84%> (-5.92%) ⬇️
pkg/controller/command/messaging/command.go 96.37% <56.25%> (-3.63%) ⬇️
pkg/didcomm/protocol/didexchange/service.go 86.32% <57.14%> (-4.38%) ⬇️
... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2d8ae9d...967f08f. Read the comment docs.

@baha-ai baha-ai force-pushed the endpoint_refactor branch 4 times, most recently from c889052 to 2f3774c Compare May 20, 2022 18:02
@baha-ai baha-ai force-pushed the endpoint_refactor branch 4 times, most recently from 6c24a7a to 6c6ebaa Compare May 20, 2022 20:47
pkg/didcomm/dispatcher/outbound/outbound.go Outdated Show resolved Hide resolved
pkg/didcomm/protocol/didexchange/states.go Outdated Show resolved Hide resolved
@baha-ai baha-ai force-pushed the endpoint_refactor branch 4 times, most recently from 3ba93f7 to af4c507 Compare May 25, 2022 13:44
@fqutishat fqutishat self-requested a review May 25, 2022 13:46
@fqutishat fqutishat dismissed their stale review May 25, 2022 13:46

dismiss

@baha-ai baha-ai force-pushed the endpoint_refactor branch 3 times, most recently from 692c461 to 0239db4 Compare May 26, 2022 19:11
This change sets back the original DIDComm V1's Endpoint logic that was supperceded with the Endpoint struct update for DIDComm V2.
With this change, both ServiceEndpoint structures for V1 and V2 are not supported.

closes hyperledger-archives#3219

Signed-off-by: Baha Shaaban <[email protected]>
@baha-ai baha-ai force-pushed the endpoint_refactor branch from 0239db4 to 967f08f Compare May 26, 2022 20:13
@fqutishat fqutishat merged commit 18d510d into hyperledger-archives:main May 26, 2022
@baha-ai baha-ai deleted the endpoint_refactor branch September 16, 2022 21:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Update DIDComm V2 Endpoint to include a URI, routingKeys and Accept fields
2 participants