api: Add playbackPolicy to assets & streams

[gioelecerati]

What does this pull request do? Explain your changes. (required)

Add the ability to specify a playback policy to an Asset, making them public, signed or private
Comes with #1275

This is also applied to stream

Specific updates (required)

How did you test each of these updates (required)

Does this pull request close any open issues?

Fixes #1289

Screenshots (optional):

Checklist:

• I have read the CONTRIBUTING document.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
livepeer-studio	✅ Ready (Inspect)	Visit Preview	Sep 23, 2022 at 2:33PM (UTC)

[codecov]

Codecov Report

Merging #1286 (adfcec4) into master (d6044aa) will increase coverage by 0.09236%.
The diff coverage is 100.00000%.

@@                 Coverage Diff                 @@
##              master       #1286         +/-   ##
===================================================
+ Coverage   51.28146%   51.37382%   +0.09235%     
===================================================
  Files             66          66                 
  Lines           4331        4331                 
  Branches         802         802                 
===================================================
+ Hits            2221        2225          +4     
+ Misses          1829        1826          -3     
+ Partials         281         280          -1     

Impacted Files	Coverage Δ
packages/api/src/controllers/asset.ts	54.76190% <100.00000%> (ø)
packages/api/src/webhooks/cannon.ts	49.70760% <0.00000%> (+2.33917%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d6044aa...adfcec4. Read the comment docs.

Impacted Files	Coverage Δ
packages/api/src/controllers/asset.ts	54.76190% <100.00000%> (ø)
packages/api/src/webhooks/cannon.ts	49.70760% <0.00000%> (+2.33917%)	⬆️

[victorges]

LGTM!

[victorges]

Sounds great!

…

On Thu, 22 Sept 2022, 19:04 gioelecerati, ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In packages/api/src/schema/schema.yaml <#1286 (comment)>: > @@ -732,6 +738,15 @@ components: response: $ref: "#/components/schemas/webhook-response" + playback-policy: So, are you proposing to make something like "playbackPolicy":{ "type":"signed" } for now, where type is the enum and we can expand later? Sounds good to me, on first iteration when I was putting the signingKey ids inside the policy it was actually like that. wdyt of the object like this? — Reply to this email directly, view it on GitHub <#1286 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAMJ4R2LP7YK63YSVJIFPWLV7TJX3ANCNFSM6AAAAAAQOXCFGE> . You are receiving this because your review was requested.Message ID: ***@***.***>

[victorges]

Yeah! Twas just a historical joke hahaha

…

On Thu, 22 Sept 2022, 19:02 gioelecerati, ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In packages/api/src/schema/schema.yaml <#1286 (comment)>: > @@ -964,6 +979,8 @@ components: readOnly: true example: https://livepeercdn.com/asset/ea03f37e-f861-4cdd-b495-0e60b6d753ad/index.m3u8 description: URL for HLS playback + playbackPolicy: + $ref: "#/components/schemas/playback-policy" wait what am I missing 👀 isn't the $ right there? — Reply to this email directly, view it on GitHub <#1286 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAMJ4RYRNMF2OLLNDFASETLV7TJRFANCNFSM6AAAAAAQOXCFGE> . You are receiving this because your review was requested.Message ID: ***@***.***>

[victorges]

Turns out GitHub doesn't send the replies in the right conversation lol