Skip to content

Commit

Permalink
feat: adjust cache duration in application.yml
Browse files Browse the repository at this point in the history
  • Loading branch information
Quentin-Guillemin committed Dec 22, 2022
1 parent a8b4696 commit 8383b7b
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,10 @@ public class ServiceGarHttpGet implements ServiceGar {
@Autowired
private ServiceCacheHistorique serviceCacheHistorique;

@Value("${service-gar-http-get.ressources-diffusables-uri:}")
@Value("${service-gar-http-get.cache-duration}")

This comment has been minimized.

Copy link
@jgribonvald

jgribonvald Jan 3, 2023

Member

Peut-être remplacé par service-gar-http-get.cache-duration:86400 pour définir une valeur par défaut et surchargeable via les fichier de properties

private int cacheDuration;

@Value("${service-gar-http-get.ressources-diffusables-uri}")
private String ressourcesDiffusablesUri;

@Value("${service-gar-http-get.download-location-path}")
Expand Down Expand Up @@ -99,7 +102,8 @@ private void ajouterRessource(RessourceDiffusable ressourceDiffusable) {
}

private void verifValidite() {
if (this.dateTelechargement == null || SECONDS.between(this.dateTelechargement, LocalDateTime.now()) > 86400) { // 24h.
if (log.isDebugEnabled()) log.debug("Check validity of file");
if (this.dateTelechargement == null || SECONDS.between(this.dateTelechargement, LocalDateTime.now()) > this.cacheDuration) {
this.telechargerFichier();
}
}
Expand Down
6 changes: 3 additions & 3 deletions src/main/resources/application.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ spring:
config.import: classpath:private.yml # Variables privées à ne pas diffuser.

service-gar-http-get:
cache-duration: 86400 # 24h.
ressources-diffusables-uri: '=== Ressources Diffusables URI ===' # Variable privée surchargée dans private.yml.
download-location-path: 'src/main/resources/downloads/ressourcesDiffusables.json' # Le fichier sera toujours JSON.

Expand All @@ -28,9 +29,8 @@ security-configuration:
cors:
enable: true
allow-credentials: true
allowed-origins: > # IP = PC de Anaël via VPN.
http://localhost:8080,
http://192.168.36.10:8080,
allowed-origins: >
http://localhost:5173,

This comment has been minimized.

Copy link
@jgribonvald

jgribonvald Jan 3, 2023

Member

alors le port par défaut est le 8080, si tu veux appliquer une conf local spécifique à ton poste il faut que tu passe un fichier de configuration additional-properties en paramètre de ta ligne de commande où spécifique à un profil de lancement de ta commande (cf dev docs de esup-publisher)

https://*.giprecia.net
exposed-headers: >
x-auth-token,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,12 +1,5 @@
package fr.recia.ressourcesdiffusablesapi.web.rest;

import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.content;
import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath;
import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;

import javax.annotation.PostConstruct;

import fr.recia.ressourcesdiffusablesapi.service.gar.ServiceGar;
import fr.recia.ressourcesdiffusablesapi.test.TestUtil;
import lombok.extern.slf4j.Slf4j;
Expand All @@ -22,6 +15,11 @@
import org.springframework.test.web.servlet.result.MockMvcResultHandlers;
import org.springframework.test.web.servlet.setup.MockMvcBuilders;

import javax.annotation.PostConstruct;

import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*;

This comment has been minimized.

Copy link
@jgribonvald

jgribonvald Jan 3, 2023

Member

les .* son éviter, et l'ordre a son impostance, les statics sont toujours en premier.
il faut que je te fournisse la doc pour les "checkstyle"


@ExtendWith(SpringExtension.class)
@Slf4j
@SpringBootTest
Expand Down

0 comments on commit 8383b7b

Please sign in to comment.