From dffb79ed766e9a73225764afda73149560c76bb3 Mon Sep 17 00:00:00 2001 From: Olli-Pekka Lehtokallio Date: Fri, 17 Jan 2025 12:46:47 +0200 Subject: [PATCH] actuator --- eperusteet/eperusteet-service/pom.xml | 2 +- .../config/WebSecurityConfiguration.java | 2 +- .../config/WebSecurityConfigurationDev.java | 2 +- .../lops2019/oppiaineet/Lops2019Oppiaine.java | 1 + .../vm/sade/eperusteet/domain/yl/AIPEKurssi.java | 1 + .../sade/eperusteet/domain/yl/AIPEOppiaine.java | 1 + .../domain/yl/LaajaalainenOsaaminen.java | 4 ++-- .../service/impl/yl/OppiaineServiceImpl.java | 16 ++++++++-------- .../eperusteet/service/Lops2019ServiceIT.java | 2 -- .../src/test/resources/logback.xml | 8 ++++++++ 10 files changed, 24 insertions(+), 15 deletions(-) diff --git a/eperusteet/eperusteet-service/pom.xml b/eperusteet/eperusteet-service/pom.xml index 3d2bcc412..7f449a4e5 100644 --- a/eperusteet/eperusteet-service/pom.xml +++ b/eperusteet/eperusteet-service/pom.xml @@ -5,7 +5,7 @@ fi.vm.sade.eperusteet eperusteet-parent-pom - 1.0.3 + 1.0.4 diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/config/WebSecurityConfiguration.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/config/WebSecurityConfiguration.java index b98bdb212..a4e432da3 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/config/WebSecurityConfiguration.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/config/WebSecurityConfiguration.java @@ -136,7 +136,7 @@ public SecurityFilterChain securityFilterChain(HttpSecurity http) throws Excepti .headers(AbstractHttpConfigurer::disable) .csrf(AbstractHttpConfigurer::disable) .authorizeHttpRequests((authorize) -> authorize - .requestMatchers("/buildversion.txt").permitAll() + .requestMatchers("/actuator/health").permitAll() .requestMatchers(HttpMethod.GET, "/api/**").permitAll() .requestMatchers(HttpMethod.GET, "/").permitAll() .anyRequest().authenticated()) diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/config/WebSecurityConfigurationDev.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/config/WebSecurityConfigurationDev.java index ac948da52..a653cd362 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/config/WebSecurityConfigurationDev.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/config/WebSecurityConfigurationDev.java @@ -38,7 +38,7 @@ public SecurityFilterChain securityFilterChain(HttpSecurity http) throws Excepti http .csrf(AbstractHttpConfigurer::disable) .authorizeHttpRequests((authorize) -> authorize - .requestMatchers("/buildversion.txt").permitAll() + .requestMatchers("/actuator/health").permitAll() .requestMatchers(HttpMethod.GET, "/api/**").permitAll() .requestMatchers(HttpMethod.GET, "/api-docs/**").permitAll() .requestMatchers(HttpMethod.GET, "/").permitAll() diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/lops2019/oppiaineet/Lops2019Oppiaine.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/lops2019/oppiaineet/Lops2019Oppiaine.java index e9e7527da..93c71245b 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/lops2019/oppiaineet/Lops2019Oppiaine.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/lops2019/oppiaineet/Lops2019Oppiaine.java @@ -117,6 +117,7 @@ public class Lops2019Oppiaine extends AbstractAuditedReferenceableEntity impleme private Integer jarjestys; @Getter + @Setter @NotAudited @ManyToOne(fetch = FetchType.LAZY) @JoinTable(name = "yl_lops2019_oppiaine_oppimaara", diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/AIPEKurssi.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/AIPEKurssi.java index 6ef23d4f0..4f953d964 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/AIPEKurssi.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/AIPEKurssi.java @@ -55,6 +55,7 @@ public class AIPEKurssi extends AbstractAuditedReferenceableEntity implements AI private Integer jarjestys; @Getter + @Setter @ManyToOne(fetch = FetchType.LAZY) @NotAudited @JoinTable(name = "aipeoppiaine_aipekurssi", diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/AIPEOppiaine.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/AIPEOppiaine.java index 691e566ff..65df38707 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/AIPEOppiaine.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/AIPEOppiaine.java @@ -163,6 +163,7 @@ public class AIPEOppiaine extends AbstractAuditedReferenceableEntity implements private TekstiPalanen kielikasvatus; @Getter + @Setter @NotAudited @ManyToOne(fetch = FetchType.LAZY) @JoinTable(name = "aipeoppiaine_aipeoppiaine", diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/LaajaalainenOsaaminen.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/LaajaalainenOsaaminen.java index 4ef649c1f..852f4eb78 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/LaajaalainenOsaaminen.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/LaajaalainenOsaaminen.java @@ -70,7 +70,7 @@ public class LaajaalainenOsaaminen extends AbstractReferenceableEntity implement @RelatesToPeruste @ManyToMany(fetch = FetchType.LAZY) @JoinTable(name = "yl_aipe_opetuksensisalto_yl_laajaalainen_osaaminen", - joinColumns = @JoinColumn(name = "laajaalaisetosaamiset_id", nullable = false), + joinColumns = @JoinColumn(name = "laajaalaisetosaamiset_id",nullable = false), inverseJoinColumns = @JoinColumn(name = "yl_aipe_opetuksensisalto_id", nullable = false)) private Set aipeSisallot = new HashSet<>(); @@ -79,7 +79,7 @@ public class LaajaalainenOsaaminen extends AbstractReferenceableEntity implement @RelatesToPeruste @ManyToMany(fetch = FetchType.LAZY) @JoinTable(name = "yl_opetuksen_tavoite_yl_laajaalainen_osaaminen", - joinColumns = @JoinColumn(name = "laajattavoitteet_id", nullable = false), + joinColumns = @JoinColumn(name = "laajattavoitteet_id",nullable = false), inverseJoinColumns = @JoinColumn(name = "yl_opetuksen_tavoite_id", nullable = false)) private Set opetuksenTavoitteet = new HashSet<>(); diff --git a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/yl/OppiaineServiceImpl.java b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/yl/OppiaineServiceImpl.java index ce74cdb18..e24fffecd 100644 --- a/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/yl/OppiaineServiceImpl.java +++ b/eperusteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/yl/OppiaineServiceImpl.java @@ -44,7 +44,7 @@ import static java.util.stream.Collectors.toSet; @Service -@Transactional(readOnly = true) +@Transactional public class OppiaineServiceImpl implements OppiaineService { @Autowired @@ -83,7 +83,7 @@ public class OppiaineServiceImpl implements OppiaineService { private static final Logger LOG = LoggerFactory.getLogger(OppiaineServiceImpl.class); @Override - @Transactional(readOnly = false) + @Transactional public OppiaineDto addOppiaine(Long perusteId, OppiaineDto dto, OppiaineOpetuksenSisaltoTyyppi tyyppi) { AbstractOppiaineOpetuksenSisalto sisalto = tyyppi.getLockedByPerusteId(applicationContext, perusteId); if (sisalto != null) { @@ -111,7 +111,7 @@ private Oppiaine saveOppiaine(OppiaineBaseUpdateDto dto) { } @Override - @Transactional(readOnly = false) + @Transactional public OppiaineenVuosiluokkaKokonaisuusDto addOppiaineenVuosiluokkaKokonaisuus(Long perusteId, Long oppiaineId, OppiaineenVuosiluokkaKokonaisuusDto dto) { PerusopetuksenPerusteenSisalto sisalto = perusOpetuksenSisaltoRepository.findByPerusteId(perusteId); @@ -134,7 +134,7 @@ private OppiaineenVuosiluokkaKokonaisuus addOppiaineenVuosiluokkaKokonaisuus(Opp } @Override - @Transactional(readOnly = false) + @Transactional public void deleteOppiaine(Long perusteId, Long oppiaineId, OppiaineOpetuksenSisaltoTyyppi tyyppi) { Oppiaine aine = oppiaineRepository.findOne(oppiaineId); AbstractOppiaineOpetuksenSisalto sisalto = tyyppi.getRepository(applicationContext).findByPerusteId(perusteId); @@ -174,7 +174,7 @@ public void deleteOppiaine(Long perusteId, Long oppiaineId, OppiaineOpetuksenSis } @Override - @Transactional(readOnly = false) + @Transactional public void deleteOppiaineenVuosiluokkaKokonaisuus(Long perusteId, Long oppiaineId, Long vuosiluokkaKokonaisuusId) { deleteOppiaineenVuosiluokkaKokonaisuus(perusteId, oppiaineId, vuosiluokkaKokonaisuusId, true); } @@ -295,7 +295,7 @@ public List getOppimaarat(Long perusteId, Long oppiaineId, Op } @Override - @Transactional(readOnly = false) + @Transactional public OppiaineDto updateOppiaine(Long perusteId, UpdateDto updateDto, OppiaineOpetuksenSisaltoTyyppi tyyppi) { T dto = updateDto.getDto(); @@ -377,7 +377,7 @@ public void updateOppiaineJarjestys(Long perusteId, List oppi } @Override - @Transactional(readOnly = false) + @Transactional public OppiaineenVuosiluokkaKokonaisuusDto updateOppiaineenVuosiluokkaKokonaisuus(Long perusteId, Long oppiaineId, UpdateDto updateDto) { PerusopetuksenPerusteenSisalto sisalto = perusOpetuksenSisaltoRepository.findByPerusteId(perusteId); OppiaineenVuosiluokkaKokonaisuusDto tmp = mapper.map(doUpdateOppiaineenVuosiluokkaKokonaisuus(sisalto, oppiaineId, updateDto.getDto(), true), OppiaineenVuosiluokkaKokonaisuusDto.class); @@ -436,7 +436,7 @@ private Oppiaine getAndLockOppiaine(Long perusteId, Long oppiaineId) { } @Override - @Transactional(readOnly = false) + @Transactional public OpetuksenKohdealueDto addKohdealue(Long perusteId, Long oppiaineId, OpetuksenKohdealueDto kohdealue) { Oppiaine aine = getAndLockOppiaine(perusteId, oppiaineId); OpetuksenKohdealue kohde = null; diff --git a/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/Lops2019ServiceIT.java b/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/Lops2019ServiceIT.java index 40b902d1c..9b6838965 100644 --- a/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/Lops2019ServiceIT.java +++ b/eperusteet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/Lops2019ServiceIT.java @@ -496,7 +496,6 @@ private Long createLops2019PerusteAndChangeTilaJulkaistu() { Lops2019OppiaineDto oppimaara1Dto = new Lops2019OppiaineDto(); oppimaara1Dto.setNimi(LokalisoituTekstiDto.of("om1")); - oppimaara1Dto.setOppiaine(Reference.of(dto.getId())); final KoodiDto koodi1Dto = new KoodiDto(); koodi1Dto.setUri("oppiaineetjaoppimaaratlops2021_om1"); koodi1Dto.setKoodisto("oppiaineetjaoppimaaratlops2021"); @@ -506,7 +505,6 @@ private Long createLops2019PerusteAndChangeTilaJulkaistu() { Lops2019OppiaineDto oppimaara2Dto = new Lops2019OppiaineDto(); oppimaara2Dto.setNimi(LokalisoituTekstiDto.of("om2")); - oppimaara2Dto.setOppiaine(Reference.of(dto.getId())); final KoodiDto koodi2Dto = new KoodiDto(); koodi2Dto.setUri("oppiaineetjaoppimaaratlops2021_om2"); koodi2Dto.setKoodisto("oppiaineetjaoppimaaratlops2021"); diff --git a/eperusteet/eperusteet-service/src/test/resources/logback.xml b/eperusteet/eperusteet-service/src/test/resources/logback.xml index 4a2975413..51a6c1ce4 100644 --- a/eperusteet/eperusteet-service/src/test/resources/logback.xml +++ b/eperusteet/eperusteet-service/src/test/resources/logback.xml @@ -18,6 +18,14 @@ + + + + + + + +