Fixed: Assume category path from qBittorent starting with '//' is a Windows UNC path
Radarr/Radarr#10162
This commit is contained in:
parent
4b5ef4907b
commit
19466aa290
|
@ -560,6 +560,34 @@ namespace NzbDrone.Core.Test.Download.DownloadClientTests.QBittorrentTests
|
||||||
result.OutputRootFolders.First().Should().Be(@"C:\Downloads\Finished\QBittorrent".AsOsAgnostic());
|
result.OutputRootFolders.First().Should().Be(@"C:\Downloads\Finished\QBittorrent".AsOsAgnostic());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Test]
|
||||||
|
public void should_correct_category_output_path()
|
||||||
|
{
|
||||||
|
var config = new QBittorrentPreferences
|
||||||
|
{
|
||||||
|
SavePath = @"C:\Downloads\Finished\QBittorrent".AsOsAgnostic()
|
||||||
|
};
|
||||||
|
|
||||||
|
Mocker.GetMock<IQBittorrentProxy>()
|
||||||
|
.Setup(v => v.GetConfig(It.IsAny<QBittorrentSettings>()))
|
||||||
|
.Returns(config);
|
||||||
|
|
||||||
|
Mocker.GetMock<IQBittorrentProxy>()
|
||||||
|
.Setup(v => v.GetApiVersion(It.IsAny<QBittorrentSettings>()))
|
||||||
|
.Returns(new Version(2, 0));
|
||||||
|
|
||||||
|
Mocker.GetMock<IQBittorrentProxy>()
|
||||||
|
.Setup(s => s.GetLabels(It.IsAny<QBittorrentSettings>()))
|
||||||
|
.Returns(new Dictionary<string, QBittorrentLabel>
|
||||||
|
{ { "tv", new QBittorrentLabel { Name = "tv", SavePath = "//server/store/downloads" } } });
|
||||||
|
|
||||||
|
var result = Subject.GetStatus();
|
||||||
|
|
||||||
|
result.IsLocalhost.Should().BeTrue();
|
||||||
|
result.OutputRootFolders.Should().NotBeNull();
|
||||||
|
result.OutputRootFolders.First().Should().Be(@"\\server\store\downloads");
|
||||||
|
}
|
||||||
|
|
||||||
[Test]
|
[Test]
|
||||||
public async Task Download_should_handle_http_redirect_to_magnet()
|
public async Task Download_should_handle_http_redirect_to_magnet()
|
||||||
{
|
{
|
||||||
|
|
|
@ -377,7 +377,15 @@ namespace NzbDrone.Core.Download.Clients.QBittorrent
|
||||||
{
|
{
|
||||||
if (Proxy.GetLabels(Settings).TryGetValue(Settings.TvCategory, out var label) && label.SavePath.IsNotNullOrWhiteSpace())
|
if (Proxy.GetLabels(Settings).TryGetValue(Settings.TvCategory, out var label) && label.SavePath.IsNotNullOrWhiteSpace())
|
||||||
{
|
{
|
||||||
var labelDir = new OsPath(label.SavePath);
|
var savePath = label.SavePath;
|
||||||
|
|
||||||
|
if (savePath.StartsWith("//"))
|
||||||
|
{
|
||||||
|
_logger.Trace("Replacing double forward slashes in path '{0}'. If this is not meant to be a Windows UNC path fix the 'Save Path' in qBittorrent's {1} category", savePath, Settings.TvCategory);
|
||||||
|
savePath = savePath.Replace('/', '\\');
|
||||||
|
}
|
||||||
|
|
||||||
|
var labelDir = new OsPath(savePath);
|
||||||
|
|
||||||
if (labelDir.IsRooted)
|
if (labelDir.IsRooted)
|
||||||
{
|
{
|
||||||
|
|
Loading…
Reference in New Issue