Fixes Issue17

Author: phrocker

api/src/main/java/io/sentrius/sso/controllers/api/SystemApiController.java

@@ -149,7 +149,7 @@ public String setOption(HttpServletRequest request, HttpServletResponse response
         } else if (someFailed) {
             updateOption = MessagingUtil.SOME_SETTINGS_FAIL;
         }
-        return "redirect:/sso/v1/settings?message=" + MessagingUtil.getMessageId(updateOption);
+        return "redirect:/sso/v1/system/settings?message=" + MessagingUtil.getMessageId(updateOption);
 
     }
 

core/src/main/java/io/sentrius/sso/core/config/ThreadSafeDynamicPropertiesService.java

@@ -79,7 +79,7 @@ private void loadProperties() {
     public void updateProperty(String key, String value) throws IOException {
         lock.writeLock().lock();
         try{
-            configurationOptionRepository.save(ConfigurationOption.builder().build());
+            configurationOptionRepository.save(ConfigurationOption.builder().configurationName(key).configurationValue(null == value ? "" : value).build());
         } finally {
             lock.writeLock().unlock();
         }
@@ -89,7 +89,7 @@ public void updateProperty(String key, String value) throws IOException {
     public String getProperty(String key, String defaultValue) {
         lock.readLock().lock();
         try {
-            var dbOption = configurationOptionRepository.findByConfigurationName(key);
+            var dbOption = configurationOptionRepository.findLatestByConfigurationName(key);
             if (dbOption.isEmpty()) {
                 return properties.getProperty(key, defaultValue);
             }

core/src/main/java/io/sentrius/sso/core/repository/ConfigurationOptionRepository.java

@@ -3,8 +3,11 @@
 import java.util.Optional;
 import io.sentrius.sso.core.model.ConfigurationOption;
 import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.data.jpa.repository.Query;
+import org.springframework.data.repository.query.Param;
 
 public interface ConfigurationOptionRepository extends JpaRepository<ConfigurationOption, Long> {
 
-    Optional<ConfigurationOption> findByConfigurationName(String configurationName);
+    @Query("SELECT c FROM ConfigurationOption c WHERE c.configurationName = :configurationName ORDER BY c.id DESC LIMIT 1")
+    Optional<ConfigurationOption> findLatestByConfigurationName(@Param("configurationName") String configurationName);
 }