Skip to content

SuomiFi cleanup#99

Merged
LauriGofore merged 2 commits intomainfrom
VFD-326-users-api-siivotaan-pois-tuki-suomifi-tunnistautumistoteutukselle
Oct 26, 2023
Merged

SuomiFi cleanup#99
LauriGofore merged 2 commits intomainfrom
VFD-326-users-api-siivotaan-pois-tuki-suomifi-tunnistautumistoteutukselle

Conversation

@LauriGofore
Copy link
Copy Markdown
Contributor

@LauriGofore LauriGofore commented Oct 26, 2023

https://virtualfindev.atlassian.net/jira/software/projects/VFD/boards/3?selectedIssue=VFD-326

Tekemisen puutteessa otin luuttua käteen!

  • SuomiFi security feature ja conffit / viitteet poisteltu
  • Pulumi projektista viitteet ja envit veks
  • docker composesta viite pois
  • Buildattu ja ajettu, ei tunnu rikkovan mitään. Jäiköhän jotain huomioimatta?

e: jaa features puolella näköjään jotain rikspoks testbed authin kanssa
e2: Ei vissiin liitykään tähän, mainissa sama. 404 erroreita heittää profiilin tallennuksessa. Devillä toimii, ilmeisesti jokin muu lokaaliongelma.

Siivotaanko Auth GW:stä pois? Sehän ei tietysti mene mvp-tuotantoon, mutta features puolella on käytössä (missä tosin ei käytellä SuomiFi, eikä toimisi enää muutenkaan tämän users-api siivouksen myötä)

@LauriGofore LauriGofore requested a review from lsipii October 26, 2023 07:06
@lsipii
Copy link
Copy Markdown
Contributor

lsipii commented Oct 26, 2023

e2: Ei vissiin liitykään tähän, mainissa sama. 404 erroreita heittää profiilin tallennuksessa. Devillä toimii, ilmeisesti jokin muu lokaaliongelma.

Olisikohan että testbed-api:ssa ei ole päivitetty data-tuotteiden urleja lokaalia varten?
edit: ei näyttänyt olevan, en saanut 404-erroreita mutta 401 kyllä tuli kun lokaalissa oli ToS-vaatimus päällä ja sille ei tukea features-puolella (sen vois laittaa lokaalista pois päältä [vaikkapa tässä pr:ssä] vaatimuksena TermsOfServiceAgreementRequired = false appsettings.local.json, liveissä tuo yliajautuu pulumin konffeilla oikein.)

Siivotaanko Auth GW:stä pois?

Ei tarvii sieltä siivota, siellä on myös tekkidemo missä se käytössä (joka ei käytä users-apia)

Copy link
Copy Markdown
Contributor

@lsipii lsipii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Näyttäisi ja toimisi siististi!

@LauriGofore
Copy link
Copy Markdown
Contributor Author

tuli kun lokaalissa oli ToS-vaatimus päällä ja sille ei tukea features-puolella (sen vois laittaa lokaalista pois päältä

tämähän se culpritti olikin.

@LauriGofore LauriGofore merged commit 28a1ec7 into main Oct 26, 2023
@LauriGofore LauriGofore deleted the VFD-326-users-api-siivotaan-pois-tuki-suomifi-tunnistautumistoteutukselle branch October 26, 2023 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants