-
Notifications
You must be signed in to change notification settings - Fork 40
Fix message delivery to open session #92
Fix message delivery to open session #92
Conversation
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
This short circuits session return Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
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.
I believe the changes to Alice and Bob's config files were not intended to be changed. In Alice's config, all auto responses were switched to manual and the tails server was removed. In both, the genesis-url was switched from the Sovrin network to the Indicio network. Aside from that, I think the changes look good to me.
Yes, I think we want to drop the last commit. These config changes should be captured elsewhere for David's use case. |
ef7895c
to
ec6e0b3
Compare
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
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.
After the submitted comebacks, the passing unit tests, and the initial review, I believe this looks good to merge. Great job!
While debugging usage with mobile agents, it was discovered that there is some quirky behavior in message delivery to open sockets from the toolbox plugin. This PR fixes these quirks.