@mux/mux-node update

[dylanjha]

npx @getgrit/launcher apply mux_v8

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
stream-new	❌ Failed (Inspect)			Jan 3, 2024 9:42pm

[dylanjha]

Looks like it didn't add

const mux = new Mux()

and it went with Mux.Webhooks.prototype.verifySignature, which I don't think is right

[dylanjha]

Also interesting that it added this ternary, which wasn't there before:

Buffer.isBuffer(rawBody) ? rawBody.toString('utf8') : rawBody, req.headers

[rattrayalex]

cc @morgante can you take a look at this?

[morgante]

What's the expected output here?

[rattrayalex]

const mux = Mux() // maybe earlier in the file
mux.webhooks.verifyHeader(Buffer.isBuffer(rawBody) ? rawBody.toString('utf8') : rawBody, req.headers, webhookSignatureSecret)

[dylanjha]

Build error

./pages/api/uploads/index.ts:13:11
Type error: Type '{ playback_policy: "public"[]; }' is missing the following properties from type 'NewAssetSettings': input, master_access, mp4_support, normalize_audio, and 3 more.

  11 |       try {
  12 |         const upload = await mux.video.uploads.create({
> 13 |           new_asset_settings: { playback_policy: ['public'] },
     |           ^
  14 |           cors_origin: '*'
  15 |         });
  16 |         res.json({
error Command failed with exit code 1.

[rattrayalex]

Ah, that build error is likely because we marked properties as required in NewAssetSettings despite them being request params – @jsanford8 , does that sound right? Could you get that fixed in the OpenAPI Spec?