[Git][ultreiaio/ird-observe][develop-7.x] 2 commits: Erreur calcul des données sur V9 - See #2423 (can also happen en v7)^^ - Observe-commits

29 Jul 2022


      Tony CHEMIT pushed to branch develop-7.x at ultreiaio / ird-observe


Commits:
ab4772ca by Tony Chemit at 2022-07-29T14:21:55+02:00
Erreur calcul des données sur V9 - See #2423 (can also happen en v7)^^

- - - - -
45f8d489 by Tony Chemit at 2022-07-29T14:25:59+02:00
 Nommage des fichiers de log  - Closes #2149

- - - - -


3 changed files:

- server-configuration/src/main/java/fr/ird/observe/server/configuration/ServerConfig.java
- services/src/main/java/fr/ird/observe/services/service/actions/consolidate/ConsolidateDataService.java
- services/src/main/java/fr/ird/observe/services/service/actions/synchro/referential/ng/ReferentialSynchronizeService.java


Changes:

=====================================
server-configuration/src/main/java/fr/ird/observe/server/configuration/ServerConfig.java
=====================================
@@ -37,6 +37,7 @@ import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
+import java.util.Objects;
 
 /**
  * La configuration de l'application web.
@@ -71,9 +72,7 @@ public class ServerConfig extends GeneratedServerConfig implements CleanTemporar
 
     public void init(String... args) {
 
-        if (log.isInfoEnabled()) {
-            log.info("Starts to init ObServe server configuration...");
-        }
+        log.info("Starts to init ObServe server configuration...");
 
         try {
             get().parse(args);
@@ -86,9 +85,7 @@ public class ServerConfig extends GeneratedServerConfig implements CleanTemporar
         if (isDevMode() && !baseDirectory.exists()) {
             // on utilise un répertoire temporaire comme basedir
 
-            if (log.isInfoEnabled()) {
-                log.info("Using a dev mode configuration.");
-            }
+            log.info("Using a dev mode configuration.");
             try {
                 // Toujours s'assurer que le répertoire temporarie du système existe
                 Path tmpdir = Paths.get(System.getProperty("java.io.tmpdir"));
@@ -100,9 +97,7 @@ public class ServerConfig extends GeneratedServerConfig implements CleanTemporar
             } catch (IOException e) {
                 throw new ObserveWebApplicationConfigInitException("could not create temporary basedir", e);
             }
-            if (log.isInfoEnabled()) {
-                log.info("Dev mode detected, use temporary basedir: " + baseDirectory);
-            }
+            log.info("Dev mode detected, use temporary basedir: " + baseDirectory);
             setBaseDirectory(baseDirectory);
         }
 
@@ -125,12 +120,10 @@ public class ServerConfig extends GeneratedServerConfig implements CleanTemporar
         File databasesConfigurationFile = getDatabasesConfigurationFile();
         if (!databasesConfigurationFile.exists()) {
 
-            if (log.isInfoEnabled()) {
-                log.info("Generate default databases.yml");
-            }
+            log.info("Generate default databases.yml");
 
             try {
-                CharSource charSource = Resources.asCharSource(getClass().getResource("/defaultDatabases.yml"), StandardCharsets.UTF_8);
+                CharSource charSource = Resources.asCharSource(Objects.requireNonNull(getClass().getResource("/defaultDatabases.yml")), StandardCharsets.UTF_8);
                 Files.write(databasesConfigurationFile.toPath(), charSource.readLines());
             } catch (IOException e) {
                 throw new ObserveWebApplicationConfigInitException("Impossible de créer un fichier de configuration des bases", e);
@@ -140,11 +133,9 @@ public class ServerConfig extends GeneratedServerConfig implements CleanTemporar
         File usersConfigurationFile = getUsersConfigurationFile();
         if (!usersConfigurationFile.exists()) {
 
-            if (log.isInfoEnabled()) {
-                log.info("Generate default users.yml");
-            }
+            log.info("Generate default users.yml");
             try {
-                CharSource charSource = Resources.asCharSource(getClass().getResource("/defaultUsers.yml"), StandardCharsets.UTF_8);
+                CharSource charSource = Resources.asCharSource(Objects.requireNonNull(getClass().getResource("/defaultUsers.yml")), StandardCharsets.UTF_8);
                 Files.write(usersConfigurationFile.toPath(), charSource.readLines());
             } catch (IOException e) {
                 throw new ObserveWebApplicationConfigInitException("Impossible de créer un fichier de configuration des utilisateurs", e);
@@ -163,7 +154,9 @@ public class ServerConfig extends GeneratedServerConfig implements CleanTemporar
     private void initLog() throws IOException {
 
         File logFile = getLog4jConfigurationFile();
-
+        // always delete log file to regenerate it
+        // See https://gitlab.com/ultreiaio/ird-observe/-/issues/2149
+        Files.deleteIfExists(logFile.toPath());
         log.info("Chargement du fichier de log : " + logFile);
 
         ObserveUtil.loadLogConfiguration(getClass().getResource("/observeweb-log4j2.xml"), logFile.toPath(), this);


=====================================
services/src/main/java/fr/ird/observe/services/service/actions/consolidate/ConsolidateDataService.java
=====================================
@@ -50,13 +50,13 @@ public interface ConsolidateDataService extends ObserveService {
      * Consolidate trip data using given {@code request} and produce a result.
      *
      *
-     * @param simplifiedObjectTypeSpecializedRules
+     * @param simplifiedObjectTypeSpecializedRules rules to apply
      * @param request la demande de consolidation
      * @return le résultat de la consolidation de données pour chaque marée qui a été modifée.
      */
     @ReadDataPermission
     @WriteDataPermission
-    @Post
+    @Post(timeOut = 30)
     @Write
     ConsolidateTripSeineDataResult consolidateTripSeine(SimplifiedObjectTypeSpecializedRules simplifiedObjectTypeSpecializedRules, ConsolidateTripSeineDataRequest request);
 
@@ -65,7 +65,7 @@ public interface ConsolidateDataService extends ObserveService {
      * modifiées.
      *
      *
-     * @param simplifiedObjectTypeSpecializedRules
+     * @param simplifiedObjectTypeSpecializedRules rules to apply
      * @param request request
      * @return {@code null} if no modification done, or {@code result} of operation.
      */


=====================================
services/src/main/java/fr/ird/observe/services/service/actions/synchro/referential/ng/ReferentialSynchronizeService.java
=====================================
@@ -35,7 +35,7 @@ import io.ultreia.java4all.http.spi.Post;
 public interface ReferentialSynchronizeService extends ObserveService {
 
     // Note : sur cette méthode aucun droit métier à poser, les actions sont légitimes
-    @Post
+    @Post(timeOut = 10)
     ReferentialSynchronizeSqlListRequest produceSqlListRequest(ReferentialSynchronizeRequest request);
 
     // Note : sur cette méthode aucun droit métier à poser, les actions sont légitimes



View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/7b2ec932076171d53fe64e852...

-- 
View it on GitLab: https://gitlab.com/ultreiaio/ird-observe/-/compare/7b2ec932076171d53fe64e852...
You're receiving this email because of your account on gitlab.com.

    

[Git][ultreiaio/ird-observe][develop-7.x] 2 commits: Erreur calcul des données sur V9 - See #2423 (can also happen en v7)^^

Tony CHEMIT (＠tchemit)

tags

participants (1)