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

Manglende kopiering av key vault refs til app configuration #571

Closed
elsand opened this issue Mar 21, 2024 · 1 comment
Closed

Manglende kopiering av key vault refs til app configuration #571

elsand opened this issue Mar 21, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@elsand
Copy link
Collaborator

elsand commented Mar 21, 2024

Beskrivelse

Det mangler referanser fra miljøets app configuration til keyvaultet

Reproduksjon

Ved (re)deploy av et miljø blir det (korrekt) kopiert over nøkler fra kildekeyvault til det akutelle miljøets keyvault. Imidlertid blir det ikke opprettet referanser til noen av disse fra miljøet App Configuration, som er det som er den eneste kilden for applikasjons-runtimens IConfiguration.

Forventet oppførsel

.azure/infrastructure/main.bicep sørger for å opprette referanser i app configuraiton for alle nøkler som er finnes i miljøets keyvault.

Faktisk oppførsel

Ingen referanser til hemmeligheter som stammer fra kildekeyvault blir oppretttet

Ytterligere informasjon

Det må gjøres en search-replace av "--" til ":" siden det er ulikheter mellom hvordan strukturerte nøkler oppgis i KeyVault og App Configuration.

Sentinel-verdi må oppdateres (unconditionally?) når referanser er synket.

@elsand elsand added the bug Something isn't working label Mar 21, 2024
@elsand elsand moved this to 🔖 Klar for implementering in Dialogporten / Arbeidsflate Mar 21, 2024
@arealmaas arealmaas self-assigned this Apr 3, 2024
@arealmaas arealmaas moved this from 🔖 Klar for implementering to Under arbeid in Dialogporten / Arbeidsflate Apr 3, 2024
@arealmaas arealmaas moved this from Under arbeid to Code Review og PR in Dialogporten / Arbeidsflate Apr 5, 2024
@arealmaas
Copy link
Collaborator

Sentinel-verdi må oppdateres (unconditionally?) når referanser er synket.

Vi burde vel kanskje ha en cloud function som lytter på endringer i app configuration keys og øker Sentinel 🤔 Vi legger til verdier direkte inn i environment keyvault og app configuration andre steder i Bicep her også + at vi vil vel gjerne støtte å kunne endre verdier i app configuration direkte og forårsake refresh av verdier automatisk?

Lytt på endringer i app configuration og compute hash(evt. Sentinel++)

arealmaas added a commit that referenced this issue Apr 9, 2024
Adds references to keyvault from app configuration

Related to #571

TODO: Have to update sentinel key 🤔
@arealmaas arealmaas moved this from Code Review og PR to Testing / review in Dialogporten / Arbeidsflate Apr 10, 2024
@elsand elsand closed this as completed Apr 29, 2024
@github-project-automation github-project-automation bot moved this from Testing / review to Ferdig in Dialogporten / Arbeidsflate Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

2 participants