- edited description
broadcast: parameterise broadcast service
As part of issue #257.
We currently have a direct dependency on our implementation of the youtube live API. We should instead write a BroadcastService interface for which the youtube API wrapper implements and then have functions parameterise the BroadcastService so that we can swap it out for dummy testing implementations. This might also help future proof the code if we consider broadcasting to other types of streaming services (which is not hard to imagine).
Comments (7)
-
reporter -
VidGrind’s limited support for forwarding over RTP might conceivably be generalized to use such a service.
-
reporter Yes, this a good idea.
-
reporter - changed status to resolved
broadcast: add BroadcastService interface
Resolves issue
#259This change adds the BroadcastService interface and a youtube based implementation in preparation to update the broadcast.go code to parameterise the broadcast service stuff.Approved-by: Trek Hopton
→ <<cset 1535f7350523>>
-
reporter broadcast: utilising BroadcastService interface
Resolves issue
#259This change updates the broadcast.go code to parameterise the broadcast service by using the BroadcastService interface. We're using the YouTubeBroadcastService implementation in most cases besides for testing where a dummyService implementation has been added.Approved-by: Trek Hopton
→ <<cset be8834122a8a>>
-
reporter broadcast: add BroadcastService interface
Resolves issue
#259This change adds the BroadcastService interface and a youtube based implementation in preparation to update the broadcast.go code to parameterise the broadcast service stuff.Approved-by: Trek Hopton
→ <<cset 8e0ad42aed9f>>
-
reporter broadcast: utilising BroadcastService interface
Resolves issue
#259This change updates the broadcast.go code to parameterise the broadcast service by using the BroadcastService interface. We're using the YouTubeBroadcastService implementation in most cases besides for testing where a dummyService implementation has been added.Approved-by: Trek Hopton
→ <<cset 417f8cbd9054>>
- Log in to comment