WWBN / AVideo-Encoder

Encoder Server for AVideo Platform Open-Source
https://platform.avideo.com
GNU Affero General Public License v3.0
152 stars 189 forks source link

Consider alternate methods to push encoded videos to streamer #174

Open cooperthompson opened 5 years ago

cooperthompson commented 5 years ago

Currently, the encoder requires the use of the user's streamer credentials in order to push encoded videos to the streamer. This is problematic for a few reasons:

There are a few options that would address these concerns:

  1. OAuth2 between the encoder and streamer. This has been discussed in other security related issues (https://github.com/YouPHPTube/YouPHPTube/issues/400)
  2. Instead of http, use a filesystem volume shared between the encoder and streamer to transfer both video metadata and the encoded file itself.
  3. Use browser-side JS to send the metadata to the streamer at the time of upload, and a filesystem volume to transfer the encoded file. The browser-side JS upload lets you use the login cookie of the streamer instead of needing the password on the encoder server.

Option 1 is probably the ideal/fancy option, but I feel like option 2 is an reasonable interim option. My proposal is this:

From a security perspective, this is just establishing system-level trust between the encoder and streamer via the volume.

DanielnetoDotCom commented 5 years ago

Hi, thanks for sharing your ideas,

the Encoder will be improved in the future, I agree the current method of communication is not good.