diff options
author | Antoine Vigneau <antoine.vigneau@sonarsource.com> | 2024-09-24 17:35:15 +0200 |
---|---|---|
committer | sonartech <sonartech@sonarsource.com> | 2024-09-26 20:02:49 +0000 |
commit | ad84f39913decd027de30da8e1c10b9acafba2c1 (patch) | |
tree | ffa4aef2be3a9b0f71af0ee5b0239a498787e334 /server | |
parent | c29cd5af0d405717931895a73c57be3779fb0fd9 (diff) | |
download | sonarqube-ad84f39913decd027de30da8e1c10b9acafba2c1.tar.gz sonarqube-ad84f39913decd027de30da8e1c10b9acafba2c1.zip |
SONAR-23065 Fix SSF-607
Diffstat (limited to 'server')
2 files changed, 22 insertions, 9 deletions
diff --git a/server/sonar-webserver-webapi/src/main/java/org/sonar/server/setting/ws/SetAction.java b/server/sonar-webserver-webapi/src/main/java/org/sonar/server/setting/ws/SetAction.java index 9a2cffe9c12..aa9e86a8728 100644 --- a/server/sonar-webserver-webapi/src/main/java/org/sonar/server/setting/ws/SetAction.java +++ b/server/sonar-webserver-webapi/src/main/java/org/sonar/server/setting/ws/SetAction.java @@ -73,7 +73,8 @@ public class SetAction implements SettingsWsAction { static final Map<String, String> KEY_CONSTRAINTS = Map.of( "sonar.auth.gitlab.url", "sonar.auth.gitlab.secret.secured", "sonar.auth.github.webUrl", "sonar.auth.github.clientSecret.secured", - "sonar.auth.github.apiUrl", "sonar.auth.github.clientSecret.secured" + "sonar.auth.github.apiUrl", "sonar.auth.github.clientSecret.secured", + "email.smtp_host.secured", "email.smtp_password.secured" ); private final PropertyDefinitions propertyDefinitions; diff --git a/server/sonar-webserver-webapi/src/test/java/org/sonar/server/setting/ws/SetActionTest.java b/server/sonar-webserver-webapi/src/test/java/org/sonar/server/setting/ws/SetActionTest.java index 1cf965a2fc3..e2262a6fa9e 100644 --- a/server/sonar-webserver-webapi/src/test/java/org/sonar/server/setting/ws/SetActionTest.java +++ b/server/sonar-webserver-webapi/src/test/java/org/sonar/server/setting/ws/SetActionTest.java @@ -1191,23 +1191,35 @@ public class SetActionTest { } @Test - public void fail_when_key_constraints_are_not_met() { - propertyDb.insertProperty(newGlobalPropertyDto("sonar.auth.gitlab.secret.secured", "secret"), null, null, null, null); + @UseDataProvider("keyConstraints") + public void fail_when_key_constraints_are_not_met(String constrainedKey, String conditionKey) { + propertyDb.insertProperty(newGlobalPropertyDto(conditionKey, "secret"), null, null, null, null); assertThatThrownBy(() -> { - callForGlobalSetting("sonar.auth.gitlab.url", "http://new.url"); + callForGlobalSetting(constrainedKey, "newValue"); }) .isInstanceOf(BadRequestException.class) - .hasMessage("Setting 'sonar.auth.gitlab.secret.secured' must be empty to set 'sonar.auth.gitlab.url'"); + .hasMessage(format("Setting '%s' must be empty to set '%s'", conditionKey, constrainedKey)); } @Test - public void succeed_when_key_constraints_are_met() { - assertGlobalSettingIsNotSet("sonar.auth.gitlab.secret.secured"); + @UseDataProvider("keyConstraints") + public void succeed_when_key_constraints_are_met(String constrainedKey, String conditionKey) { + assertGlobalSettingIsNotSet(conditionKey); - callForGlobalSetting("sonar.auth.gitlab.url", "http://new.url"); + callForGlobalSetting(constrainedKey, "newValue"); - assertGlobalSetting("sonar.auth.gitlab.url", "http://new.url"); + assertGlobalSetting(constrainedKey, "newValue"); + } + + @DataProvider + public static Object[][] keyConstraints() { + return new Object[][] { + {"sonar.auth.gitlab.url", "sonar.auth.gitlab.secret.secured"}, + {"sonar.auth.github.webUrl", "sonar.auth.github.clientSecret.secured"}, + {"sonar.auth.github.apiUrl", "sonar.auth.github.clientSecret.secured"}, + {"email.smtp_host.secured", "email.smtp_password.secured"} + }; } @Test |