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.