Fix wrong check for empty API key
This commit is contained in:
@@ -18,7 +18,7 @@ namespace UmlautAdaptarr.Controllers
|
|||||||
[HttpGet]
|
[HttpGet]
|
||||||
public async Task<IActionResult> Caps([FromRoute] string apiKey, [FromRoute] string domain, [FromQuery] string? apikey)
|
public async Task<IActionResult> Caps([FromRoute] string apiKey, [FromRoute] string domain, [FromQuery] string? apikey)
|
||||||
{
|
{
|
||||||
if (_options.ApiKey != null && !apiKey.Equals(apiKey))
|
if (!string.IsNullOrEmpty(apikey) && !apiKey.Equals(apiKey))
|
||||||
{
|
{
|
||||||
_logger.LogWarning("Invalid or missing API key for request.");
|
_logger.LogWarning("Invalid or missing API key for request.");
|
||||||
return Unauthorized("Unauthorized: Invalid or missing API key.");
|
return Unauthorized("Unauthorized: Invalid or missing API key.");
|
||||||
|
|||||||
@@ -169,7 +169,7 @@ namespace UmlautAdaptarr.Controllers
|
|||||||
|
|
||||||
internal bool AssureApiKey(string apiKey)
|
internal bool AssureApiKey(string apiKey)
|
||||||
{
|
{
|
||||||
if (options.Value.ApiKey != null && !apiKey.Equals(options.Value.ApiKey))
|
if (!string.IsNullOrEmpty(options.Value.ApiKey) && !apiKey.Equals(options.Value.ApiKey))
|
||||||
{
|
{
|
||||||
logger.LogWarning("Invalid or missing API key for request.");
|
logger.LogWarning("Invalid or missing API key for request.");
|
||||||
return false;
|
return false;
|
||||||
|
|||||||
Reference in New Issue
Block a user