Merge pull request #2985 from MediaBrowser/dev

revise unknown audio stream handling
This commit is contained in:
Luke 2017-10-28 23:53:00 -04:00 committed by GitHub
commit 1dcb3a1369
3 changed files with 4 additions and 16 deletions

View File

@ -106,6 +106,7 @@ namespace Emby.Server.Implementations.Networking
endpoint.StartsWith("127.", StringComparison.OrdinalIgnoreCase) || endpoint.StartsWith("127.", StringComparison.OrdinalIgnoreCase) ||
endpoint.StartsWith("192.168", StringComparison.OrdinalIgnoreCase) || endpoint.StartsWith("192.168", StringComparison.OrdinalIgnoreCase) ||
endpoint.StartsWith("169.", StringComparison.OrdinalIgnoreCase) || endpoint.StartsWith("169.", StringComparison.OrdinalIgnoreCase) ||
endpoint.StartsWith("10.", StringComparison.OrdinalIgnoreCase) ||
IsInPrivateAddressSpaceAndLocalSubnet(endpoint); IsInPrivateAddressSpaceAndLocalSubnet(endpoint);
} }
@ -113,12 +114,7 @@ namespace Emby.Server.Implementations.Networking
{ {
var endpointFirstPart = endpoint.Split('.')[0]; var endpointFirstPart = endpoint.Split('.')[0];
if ( if (endpoint.StartsWith("10.", StringComparison.OrdinalIgnoreCase))
endpoint.StartsWith("127.", StringComparison.OrdinalIgnoreCase) ||
endpoint.StartsWith("10.", StringComparison.OrdinalIgnoreCase) ||
endpoint.StartsWith("192.168", StringComparison.OrdinalIgnoreCase) ||
endpoint.StartsWith("169.", StringComparison.OrdinalIgnoreCase)
)
{ {
var subnets = GetSubnets(endpointFirstPart); var subnets = GetSubnets(endpointFirstPart);

View File

@ -1099,15 +1099,6 @@ namespace MediaBrowser.Model.Dlna
{ {
string audioCodec = audioStream.Codec; string audioCodec = audioStream.Codec;
if (string.IsNullOrEmpty(audioCodec))
{
_logger.Info("Profile: {0}, DirectPlay=false. Reason=Unknown audio codec. Path: {1}",
profile.Name ?? "Unknown Profile",
mediaSource.Path ?? "Unknown path");
return new Tuple<PlayMethod?, List<TranscodeReason>>(null, new List<TranscodeReason> { TranscodeReason.UnknownAudioStreamInfo });
}
conditions = new List<ProfileCondition>(); conditions = new List<ProfileCondition>();
bool? isSecondaryAudio = audioStream == null ? null : mediaSource.IsSecondaryAudio(audioStream); bool? isSecondaryAudio = audioStream == null ? null : mediaSource.IsSecondaryAudio(audioStream);

View File

@ -948,7 +948,8 @@ namespace MediaBrowser.Providers.Manager
} }
else else
{ {
throw new Exception(string.Format("Refresh for item {0} {1} is not in progress", item.GetType().Name, item.Id.ToString("N"))); // TODO: Need to hunt down the conditions for this happening
//throw new Exception(string.Format("Refresh for item {0} {1} is not in progress", item.GetType().Name, item.Id.ToString("N")));
} }
} }
} }