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

[proposal] Add Trust Ping Protocol #1079

Closed
nickreynolds opened this issue Nov 29, 2022 · 0 comments · Fixed by #1080
Closed

[proposal] Add Trust Ping Protocol #1079

nickreynolds opened this issue Nov 29, 2022 · 0 comments · Fixed by #1080
Labels
enhancement New feature or request good first issue Good for newcomers incomplete Issue is not well defined. Please ask for clarification before starting to work on it!

Comments

@nickreynolds
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Implement the functionality of the Trust Ping Protocol, as defined here: https://identity.foundation/didcomm-messaging/spec/#trust-ping-protocol-10

Describe the solution you'd like
A helper function for sending Trust Pings (function should accept recipient DID, and handle packing and sending of correctly defined DIDCommMessage) as well as a message handler for processing Trust Pings (and sending Trust Ping Responses, when required, which is the standard flow)

Describe alternatives you've considered
Work in progress here: https://github.com/uport-project/veramo/tree/nickreynolds/trust-ping

Additional context
Implementation from Aviary Tech here: https://github.com/aviarytech/didcomm/blob/master/src/lib/protocols/trust-ping/2.0/ping.ts

@nickreynolds nickreynolds added enhancement New feature or request good first issue Good for newcomers incomplete Issue is not well defined. Please ask for clarification before starting to work on it! labels Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers incomplete Issue is not well defined. Please ask for clarification before starting to work on it!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant