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.

Sign up for GitHub

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

Jump to bottom

Add Message.Respond() #281

Closed
ColinSullivan1 opened this issue Aug 26, 2019 · 1 comment
Closed

Add Message.Respond() #281

ColinSullivan1 opened this issue Aug 26, 2019 · 1 comment

Comments

@ColinSullivan1
Copy link
Member

ColinSullivan1 commented Aug 26, 2019 •
edited
Loading

See:
watfordgnf added a commit to watfordgnf/csharp-nats that referenced this issue Aug 27, 2019
@watfordgnf
Add Msg.Respond nats-io#281
2eb7dda 
- Adds Respond to Msg to simplify responses to Request-Reply
- No longer clears Subscription.conn, until disposing
@watfordgnf watfordgnf mentioned this issue Aug 27, 2019
Merged
ColinSullivan1 pushed a commit that referenced this issue Aug 27, 2019
@watfordgnf @ColinSullivan1
Add Message.Respond implementing #281 (#283)
d2cfbe6 
* Add Msg.Respond

- Adds Respond to Msg to simplify responses to Request-Reply
- No longer clears Subscription.conn, until disposing
- Add tests for Msg.Respond with both closed connections and lost servers
- Throw NATSConnectionClosedException when appropriate
@ColinSullivan1
Copy link
Member Author

Resolved by #283

@bruth bruth removed the help wanted label Aug 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bruth @ColinSullivan1