Fixed: Ping endpoint no longer requires authentication

Closes #5396
This commit is contained in:
Mark McDowall 2023-01-31 23:39:59 -08:00
parent faccfe17a2
commit ad42d4a14c
1 changed files with 8 additions and 2 deletions

View File

@ -1,6 +1,9 @@
using System; using System;
using System.Collections.Generic;
using Microsoft.AspNetCore.Authorization;
using Microsoft.AspNetCore.Http; using Microsoft.AspNetCore.Http;
using Microsoft.AspNetCore.Mvc; using Microsoft.AspNetCore.Mvc;
using NzbDrone.Common.Cache;
using NzbDrone.Core.Configuration; using NzbDrone.Core.Configuration;
using NzbDrone.Http.Ping; using NzbDrone.Http.Ping;
@ -9,19 +12,22 @@ namespace NzbDrone.Http
public class PingController : Controller public class PingController : Controller
{ {
private readonly IConfigRepository _configRepository; private readonly IConfigRepository _configRepository;
private readonly ICached<IEnumerable<Config>> _cache;
public PingController(IConfigRepository configRepository) public PingController(IConfigRepository configRepository, ICacheManager cacheManager)
{ {
_configRepository = configRepository; _configRepository = configRepository;
_cache = cacheManager.GetCache<IEnumerable<Config>>(GetType());
} }
[AllowAnonymous]
[HttpGet("/ping")] [HttpGet("/ping")]
[Produces("application/json")] [Produces("application/json")]
public ActionResult<PingResource> GetStatus() public ActionResult<PingResource> GetStatus()
{ {
try try
{ {
_configRepository.All(); _cache.Get("ping", _configRepository.All, TimeSpan.FromSeconds(5));
} }
catch (Exception) catch (Exception)
{ {