Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bruk av felles APIM #377

Closed
5 tasks done
Tracked by #33 ...
elsand opened this issue Jan 24, 2024 · 2 comments
Closed
5 tasks done
Tracked by #33 ...

Bruk av felles APIM #377

elsand opened this issue Jan 24, 2024 · 2 comments
Assignees

Comments

@elsand
Copy link
Member

elsand commented Jan 24, 2024

Introduksjon

Det er besluttet at alle produkter skal benytte felles APIM i Altinn

Beskrivelse

For å kunne samkjøre API management og onboarding er det bestemt at alle produkter skal bruke en og samme APIM-instans. Dette innbærer at vi fjerner all bruk av egen APIM i IAC-en, og se på hva som kan gjøres for å automatisere integrasjon med felles-APIM for det aktuelle miljøet.

Implementasjon

Hvis det foreligger føringer på arkitektur eller andre implementasjonsvalg, legges de inn her. Her kan også ulike tilnærminger diskuteres.

Prosessen for å integrere innebærer bruk av en APIM-instans som ligger i et annet subscription, og at config pushes til et privat repo i Azure Devops. Det finnes en guide på https://pedia.altinn.cloud/altinn-3/ops/release-and-deploy/api-management/ som beskriver hva som må gjøres. Denne er tilsynelatende selvbetjent, men er ikke spesielt automasjons-vennlig, og vil innebære manuelle trinn som må gjennomføres for hver endring som påvirker swagger for alle mijøer.

Oppgaver

Preview Give feedback
@knuhau
Copy link
Collaborator

knuhau commented Feb 13, 2024

Har hatt en kort gjennomgang med plattform-teamet:

  • Guiden på AltinnPedia er myntet på applikasjoner som allerede er en del av felles APIM. Dvs. at det er en del mer som må gjøres for å få lagt til Dialogporten der som ikke er dokumentert, men det lar seg gjøre.
  • Dagens OpenAPI spesifikasjon for Dialogporten inneholder /api/v1 i url. Dette vil ut i fra dagens felles APIM-oppsett føre til at v1 vil forekomme 2 ganger i URL. Vurdere om vi skal fjerne dette fra vår URL, og la APIM ta seg av det.
  • Felles APIM er i dag kun satt opp for å eksponere API under URLen "plattform.altinn.no", deretter navnet på APIet ("https://plattform.altinn.no/autorisasjon/api/v1"). Annen base-URL vil føre til behov for endringer i felles APIM som går langt utenfor dagens guidelines, og ble sterkt rådet mot. Vi må diskutere om vi skal legge oss inn i eksisterende struktur, forsøke å få gjennom større endringer i felles APIM, eller om det er en annen mulig løsning.

@elsand
Copy link
Member Author

elsand commented Feb 13, 2024

Fra refinement:

  • Siden hele regimet rundt provisjonering av felles APIM skal endres - vi må gjøre tilpasninger/kopieringer av script for Dialogporten. Ikke ta hensyn til vedlikeholdbarhet rundt disse scriptene siden de skal uansett vekk.
  • Vi tar sikte på å unngå å måtte endre noe i vår applikasjon mht URL-er, dette skal kunne håndteres i APIM slik vi har satt ting opp og samtidig la APIM ha "eierskap" til versjoner
  • Vi bruker det samme domenet som alle andre - her skal det "etterhvert" komme api.altinn.no - dette skal vi pushe på for å få fremskyndet.

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

No branches or pull requests

3 participants