fix: make string.Split calls explicit to resolve ambiguous overloads

This commit is contained in:
Svend Roperos
2025-09-04 20:14:52 +02:00
parent 1aff3f86bd
commit c1e22faa03
5 changed files with 6 additions and 6 deletions
+1 -1
View File
@@ -115,7 +115,7 @@ namespace Duplicati.Library.Backend.Box
{
var parentid = "0";
foreach (var p in _path.Split(["/"], StringSplitOptions.RemoveEmptyEntries))
foreach (var p in _path.Split(new[] { '/' }, StringSplitOptions.RemoveEmptyEntries))
{
var el = (MiniFolder?)await PagedFileListResponse(parentid, true, cancelToken).FirstOrDefaultAsync(x => x.Name == p, cancellationToken: cancelToken).ConfigureAwait(false);
if (el == null)
@@ -221,7 +221,7 @@ namespace Duplicati.Library.Modules.Builtin
.ToList();
var filterhandler = pathFilters.Select(x => x.Filter)
.Concat(filter.Split([System.IO.Path.PathSeparator], StringSplitOptions.RemoveEmptyEntries))
.Concat(filter.Split(new[] { System.IO.Path.PathSeparator }, StringSplitOptions.RemoveEmptyEntries))
.Where(x => !string.IsNullOrWhiteSpace(x))
.Select(x => Utility.FilterExpression.StringToIFilter(x))
.Append(new Utility.FilterExpression())
@@ -480,7 +480,7 @@ namespace Duplicati.Library.Modules.Builtin
return;
// Explicit CR/LF types for all OSes instead of Environment.NewLine in case stdout producer
// script explicitly uses a different line ending from the OS the process is ran on.
foreach (var line in stdout.Split(["\r\n", "\r", "\n"], StringSplitOptions.RemoveEmptyEntries))
foreach (var line in stdout.Split(new[] { "\r\n", "\r", "\n" }, StringSplitOptions.RemoveEmptyEntries))
{
var prefix = LogActions.Keys.FirstOrDefault(p => line.StartsWith(p));
if (prefix == null) continue;
@@ -293,7 +293,7 @@ namespace Duplicati.Library.Modules.Builtin
commandlineOptions.TryGetValue(OPTION_EXTRA_PARAMETERS, out m_extraParameters);
m_acceptAnyCertificate = Utility.Utility.ParseBoolOption(commandlineOptions.AsReadOnly(), OPTION_ACCEPT_ANY_CERTIFICATE);
m_acceptSpecificCertificates = commandlineOptions.ContainsKey(OPTION_ACCEPT_SPECIFIED_CERTIFICATE) ? commandlineOptions[OPTION_ACCEPT_SPECIFIED_CERTIFICATE].Split([",", ";"], StringSplitOptions.RemoveEmptyEntries) : null;
m_acceptSpecificCertificates = commandlineOptions.ContainsKey(OPTION_ACCEPT_SPECIFIED_CERTIFICATE) ? commandlineOptions[OPTION_ACCEPT_SPECIFIED_CERTIFICATE].Split(new[] { ',', ';' }, StringSplitOptions.RemoveEmptyEntries) : null;
m_retries = Utility.Utility.ParseIntOption(commandlineOptions.AsReadOnly(), OPTION_SEND_HTTP_RETRIES, DEFAULT_RETRIES);
m_retryDelay = Utility.Utility.ParseTimespanOption(commandlineOptions.AsReadOnly(), OPTION_SEND_HTTP_RETRY_DELAY, DEFAULT_RETRY_DELAY);
@@ -90,7 +90,7 @@ public static class SslOptionsHelper
return new SslCertificateOptions(
Utility.ParseBoolOption(options, $"{prefix}{UseSSLOption}"),
Utility.ParseBoolOption(options, $"{prefix}{AcceptAnySslCertificateOption}"),
string.IsNullOrWhiteSpace(acceptSpecificCertificatesString) ? [] : acceptSpecificCertificatesString.Split([",", ";"], StringSplitOptions.RemoveEmptyEntries)
string.IsNullOrWhiteSpace(acceptSpecificCertificatesString) ? Array.Empty<string>() : acceptSpecificCertificatesString.Split(new[] { ',', ';' }, StringSplitOptions.RemoveEmptyEntries)
);
}
@@ -123,4 +123,4 @@ public static class SslOptionsHelper
return handler;
}
}
}
}