livepeer / go-livepeer-basicnet

Basic p2p video streaming for Livepeer
MIT License
18 stars 8 forks source link

Integration Test And Bug Fix #16

Closed ericxtang closed 6 years ago

ericxtang commented 6 years ago

Added some integration test for ABS scenarios, and found some bugs through that.

We have been handling the Cancel request incorrectly - shouldn't pass on the request if there is a subscriber that needs the stream.

Also did a small change in BasicSubscriber to directly send the Cancel request instead of relying on the cancel function to invoke ctxW.Done() in the worker.