From 4aeaf04b625245b268f9ce053ffb5c345cb77c13 Mon Sep 17 00:00:00 2001 From: Qstick Date: Sun, 28 Apr 2019 00:19:38 -0400 Subject: [PATCH] Fixed: Don't allow HTTPS to be set to same port via API/UI --- src/Lidarr.Api.V1/Config/HostConfigModule.cs | 1 + 1 file changed, 1 insertion(+) diff --git a/src/Lidarr.Api.V1/Config/HostConfigModule.cs b/src/Lidarr.Api.V1/Config/HostConfigModule.cs index da4fce8ea..05cdf7b89 100644 --- a/src/Lidarr.Api.V1/Config/HostConfigModule.cs +++ b/src/Lidarr.Api.V1/Config/HostConfigModule.cs @@ -43,6 +43,7 @@ namespace Lidarr.Api.V1.Config SharedValidator.RuleFor(c => c.Password).NotEmpty().When(c => c.AuthenticationMethod != AuthenticationType.None); SharedValidator.RuleFor(c => c.SslPort).ValidPort().When(c => c.EnableSsl); + SharedValidator.RuleFor(c => c.SslPort).NotEqual(c => c.Port).When(c => c.EnableSsl); SharedValidator.RuleFor(c => c.SslCertHash).NotEmpty().When(c => c.EnableSsl && OsInfo.IsWindows); SharedValidator.RuleFor(c => c.Branch).NotEmpty().WithMessage("Branch name is required, 'master' is the default");