EP-4532

Opetushallitus · Dec 12, 2024 · 2776519 · 2776519
1 parent 2d3ee64
commit 2776519
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 15 deletions.
diff --git a/...service/src/main/java/fi/vm/sade/eperusteet/ylops/config/WebSecurityConfigurationDev.java b/...service/src/main/java/fi/vm/sade/eperusteet/ylops/config/WebSecurityConfigurationDev.java
@@ -96,7 +96,8 @@ public UserDetailsService users() {
                         DevSecurityRole.ylops().crud().oid("1.2.246.562.10.61057016927").build(),
                         DevSecurityRole.ylops().crud().oid("1.2.246.562.10.68534785412").build(),
                         DevSecurityRole.ylops().crud().oid("1.2.246.562.10.83037752777").build(),
-                        DevSecurityRole.ylops().crud().oid("1.2.246.562.28.11332956371").build()
+                        DevSecurityRole.ylops().crud().oid("1.2.246.562.28.11332956371").build(),
+                        DevSecurityRole.ylops().crud().oid("1.2.246.562.10.81269623245").build()
                 )
                 .build();
         UserDetails ylops_helsinki = User.withDefaultPasswordEncoder()

diff --git a/.../src/main/java/fi/vm/sade/eperusteet/ylops/service/external/impl/KoodistoServiceImpl.java b/.../src/main/java/fi/vm/sade/eperusteet/ylops/service/external/impl/KoodistoServiceImpl.java
@@ -27,10 +27,7 @@ public class KoodistoServiceImpl implements KoodistoService {
 
     @Value("${koodisto.service.url:https://virkailija.opintopolku.fi/koodisto-service}")
     private String koodistoServiceUrl;
-
-    @Value("${koodisto.service.internal.url:${koodisto.service.url:''}}")
-    private String koodistoServiceInternalUrl;
-
+
     private static final String KOODISTO_API = "/rest/json/";
     private static final String YLARELAATIO = "relaatio/sisaltyy-ylakoodit/";
     private static final String ALARELAATIO = "relaatio/sisaltyy-alakoodit/";
@@ -60,7 +57,7 @@ public <T> T getForObject(String url, Class<T> responseType) {
 
     @Override
     public List<KoodistoKoodiDto> getAll(String koodisto) {
-        String url = koodistoServiceInternalUrl + KOODISTO_API + koodisto + "/koodi/";
+        String url = koodistoServiceUrl + KOODISTO_API + koodisto + "/koodi/";
         KoodistoKoodiDto[] koodistot = client.getForObject(url, KoodistoKoodiDto[].class);
         List<KoodistoKoodiDto> koodistoLista;
         if ("kunta".equals(koodisto)) {
@@ -79,7 +76,7 @@ public List<KoodistoKoodiDto> getAll(String koodisto) {
 
     @Override
     public KoodistoKoodiDto get(String koodisto, String koodi) {
-        String url = koodistoServiceInternalUrl + KOODISTO_API + koodisto + "/koodi/" + koodi;
+        String url = koodistoServiceUrl + KOODISTO_API + koodisto + "/koodi/" + koodi;
         return client.getForObject(url, KoodistoKoodiDto.class);
     }
 
@@ -100,14 +97,14 @@ public List<KoodistoKoodiDto> filterBy(String koodisto, String haku) {
 
     @Override
     public List<KoodistoKoodiDto> getAlarelaatio(String koodi) {
-        String url = koodistoServiceInternalUrl + KOODISTO_API + ALARELAATIO + koodi;
+        String url = koodistoServiceUrl + KOODISTO_API + ALARELAATIO + koodi;
         KoodistoKoodiDto[] koodistot = client.getForObject(url, KoodistoKoodiDto[].class);
         return koodistot == null ? null : Arrays.asList(koodistot);
     }
 
     @Override
     public List<KoodistoKoodiDto> getYlarelaatio(String koodi) {
-        String url = koodistoServiceInternalUrl + KOODISTO_API + YLARELAATIO + koodi;
+        String url = koodistoServiceUrl + KOODISTO_API + YLARELAATIO + koodi;
         KoodistoKoodiDto[] koodistot = client.getForObject(url, KoodistoKoodiDto[].class);
         return koodistot == null ? null : Arrays.asList(koodistot);
     }

diff --git a/.../main/java/fi/vm/sade/eperusteet/ylops/service/external/impl/OrganisaatioServiceImpl.java b/.../main/java/fi/vm/sade/eperusteet/ylops/service/external/impl/OrganisaatioServiceImpl.java
@@ -92,9 +92,6 @@ public static class Client {
         @Value("${cas.service.organisaatio-service:''}")
         private String serviceUrl;
 
-        @Value("${cas.service.organisaatio-service.internal:${cas.service.organisaatio-service:''}}")
-        private String serviceInternalUrl;
-
         @Value("#{'${fi.vm.sade.eperusteet.ylops.organisaatio-service.peruskoulu-oppilaitostyypit}'.split(',')}")
         private List<String> oppilaitostyypit;
 
@@ -120,7 +117,7 @@ public void init() {
         @Cacheable("organisaatiot")
         public JsonNode getOrganisaatio(String organisaatioOid) {
             OphHttpClient client = restClientFactory.get(serviceUrl, false);
-            String url = serviceInternalUrl + ORGANISAATIOT + organisaatioOid;
+            String url = serviceUrl + ORGANISAATIOT + organisaatioOid;
 
             OphHttpRequest request = OphHttpRequest.Builder
                     .get(url)
@@ -176,7 +173,7 @@ private JsonNode getLukiot(String hakuehto) {
 
         private JsonNode get(String hakuehto) {
             OphHttpClient client = restClientFactory.get(serviceUrl, false);
-            String url = serviceInternalUrl + ORGANISAATIOT + hakuehto + STATUS_KRITEERI;
+            String url = serviceUrl + ORGANISAATIOT + hakuehto + STATUS_KRITEERI;
 
             OphHttpRequest request = OphHttpRequest.Builder
                     .get(url)
@@ -286,7 +283,7 @@ public JsonNode getLukiotByOid(String oid) {
 
         public List<JsonNode> getRyhmat() {
             OphHttpClient client = restClientFactory.get(serviceUrl, false);
-            String url = serviceInternalUrl + ORGANISAATIOT + "v3/ryhmat";
+            String url = serviceUrl + ORGANISAATIOT + "v3/ryhmat";
 
             OphHttpRequest request = OphHttpRequest.Builder
                     .get(url)