actuator

Opetushallitus · Jan 24, 2025 · dffb79e · dffb79e
1 parent 93a86b6
commit dffb79e
Show file tree

Hide file tree

Showing 10 changed files with 24 additions and 15 deletions.
diff --git a/eperusteet/eperusteet-service/pom.xml b/eperusteet/eperusteet-service/pom.xml
@@ -5,7 +5,7 @@
     <parent>
         <groupId>fi.vm.sade.eperusteet</groupId>
         <artifactId>eperusteet-parent-pom</artifactId>
-        <version>1.0.3</version>
+        <version>1.0.4</version>
         <relativePath/>
     </parent>
 

diff --git a/...erusteet-service/src/main/java/fi/vm/sade/eperusteet/config/WebSecurityConfiguration.java b/...erusteet-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/...steet-service/src/main/java/fi/vm/sade/eperusteet/config/WebSecurityConfigurationDev.java b/...steet-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/...vice/src/main/java/fi/vm/sade/eperusteet/domain/lops2019/oppiaineet/Lops2019Oppiaine.java b/...vice/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
@@ -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/...usteet/eperusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/AIPEOppiaine.java b/...usteet/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/...erusteet-service/src/main/java/fi/vm/sade/eperusteet/domain/yl/LaajaalainenOsaaminen.java b/...erusteet-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<AIPEOpetuksenSisalto> 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<OpetuksenTavoite> opetuksenTavoitteet = new HashSet<>();
 

diff --git a/...teet-service/src/main/java/fi/vm/sade/eperusteet/service/impl/yl/OppiaineServiceImpl.java b/...teet-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<OppiaineSuppeaDto> getOppimaarat(Long perusteId, Long oppiaineId, Op
     }
 
     @Override
-    @Transactional(readOnly = false)
+    @Transactional
     public <T extends OppiaineBaseUpdateDto> OppiaineDto updateOppiaine(Long perusteId, UpdateDto<T> updateDto,
                                                                         OppiaineOpetuksenSisaltoTyyppi tyyppi) {
         T dto = updateDto.getDto();
@@ -377,7 +377,7 @@ public void updateOppiaineJarjestys(Long perusteId, List<OppiaineSuppeaDto> oppi
     }
 
     @Override
-    @Transactional(readOnly = false)
+    @Transactional
     public OppiaineenVuosiluokkaKokonaisuusDto updateOppiaineenVuosiluokkaKokonaisuus(Long perusteId, Long oppiaineId, UpdateDto<OppiaineenVuosiluokkaKokonaisuusDto> 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/...eet/eperusteet-service/src/test/java/fi/vm/sade/eperusteet/service/Lops2019ServiceIT.java b/...eet/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
@@ -18,6 +18,14 @@
 <!--        <appender-ref ref="CONSOLE" />-->
 <!--    </logger>-->
 
+<!--    <logger name="org.hibernate.SQL" level="DEBUG" additivity="false">-->
+<!--        <appender-ref ref="CONSOLE" />-->
+<!--    </logger>-->
+
+<!--    <logger name="org.hibernate.orm.jdbc.bind" level="TRACE" additivity="false">-->
+<!--        <appender-ref ref="CONSOLE" />-->
+<!--    </logger>-->
+
 <!--    <logger name="org.hibernate.orm.jdbc.bind" level="TRACE" additivity="false">-->
 <!--        <appender-ref ref="CONSOLE" />-->
 <!--    </logger>-->