api/stream: Add a couple more hacks for testing Trovo profiles

[victorges]

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

• When isMobileflag is set to true in the pull API endpoint, all transcode profiles set by the end user should be overwritten by fps=0. The rest of the parameters can remain as-is. So for example, if fps=30 is set by the user, we should replace that profile with fps=0 and keep remaining parameters the same. This is a temporary workaround required to resolve stuttering issues seen with Trovo's mobile app (PS-382)

For example, a /pull API request where the user set profiles like this:

"profiles": [ { "name": "360p0", "fps": 30, "bitrate": 800000, "width": 640, "height": 360, "profile": "H264ConstrainedHigh" }, { "name": "480p0", "fps": 30, "bitrate": 1600000, "width": 854, "height": 480, "profile": "H264ConstrainedHigh" }, ] }

will become the following (only fps field overwritten and set to 0):

"profiles": [ { "name": "360p0", "fps": 0, "bitrate": 800000, "width": 640, "height": 360, "profile": "H264ConstrainedHigh" }, { "name": "480p0", "fps": 0, "bitrate": 1600000, "width": 854, "height": 480, "profile": "H264ConstrainedHigh" }, ] }

• Any request that has a width=853 and height=480, should be updated so that width=854. This is because 853 is not a standard dimension and it results in a vertical green line on the right side of the frame (PS-381)

Specific updates (required)

The 2 things above.

How did you test each of these updates (required)

yarn test

Does this pull request close any open issues?

Implements PS-534

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	Comments	Updated (UTC)
livepeer-studio	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 30, 2024 0:31am

[gioelecerati]

Probably imported by mistake?

[gioelecerati]

Ah, I see it was elsewhere

[victorges]

No it's used in this file
https://github.com/livepeer/livepeer-com/blob/2a52015e9c4a0a41d94b7fe5ba5996af9447b097/packages/api/src/controllers/stream.ts

[pwilczynskiclearcode]

Deployed to prod a moment ago https://discord.com/channels/423160867534929930/1102968315137228900/1235138317050183721