danvics/rdt-client
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: default SanitizeFilenames to true only on Linux

Browse source 
The underlying bug (debrid-provider filenames with brackets causing
downloads to fail) only reproduces on Linux hosts. Windows, macOS,
and other platforms handle these characters without issue and have
no reason to have their filenames modified. Default the toggle to
OperatingSystem.IsLinux() so Windows users see no behaviour change
while Linux containers still get the fix out of the box. Users on
either platform can override via the setting.

Addresses review feedback on PR #962.
This commit is contained in:
Claude 2026-04-24 03:58:07 +00:00
parent 541489d0d3
commit 6f1ec6581a
No known key found for this signature in database
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
server/RdtClient.Data/Models/Internal/DbSettings.cs
View file

@ -178,8 +178,8 @@ http://127.0.0.1:6800/jsonrpc.")]
public DownloadClientLogLevel LogLevel { get; set; } = DownloadClientLogLevel.None;
[DisplayName("Sanitize filenames")]
[Description("Strip characters from filenames and torrent directory names that cause issues on Linux containers (square brackets, curly braces, control characters) and collapse multiple consecutive spaces. Recommended on; disable only if you need to preserve the exact filenames from the debrid provider.")]
public Boolean SanitizeFilenames { get; set; } = true;
[Description("Strip characters from filenames and torrent directory names that cause issues on Linux containers (square brackets, curly braces, control characters) and collapse multiple consecutive spaces. Defaults to on for Linux hosts and off elsewhere, since the issue only reproduces on Linux; toggle to override.")]
public Boolean SanitizeFilenames { get; set; } = OperatingSystem.IsLinux();
}
public class DbSettingsProvider