MM-36084 - Certificate Auth: Remove SAML dependency in license check (#18113)
Этот коммит содержится в:
коммит произвёл
GitHub
родитель
16d8640072
Коммит
d5e61e8093
@@ -1791,7 +1791,7 @@ func login(c *Context, w http.ResponseWriter, r *http.Request) {
|
||||
ldapOnly := props["ldap_only"] == "true"
|
||||
|
||||
if *c.App.Config().ExperimentalSettings.ClientSideCertEnable {
|
||||
if license := c.App.Srv().License(); license == nil || !*license.Features.SAML {
|
||||
if license := c.App.Srv().License(); license == nil || !*license.Features.FutureFeatures {
|
||||
c.Err = model.NewAppError("ClientSideCertNotAllowed", "api.user.login.client_side_cert.license.app_error", nil, "", http.StatusBadRequest)
|
||||
return
|
||||
}
|
||||
|
||||
@@ -3601,7 +3601,7 @@ func TestCBALogin(t *testing.T) {
|
||||
t.Run("primary", func(t *testing.T) {
|
||||
th := Setup(t).InitBasic()
|
||||
defer th.TearDown()
|
||||
th.App.Srv().SetLicense(model.NewTestLicense("saml"))
|
||||
th.App.Srv().SetLicense(model.NewTestLicense("future_features"))
|
||||
|
||||
th.App.UpdateConfig(func(cfg *model.Config) {
|
||||
*cfg.ServiceSettings.EnableBotAccountCreation = true
|
||||
@@ -3662,7 +3662,7 @@ func TestCBALogin(t *testing.T) {
|
||||
t.Run("secondary", func(t *testing.T) {
|
||||
th := Setup(t).InitBasic()
|
||||
defer th.TearDown()
|
||||
th.App.Srv().SetLicense(model.NewTestLicense("saml"))
|
||||
th.App.Srv().SetLicense(model.NewTestLicense("future_features"))
|
||||
|
||||
th.App.UpdateConfig(func(cfg *model.Config) {
|
||||
*cfg.ServiceSettings.EnableBotAccountCreation = true
|
||||
|
||||
@@ -167,8 +167,9 @@ func GenerateClientConfig(c *model.Config, telemetryID string, license *model.Li
|
||||
props["SamlLastNameAttributeSet"] = strconv.FormatBool(*c.SamlSettings.LastNameAttribute != "")
|
||||
props["SamlNicknameAttributeSet"] = strconv.FormatBool(*c.SamlSettings.NicknameAttribute != "")
|
||||
props["SamlPositionAttributeSet"] = strconv.FormatBool(*c.SamlSettings.PositionAttribute != "")
|
||||
}
|
||||
|
||||
// do this under the correct licensed feature
|
||||
if *license.Features.FutureFeatures {
|
||||
props["ExperimentalClientSideCertEnable"] = strconv.FormatBool(*c.ExperimentalSettings.ClientSideCertEnable)
|
||||
props["ExperimentalClientSideCertCheck"] = *c.ExperimentalSettings.ClientSideCertCheck
|
||||
}
|
||||
|
||||
Ссылка в новой задаче
Block a user