-
Notifications
You must be signed in to change notification settings - Fork 13
Destinations with verification and round trips #67
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
Conversation
Signed-off-by: Gabriel Indik <gabriel.indik@kaleido.io>
Signed-off-by: Gabriel Indik <gabriel.indik@kaleido.io>
Signed-off-by: Gabriel Indik <gabriel.indik@kaleido.io>
Signed-off-by: Gabriel Indik <gabriel.indik@kaleido.io>
Signed-off-by: Gabriel Indik <gabriel.indik@kaleido.io>
|
Changes included in this PR: Note that Data Exchange automatically associates destinations with peer information. This is for optimization purposes. More specifically, Data Exchange will not create multiple copies of the same certificate file. Instead each certificate will continue to be associated with a peer, and each peer can have 0-to-many destinations. Similarly to how peers work, one can optionally pre-populate destinations in the Data Exchange configuration file for both sender and recipients (see changes in the configuration schema and sample config files). When invoking the With these changes any call with an unknown destination on either the sender or the recipient will raise an error. The error messages are very detailed, showing what the valid destinations are (for sender and/or recipient) and which one was provided. Finally, backward compatibility is preserved, making it possible to use Data Exchange seamlessly with and without destinations. |
awrichar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and working with hyperledger/firefly#942
peterbroadhurst
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
No description provided.