From b44691e6315e7df7c4425bbd5560bd47fc867934 Mon Sep 17 00:00:00 2001 From: Maksym Pavlenko Date: Thu, 5 Jan 2017 12:22:50 -0800 Subject: [PATCH] Fix problem with quality while resolving videos, do not allow direct links to video --- src/Podsync/Controllers/FeedController.cs | 5 +++++ src/Podsync/Services/Resolver/YtdlWrapper.cs | 4 ++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/src/Podsync/Controllers/FeedController.cs b/src/Podsync/Controllers/FeedController.cs index e00e6ac..87a03e6 100644 --- a/src/Podsync/Controllers/FeedController.cs +++ b/src/Podsync/Controllers/FeedController.cs @@ -46,6 +46,11 @@ namespace Podsync.Controllers { var linkInfo = _linkService.Parse(new Uri(request.Url)); + if (linkInfo.LinkType == LinkType.Video) + { + throw new ArgumentException("Direct links are not supported, you should provide group, channel or user link"); + } + var feed = new FeedMetadata { Provider = linkInfo.Provider, diff --git a/src/Podsync/Services/Resolver/YtdlWrapper.cs b/src/Podsync/Services/Resolver/YtdlWrapper.cs index b7ad564..d250353 100644 --- a/src/Podsync/Services/Resolver/YtdlWrapper.cs +++ b/src/Podsync/Services/Resolver/YtdlWrapper.cs @@ -50,9 +50,9 @@ namespace Podsync.Services.Resolver switch (resolveType) { case ResolveType.VideoHigh: - return "best[ext=mp4]/low[ext=mp4]"; + return "bestvideo"; case ResolveType.VideoLow: - return "low[ext=mp4]/best[ext=mp4]"; + return "worstvideo"; case ResolveType.AudioHigh: return "bestaudio[ext=m4a]/worstaudio[ext=m4a]"; case ResolveType.AudioLow: