Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

aeskey and jwt Secrets are missing helm.sh/resource-policy: keep

Aperta Adatta ai principianti
#945 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
1/5
Tempo stimato
Meno di un'ora
Idoneità per principianti
88/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
helm, typescript
Ambito
devops

Direzione di ricerca

Confronta .helm/ctrl/templates/secrets/aeskey.yaml e jwt.yaml con postgres.yaml, che ha già l’annotazione keep. Aggiungi la stessa annotazione di metadati a entrambi i Secrets; poi controlla i template Helm renderizzati per confermare che sia presente. Il lavoro è completato quando la disinstallazione della release conserva i Secrets della chiave AES e JWT.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug

The ctrl-postgres Secret in .helm/ctrl/templates/secrets/postgres.yaml carries "helm.sh/resource-policy": keep. The aeskey and jwt Secrets don't, and never have, both were created in f453dec without it.

Helm removes the resources it created on helm uninstall unless they carry that annotation. The Postgres data volume isn't one of those, it comes from the postgresql subchart's StatefulSet volumeClaimTemplate, so Kubernetes keeps it.

So an uninstall takes the AES encryption key away and leaves the database behind. A reinstall generates a fresh key, and everything written under the old one becomes unreadable. Names, email, DOB, mobile, address, next of kin, plus Organisation.elsaToken and Study.redcapToken. The app would start normally, so this surfaces as ciphertext appearing in the UI rather than as an error.

Ordinary deploys aren't affected. The lookup in _helpers.tpl reuses the existing Secret on helm upgrade, so the key survives a normal release. This is specifically about uninstall and reinstall, which is what happens when a deployment gets cleared down or moved between clusters.

Same shape as #701 (Redeploy loses database password), which PR #709 addressed for the postgres password by adding the annotation. The other two secrets were never brought in line.

Fix: add "helm.sh/resource-policy": keep to the metadata annotations in aeskey.yaml and jwt.yaml, matching postgres.yaml.

Lingua principale
TypeScript
Stelle
4
Fork
0
Merge medio
14h 50m
PR unite (30g)
5

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Garvan-Data-Science-Platform/ctrl

Tutte le issue di Garvan-Data-Science-Platform/ctrl

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.