OY-4717 riippuvuuksien megakarsinta ja tietoturvapäivitykset #479

marjakari · 2025-01-08T08:51:14Z

Tiketin subtaskeista ilmenee tarkemmin mitä tehty, pääpiirteissään:

poistettu backendista toiminnallisuudet joita ei enää käytössä (vanhan hakemuspalvelun hakemusten muokkaus, esikatselu ym toiminnot)
korvattu haku-app-kirjasto hakemalla haku-app hakemukset suoraan documentdb-kannasta ja kopioitu tarvittavat domain-luokat
samassa yhteydessä poistettu vanha haku-appin testifixture ja kopioitu tarvittavilta osin testidataa haku-appin reposta
korvattu haku-app-kantayhteyteen käytetyt mongo-kirjastot tuoreemmilla
poistettu vanhentuneet apukirjastoriippuvuudet (scala-cas, scala-utils) ja kopioitu niistä tarvittavat asiat repoon
poistettu turhat integraatiot (vanha tarjonta, koodisto)
korvattu kaikissa integraatioissa scala-cas java-casilla (paitsi oppijan tunnistautuminen koska attribuuttien parsinta ei onnistu java-casilla)
päivitetty scala versioon 2.12 ja java versioon 17
päivitetty jäljellä olevat kirjastoriippuvuudet
otettu sähköpostin lähetystoiminnossa käyttöön uusi viestinvälityspalvelu (kirjaston avulla)
frontin osalta minimaaliset muutokset "ettei vaan mikään hajoa"

…ummy-dao

…ta, kopioitu tarvittavat koodit repoon

…tavat koodit repoon, poistettu group-emailer-kirjasto

…oista

…töön. Mockitoa käyttävät testit tilapäisesti kommentoitu koska vaatii versionnoston

…ottokuittaukseen, turhia depsuja pois

…ittuvuudet fiksattu

…ala-pluginista

vaeinoe

Näin isoa kokonaisuutta suht. tuntemattomasta palvelusta on hankalaa lukea ihan suurennuslasilla rivi riviltä läpi, mutta ei osunut läpikäynnissä silmään mitään isompia ongelmia. Yksi mahd. korjauskohta ja muutama huomio ohessa!

vaeinoe · 2025-01-09T10:40:51Z

src/main/scala/fi/vm/sade/omatsivut/db/impl/SessionRepositoryImpl.scala

-  override def deleteByServiceTicket(ticket: ServiceTicket): Unit = {
+  override def deleteByServiceTicket(ticket: String): Unit = {
    runBlocking(sqlu"""delete from sessions where ticket = ${ticket.value}""", timeout = Duration(10, TimeUnit.SECONDS))
  }


Pitäisikö tässä olla where ticket = ${ticket} tms. jos ticket on nyt pelkkä merkkijono?

Hyvä huomio! Itse asiassa nyt tuon voisi palauttaa takaisin ServiceTicket-tyyppiin. Vaihdoin välissä käyttöön Java-clientin ja siinä kohtaa muuttui ticketin tyypitys, mutta sitten piti kuitenkin palauttaa scala-client käyttöön.

vaeinoe · 2025-01-09T12:03:50Z

src/main/scala/fi/vm/sade/hakemuseditori/hakemus/ImmutableLegacyApplicationWrapper.scala

-      application.getState.toString,
-      isPostProcessing(application),
-      Option(application.getRequiredPaymentState).map(_.toString)
+      Application.State.SUBMITTED.toString // TODO fix voiko olla kovakoodattu vai tarvitaanko päättely


Yleishuomiona: TODOja on vielä aika paljon - oli varmaan tarkoituskin, mutta muistutuksena että kannattanee grepata ja katsoa läpi tarkkaan ennen lopullisia testejä ja tuotantoonvientiä. 😊

vaeinoe · 2025-01-09T12:13:53Z

src/main/scala/fi/vm/sade/omatsivut/servlet/ApplicationsServlet.scala

@@ -73,13 +57,15 @@ trait ApplicationsServletContainer {
      val oid = personOid()
      hakemusEditori.fetchByPersonOid(request, oid, Fetch) match {
        case FullSuccess(hakemukset) =>
+          logger.info("ApplicationsServlet get FullSuccess")


Jos nämä ovat jäämässä tuotantoon saakka, pitäisikö näissä lokittaa myös jotain tietoa siitä mitä koetettiin hakea (henkilön oid)?

Ei ollut tarkoitus jättää ja tuossa siis haettiin hakemuksia henkilö-oidilla.

vaeinoe · 2025-01-09T12:53:37Z

src/main/scala/fi/vm/sade/omatsivut/vastaanotto/VastaanottoService.scala

+        .withKayttooikeusRajoitukset(ViestinvalitysBuilder.kayttooikeusrajoituksetBuilder()
+             .withKayttooikeus("APP_HAKEMUS_CRUD", "1.2.246.562.10.240484683010")
+             .build())


Pitäisiköhän tähän lisätä varmuudeksi kommentti jälkipolville mistä taikasanoissa (oikeus ja organisaatio) on kyse?

Hei tästä huomiosta papukaijamerkki! Tuohon itse asiassa oli jäänyt copypaste-härönä ihan väärät oikat!

msiukola · 2025-01-15T14:41:54Z

src/main/scala/fi/vm/sade/hakemuseditori/HakemusEditori.scala

+        // jos ei löydy atarusta, haetaan haku-appista
+        val optFromHakemusRepository = hakemusRepository.getHakemus(request, hakemusOid, valintatulosFetchStrategy)
+        if (optFromHakemusRepository.isEmpty) {
+          logger.info("fetchByHakemusOid(): Hakemus repository returned no application for given hakemusOid {}. Searching from ataru.", hakemusOid)


Pikkujuttu, mutta Searching from atarun voisi siivoilla.

msiukola · 2025-01-15T14:44:37Z

src/main/scala/fi/vm/sade/omatsivut/servlet/KoodistoServlet.scala

  class KoodistoServlet extends OmatSivutServletBase with JacksonJsonSupport with JsonFormats {
    before() {
      contentType = formats("json")
    }

    get("/postitoimipaikka/:postalCode") {
-      checkNotFound(koodistoService.postOffice(params("postalCode"), language))
+      NotFound("error" -> "Not found") // ei pitäisi enää tarvita tätä


Jos ei tosiaan tarvita tätä enää, ehkä voisi siivota järeämminkin

En tainnut alunperin uskaltaa poistaa kun oli viittauksia fronttipuolella, mutta mäkeen vaan.

msiukola · 2025-01-15T14:45:34Z

src/main/scala/fi/vm/sade/omatsivut/servlet/KoulutusServlet.scala

    }

    get("/koulutus/:aoId") {
-      checkNotFound(koulutusInformaatioService.koulutus(params("aoId"), getLangParam("lang")))
+      NotFound("error" -> "Not found")


Siivousta näillekin?

Tähänkin taisi olla jotain viittauksia frontissa jäljellä mutta lienee kuollutta koodia

msiukola · 2025-01-15T14:46:31Z

src/main/scala/fi/vm/sade/omatsivut/servlet/NonSensitiveApplicationServlet.scala

@@ -83,6 +79,7 @@ trait NonSensitiveApplicationServletContainer {
    }

    private def jwtAuthorize: Try[HakemusJWT] = {
+      logger.info(s"tunnistaudutaan jwt:llä")


Haiskahtaa vähän ylimääräiseltä lokitukselta

oli jäänyt testailusta näköjään

msiukola · 2025-01-15T14:47:20Z

src/main/scala/fi/vm/sade/omatsivut/servlet/NonSensitiveApplicationServlet.scala

    private def fetchHakemus(hakemusOid: String, personOid: Option[String]): Try[HakemusInfo] = {
+      logger.info(s"haetaan legacy-hakemus hakemus-oidilla $hakemusOid")


Kai tämä palauttaa muutakin kuin legacy-hakemuksia?

Tuokin oli tarkoitettu tilapäiseksi kun testailin haku-appin hakemuksia

msiukola · 2025-01-15T15:09:50Z

src/main/scala/fi/vm/sade/omatsivut/util/Cache.scala

+
+
+
+


Ovatko nämä käytössä jossain? En nopealla vilkaisulla löytänyt

Totta, ei enää olekaan kun päivitin apit

…en retry

marjakari added 30 commits June 27, 2024 13:55

legacy-hakemuksen osalta riippuvuuden purku haku-app kirjastosta ja d…

fcee592

…ummy-dao

OY-4717 WIP haku-app-riippuvuuksien purku ja tarpeetonta koodia pois

67a1d2e

OY-4717 mock-dataa hakuappille

ea63dfc

OY-4717 turhaa koodia pois

f3c95b9

OY-4717 mockdataa

c02f432

OY-4717 customserializer legacy-hakemuksille

4a545b2

OY-4717 pieniä poistoja

5fd3eb9

OY-4717 koodistointegraatio pois

a15009c

OY-4717 depsupäivityksiä - ei käänny

3fb3424

OY-4717 vanhat depsut, buildaantuu

abfe52c

OY-4717 scala-utils scala-mongo ja java-legacy-cas pois riippuvuuksis…

e69c82f

…ta, kopioitu tarvittavat koodit repoon

OY-4717 scala-captcha -kirjastoriippuvuus pois

4b06acb

OY-4717 scala-properties kirjastoriippuvuus pois

04cee01

OY-4717 github build workflow java-versio

f7519ac

OY-4717 maven compiler java-versio

980c7de

OY-4717 oppijanumerorekisteri-integraatio java-casille

a27adbb

OY-4717 github packages

73366ed

OY-4717 poistettu loputkin scala-utils-viittaukset ja kopioitu tarvit…

07d4765

…tavat koodit repoon, poistettu group-emailer-kirjasto

OY-4717 vanhemman java-version java-properties toistaiseksi

83de1b6

OY-4717 depsusäätöä

669013f

OY-4717 puuttuvia jackson-depsuja jotka aiemmin tulivat scala-kirjast…

53761f8

…oista

OY-4717 scala-cas-clientit korvattu java-cas-clientilla, java 11 käyt…

2072178

…töön. Mockitoa käyttävät testit tilapäisesti kommentoitu koska vaatii versionnoston

OY-4717 java 11 myös build workflowhun

1ffffc5

OY-4717 java 11 pom source&target

e0f8895

OY-4717 java 11 kontin runtimeksi

821f426

OY-4717 lisää lokitusta

1e268aa

OY-4717 poistettu temp kovakoodaus-aivopieru, lisää lokitusta

4068f9b

OY-4717 lisää lokitusta

299de5f

OY-4717 cas-tiketin validointikutsu blocking-metodilla

30205dd

OY-4717 tarkempaa lokitusta

a1c1aa9

marjakari added 12 commits December 31, 2024 14:18

OY-4717 poistettu kantahaun väliaikaiset tynkätoteutukset

6970de9

OY-4717 debug-lokitusta

e8e040c

OY-4717 oppijanumerorekisterin kenttien parsinta fiksattu

11f2a1c

OY-4717 ataru clientin sessionimi

0f17c87

OY-4717 cas client konffit

cff7655

OY-4717 ataru cas client konffi fix

dbbb31f

OY-4717 spring versionnosto, viestinvälityspalveluintegraatio vastaan…

c3f80e2

…ottokuittaukseen, turhia depsuja pois

OY-4717 turhia kirjastoja pois, underscore päivitys, kriittiset haavo…

c97ab7c

…ittuvuudet fiksattu

OY-4717 mongo-kantahaut

95b4acb

OY-4717 embedded mongo kuntoon ja palautettu testejä

de97db9

OY-4717 palautettu testejä, päivitetty readme, jvm 1.8 target pois sc…

9f59239

…ala-pluginista

OY-4717 turhia testejä pois, koodisto-kirjastoriippuvuus pois

627465c

vaeinoe approved these changes Jan 9, 2025

View reviewed changes

marjakari added 5 commits January 10, 2025 09:08

OY-4717 katselmointihuomiot

01c8c39

OY-4717 pakitettu underscore-versio

24924a9

OY-4717 underscore uusin versio

a8dda2a

OY-4717 underscore fiksausyritys

a279cc4

OY-4717 lokitusta jwt-tunnistautuneen asiointiin

68c7046

msiukola reviewed Jan 15, 2025

View reviewed changes

marjakari added 6 commits January 16, 2025 09:21

OY-4717 karsittu koodia lisää ja siivottu kommentteja

73ad953

OY-5044 github actions versiopäivitys

928477c

OY-5039 github ubuntu-latest käyttöön

21c4e9b

OY-4717 hakuapp-hakemukset pois, mongo pois, ylimääräiset testit pois

93dc69b

OY-4717 cas oppijan tunnistukseen lisätty lokitusta ja tunnistautumis…

3793b53

…en retry

OY-4717 lisätty retry myös ataru-hakuun

8562dd5

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

OY-4717 riippuvuuksien megakarsinta ja tietoturvapäivitykset #479

OY-4717 riippuvuuksien megakarsinta ja tietoturvapäivitykset #479

marjakari commented Jan 8, 2025

vaeinoe left a comment

vaeinoe Jan 9, 2025

marjakari Jan 10, 2025

vaeinoe Jan 9, 2025

vaeinoe Jan 9, 2025

marjakari Jan 10, 2025

vaeinoe Jan 9, 2025

marjakari Jan 10, 2025

msiukola Jan 15, 2025

msiukola Jan 15, 2025

marjakari Jan 15, 2025

msiukola Jan 15, 2025

marjakari Jan 15, 2025

msiukola Jan 15, 2025

marjakari Jan 15, 2025

msiukola Jan 15, 2025

marjakari Jan 15, 2025

msiukola Jan 15, 2025

marjakari Jan 15, 2025

		private def fetchHakemus(hakemusOid: String, personOid: Option[String]): Try[HakemusInfo] = {
		logger.info(s"haetaan legacy-hakemus hakemus-oidilla $hakemusOid")

OY-4717 riippuvuuksien megakarsinta ja tietoturvapäivitykset #479

Are you sure you want to change the base?

OY-4717 riippuvuuksien megakarsinta ja tietoturvapäivitykset #479

Conversation

marjakari commented Jan 8, 2025

vaeinoe left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment