mirrors/jellyfin-web
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin-web synced 2025-03-30 19:56:21 +00:00
Code Activity

Backport pull request #4182 from jellyfin/release-10.8.z

Browse source 
Fix the progressive mp4 transcoding profile

Original-merge: 5bfffd6209

Merged-by: Bill Thornton <thornbill@users.noreply.github.com>

Backported-by: Joshua M. Boniface <joshua@boniface.me>
This commit is contained in:
nyanmisaka 2023-01-22 14:07:07 -05:00 • committed by Joshua M. Boniface
parent 5e7dca2c4e
commit 813845d54b
1 changed files with 5 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

13
src/scripts/browserDeviceProfile.js
View file

@ -534,7 +534,6 @@ import browser from './browser';
} }
if (canPlayVp8) { if (canPlayVp8) {
mp4VideoCodecs.push('vp8');
webmVideoCodecs.push('vp8'); webmVideoCodecs.push('vp8');
} }
@ -717,17 +716,15 @@ import browser from './browser';
} }
} }
if (webmAudioCodecs.length && webmVideoCodecs.length) { // Progressive mp4 transcoding
if (mp4VideoCodecs.length && videoAudioCodecs.length) {
profile.TranscodingProfiles.push({ profile.TranscodingProfiles.push({
Container: 'webm', Container: 'mp4',
Type: 'Video', Type: 'Video',
AudioCodec: webmAudioCodecs.join(','), AudioCodec: videoAudioCodecs.join(','),
// TODO: Remove workaround when servers migrate away from 'vpx' for transcoding profiles. VideoCodec: mp4VideoCodecs.join(','),
VideoCodec: (canPlayVp8 ? webmVideoCodecs.concat('vpx') : webmVideoCodecs).join(','),
Context: 'Streaming', Context: 'Streaming',
Protocol: 'http', Protocol: 'http',
// If audio transcoding is needed, limit channels to number of physical audio channels
// Trying to transcode to 5 channels when there are only 2 speakers generally does not sound good
MaxAudioChannels: physicalAudioChannels.toString() MaxAudioChannels: physicalAudioChannels.toString()
}); });
} }