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

Allow DIDComm protocols to use DIDs to identify agents sending/receiving messages #725

Closed
Moopli opened this issue Nov 7, 2019 · 3 comments · Fixed by #938
Closed

Allow DIDComm protocols to use DIDs to identify agents sending/receiving messages #725

Moopli opened this issue Nov 7, 2019 · 3 comments · Fixed by #938
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@Moopli
Copy link
Contributor

Moopli commented Nov 7, 2019

For one-to-one agent communication, protocol code shouldn't need to include code for retrieving destination parameters when sending messages, and shouldn't need to decipher the identity of a sender when receiving messages. Instead, the outbound dispatcher or helper functions should retrieve destination parameters given a DID, and inbound transport should provide the sender's DID, so protocols can operate solely using DIDs if they don't need more customized functionality.

@fqutishat fqutishat added this to the Backlog milestone Nov 7, 2019
@fqutishat fqutishat added the enhancement New feature or request label Nov 7, 2019
@Moopli Moopli changed the title Select Packer format based on connection Use connection ID/record as parameter to Outbound/Packager Nov 14, 2019
@Moopli
Copy link
Contributor Author

Moopli commented Nov 14, 2019

We could also return the connection ID when we unpack - if we have a way to look up the connection record using the key from the other agent (who sent the message to you).

@fqutishat fqutishat modified the milestones: Backlog, 0.1.1 Nov 14, 2019
@Moopli Moopli changed the title Use connection ID/record as parameter to Outbound/Packager Allow DIDComm protocols to use DIDs to identify agents sending/receiving messages Dec 4, 2019
@Moopli
Copy link
Contributor Author

Moopli commented Dec 4, 2019

Rewrote the issue, since it was describing one potential solution rather than the problem to be solved.

@fqutishat
Copy link
Contributor

@troyronda @rolsonquadras @Moopli
#998 need this functionality

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Development

Successfully merging a pull request may close this issue.

2 participants