v5 Request - Adopt Async/Task based requestsSee original GitHub issue
- Feature Request
Describe the issue
Currently all calls to do requests with OBS are synchronous not making use of
Task (and potentially
async), given there has been a large push to move towards this paradigm by a lot of vendors who can do potentially long running tasks, i.e IO, Network, API, DB calls etc it would be good if we could provide
async compatible requests.
Versions OBS Version: N/A OBS WebSocket Version: 5.x OBS WebSocket Dotnet (this library) Version: 5.x OS: N/A
Additional context There was a PR offered for the v4 branch which included these changes which allowed for mainly non blocking requests to OBS which helps greatly with UI facing applications, unfortunately that PR never made it through, but it would be great if the same approaches could be rolled into the v5 branch.
- Created a year ago
- Comments:6 (5 by maintainers)
Top GitHub Comments
However you spin it, bare minimum its going to need to change the signatures of almost all request methods from
T Whatever() to
Task<T> Whatever() and all the corresponding methods related to it as well as cancellation token parts as well.
So just to make sure we are on the same page this will ultimately have cascading changes everywhere, but the PR (assuming im the one who does it) will JUST be related to this issue.