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

Commits:

3 changed files:

Changes:

  • server-configuration/src/main/java/fr/ird/observe/server/configuration/ServerConfig.java
    ... ... @@ -37,6 +37,7 @@ import java.nio.charset.StandardCharsets;
    37 37 
     import java.nio.file.Files;
    38 38 
     import java.nio.file.Path;
    39 39 
     import java.nio.file.Paths;
    40 
    +import java.util.Objects;
    40 41
    41 42 
     /**
    42 43 
      * La configuration de l'application web.
    ... ... @@ -71,9 +72,7 @@ public class ServerConfig extends GeneratedServerConfig implements CleanTemporar
    71 72
    72 73 
         public void init(String... args) {
    73 74
    74 
    -        if (log.isInfoEnabled()) {
    75 
    -            log.info("Starts to init ObServe server configuration...");
    76 
    -        }
    75 
    +        log.info("Starts to init ObServe server configuration...");
    77 76
    78 77 
             try {
    79 78 
                 get().parse(args);
    ... ... @@ -86,9 +85,7 @@ public class ServerConfig extends GeneratedServerConfig implements CleanTemporar
    86 85 
             if (isDevMode() && !baseDirectory.exists()) {
    87 86 
                 // on utilise un répertoire temporaire comme basedir
    88 87
    89 
    -            if (log.isInfoEnabled()) {
    90 
    -                log.info("Using a dev mode configuration.");
    91 
    -            }
    88 
    +            log.info("Using a dev mode configuration.");
    92 89 
                 try {
    93 90 
                     // Toujours s'assurer que le répertoire temporarie du système existe
    94 91 
                     Path tmpdir = Paths.get(System.getProperty("java.io.tmpdir"));
    ... ... @@ -100,9 +97,7 @@ public class ServerConfig extends GeneratedServerConfig implements CleanTemporar
    100 97 
                 } catch (IOException e) {
    101 98 
                     throw new ObserveWebApplicationConfigInitException("could not create temporary basedir", e);
    102 99 
                 }
    103 
    -            if (log.isInfoEnabled()) {
    104 
    -                log.info("Dev mode detected, use temporary basedir: " + baseDirectory);
    105 
    -            }
    100 
    +            log.info("Dev mode detected, use temporary basedir: " + baseDirectory);
    106 101 
                 setBaseDirectory(baseDirectory);
    107 102 
             }
    108 103
    ... ... @@ -125,12 +120,10 @@ public class ServerConfig extends GeneratedServerConfig implements CleanTemporar
    125 120 
             File databasesConfigurationFile = getDatabasesConfigurationFile();
    126 121 
             if (!databasesConfigurationFile.exists()) {
    127 122
    128 
    -            if (log.isInfoEnabled()) {
    129 
    -                log.info("Generate default databases.yml");
    130 
    -            }
    123 
    +            log.info("Generate default databases.yml");
    131 124
    132 125 
                 try {
    133 
    -                CharSource charSource = Resources.asCharSource(getClass().getResource("/defaultDatabases.yml"), StandardCharsets.UTF_8);
    126 
    +                CharSource charSource = Resources.asCharSource(Objects.requireNonNull(getClass().getResource("/defaultDatabases.yml")), StandardCharsets.UTF_8);
    134 127 
                     Files.write(databasesConfigurationFile.toPath(), charSource.readLines());
    135 128 
                 } catch (IOException e) {
    136 129 
                     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
    140 133 
             File usersConfigurationFile = getUsersConfigurationFile();
    141 134 
             if (!usersConfigurationFile.exists()) {
    142 135
    143 
    -            if (log.isInfoEnabled()) {
    144 
    -                log.info("Generate default users.yml");
    145 
    -            }
    136 
    +            log.info("Generate default users.yml");
    146 137 
                 try {
    147 
    -                CharSource charSource = Resources.asCharSource(getClass().getResource("/defaultUsers.yml"), StandardCharsets.UTF_8);
    138 
    +                CharSource charSource = Resources.asCharSource(Objects.requireNonNull(getClass().getResource("/defaultUsers.yml")), StandardCharsets.UTF_8);
    148 139 
                     Files.write(usersConfigurationFile.toPath(), charSource.readLines());
    149 140 
                 } catch (IOException e) {
    150 141 
                     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
    163 154 
         private void initLog() throws IOException {
    164 155
    165 156 
             File logFile = getLog4jConfigurationFile();
    166 
    -
    157 
    +        // always delete log file to regenerate it
    158 
    +        // See https://gitlab.com/ultreiaio/ird-observe/-/issues/2149
    159 
    +        Files.deleteIfExists(logFile.toPath());
    167 160 
             log.info("Chargement du fichier de log : " + logFile);
    168 161
    169 162 
             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 {
    50 50 
          * Consolidate trip data using given {@code request} and produce a result.
    51 51 
          *
    52 52 
          *
    53 
    -     * @param simplifiedObjectTypeSpecializedRules
    53 
    +     * @param simplifiedObjectTypeSpecializedRules rules to apply
    54 54 
          * @param request la demande de consolidation
    55 55 
          * @return le résultat de la consolidation de données pour chaque marée qui a été modifée.
    56 56 
          */
    57 57 
         @ReadDataPermission
    58 58 
         @WriteDataPermission
    59 
    -    @Post
    59 
    +    @Post(timeOut = 30)
    60 60 
         @Write
    61 61 
         ConsolidateTripSeineDataResult consolidateTripSeine(SimplifiedObjectTypeSpecializedRules simplifiedObjectTypeSpecializedRules, ConsolidateTripSeineDataRequest request);
    62 62
    ... ... @@ -65,7 +65,7 @@ public interface ConsolidateDataService extends ObserveService {
    65 65 
          * modifiées.
    66 66 
          *
    67 67 
          *
    68 
    -     * @param simplifiedObjectTypeSpecializedRules
    68 
    +     * @param simplifiedObjectTypeSpecializedRules rules to apply
    69 69 
          * @param request request
    70 70 
          * @return {@code null} if no modification done, or {@code result} of operation.
    71 71 
          */

  • 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;
    35 35 
     public interface ReferentialSynchronizeService extends ObserveService {
    36 36
    37 37 
         // Note : sur cette méthode aucun droit métier à poser, les actions sont légitimes
    38 
    -    @Post
    38 
    +    @Post(timeOut = 10)
    39 39 
         ReferentialSynchronizeSqlListRequest produceSqlListRequest(ReferentialSynchronizeRequest request);
    40 40
    41 41 
         // Note : sur cette méthode aucun droit métier à poser, les actions sont légitimes

    • —
    View it on GitLab.
    You're receiving this email because of your account on gitlab.com. Manage all notifications · Help