Archive-Team
/
sonarr-repo-only
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed: (API) Prevent NullRef for empty ids in bulk endpoint

This commit is contained in:
Bogdan 2023-07-11 10:24:03 +03:00
parent edaac4ce15
commit e6ab91871d
2 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/Sonarr.Api.V3/ProviderBulkResource.cs
View File

@ -9,6 +9,11 @@ namespace Sonarr.Api.V3
public List<int> Ids { get; set; } public List<int> Ids { get; set; }
public List<int> Tags { get; set; } public List<int> Tags { get; set; }
public ApplyTags ApplyTags { get; set; } public ApplyTags ApplyTags { get; set; }
public ProviderBulkResource()
{
Ids = new List<int>();
}
} }
public class ProviderBulkResourceMapper<TProviderBulkResource, TProviderDefinition> public class ProviderBulkResourceMapper<TProviderBulkResource, TProviderDefinition>

5
src/Sonarr.Api.V3/ProviderControllerBase.cs
View File

@ -106,6 +106,11 @@ namespace Sonarr.Api.V3
[Produces("application/json")] [Produces("application/json")]
public ActionResult<TProviderResource> UpdateProvider([FromBody] TBulkProviderResource providerResource) public ActionResult<TProviderResource> UpdateProvider([FromBody] TBulkProviderResource providerResource)
{ {
if (!providerResource.Ids.Any())
{
throw new BadRequestException("ids must be provided");
}
var definitionsToUpdate = _providerFactory.Get(providerResource.Ids).ToList(); var definitionsToUpdate = _providerFactory.Get(providerResource.Ids).ToList();
foreach (var definition in definitionsToUpdate) foreach (var definition in definitionsToUpdate)