SONAR-7219 Remove sonar.security.updateUserAttributes property

diff --git a/server/sonar-web/src/main/webapp/WEB-INF/lib/need_authentication.rb b/server/sonar-web/src/main/webapp/WEB-INF/lib/need_authentication.rb
index 86c07064133743efb611dc9fc27d5cca7e06fdcf..b3fb607fa07afc4a347641da119a900f683899a0 100644 (file)
--- a/server/sonar-web/src/main/webapp/WEB-INF/lib/need_authentication.rb
+++ b/server/sonar-web/src/main/webapp/WEB-INF/lib/need_authentication.rb
@@ -148,7 +148,7 @@ class PluginRealm
         end
       else
         # Existing user
-        if details && Api::Utils.java_facade.getSettings().getBoolean('sonar.security.updateUserAttributes')
+        if details
           user.name = details.getName()
           user.email = details.getEmail()
         end

diff --git a/sonar-core/src/main/java/org/sonar/core/config/CorePropertyDefinitions.java b/sonar-core/src/main/java/org/sonar/core/config/CorePropertyDefinitions.java
index 1a4af6cd40991418023caf6576274642a65fe3ad..4a6cacffb4e1f6731c924d7d00cff86a3582595f 100644 (file)
--- a/sonar-core/src/main/java/org/sonar/core/config/CorePropertyDefinitions.java
+++ b/sonar-core/src/main/java/org/sonar/core/config/CorePropertyDefinitions.java
@@ -133,13 +133,6 @@ public class CorePropertyDefinitions {
         .defaultValue(String.valueOf(true))
         .hidden()
         .build(),
-      PropertyDefinition.builder(CoreProperties.CORE_AUTHENTICATOR_UPDATE_USER_ATTRIBUTES)
-        .name("Update user attributes")
-        .description("When using the LDAP or OpenID plugin, at each login, the user attributes (name, email, ...) are re-synchronized")
-        .hidden()
-        .type(PropertyType.BOOLEAN)
-        .defaultValue(String.valueOf(true))
-        .build(),
       PropertyDefinition.builder(CoreProperties.CORE_AUTHENTICATOR_IGNORE_STARTUP_FAILURE)
         .name("Ignore failures during authenticator startup")
         .type(PropertyType.BOOLEAN)

diff --git a/sonar-core/src/test/java/org/sonar/core/config/CorePropertyDefinitionsTest.java b/sonar-core/src/test/java/org/sonar/core/config/CorePropertyDefinitionsTest.java
index c5c39d3fb0f653a946621d606978cf1d29e84f4e..4e3e2550cde49df1f5d4cbc46d0c6b5d9087866b 100644 (file)
--- a/sonar-core/src/test/java/org/sonar/core/config/CorePropertyDefinitionsTest.java
+++ b/sonar-core/src/test/java/org/sonar/core/config/CorePropertyDefinitionsTest.java
@@ -29,6 +29,6 @@ public class CorePropertyDefinitionsTest {
   @Test
   public void all() {
     List<PropertyDefinition> defs = CorePropertyDefinitions.all();
-    assertThat(defs.size()).isGreaterThan(10);
+    assertThat(defs.size()).isGreaterThan(9);
   }
 }

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/CoreProperties.java b/sonar-plugin-api/src/main/java/org/sonar/api/CoreProperties.java
index c2dead92935356b592f61447605760f7752de24d..457d19caa021204d0dcbefb456554062c78c5580 100644 (file)
--- a/sonar-plugin-api/src/main/java/org/sonar/api/CoreProperties.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/CoreProperties.java
@@ -242,7 +242,9 @@ public interface CoreProperties {
 
   /**
    * @since 3.6
+   * @deprecated since 5.4. This feature is not supported anymore. See http://jira.sonarsource.com/browse/SONAR-7219
    */
+  @Deprecated
   String CORE_AUTHENTICATOR_UPDATE_USER_ATTRIBUTES = "sonar.security.updateUserAttributes";
 
   String SERVER_VERSION = "sonar.core.version";