search

livepeer / go-livepeer-basicnet

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

small bug fix #22

Closed ericxtang closed 6 years ago

ericxtang commented 6 years ago

Make sure the GetMasterPlaylist doesn't go out to the network when it's clearly asking for a local stream that doesn't exist.

Also, fixing bugs in the test.

ericxtang commented 6 years ago

Tried to upgrade libp2p in this PR - and it's not playing well in the linux environment. All tests pass fine in osx, but linux is giving all kinds of weird issues.

On linux boxes, tests pass if you run them individually. But if you run the entire test suite, a lot of them fail. Because of the error message indicates the node ID in the connection are different from the node ID in the requests, I suspect the errors come from improper teardown of the nodes. But I'm not sure how to confirm this.

Gonna close this PR now and try some other time. Archived this in et/libp2p-upgrade