Load secrets from files

[mdemare]

• Added support for ALLOWED_CLIENT_IDS
• Review feedback
• Support for loading secrets from files specified by env vars.

[joodie]

Dit is rommellig. Je bent eerst aan het valideren, dan dingen ongedaan aan het maken/parsen, en dan weer opnieuw valideren. Lees die secrets eerst, en valideer daarna:

(defn- load-secret-from-file
  [env k]
  (let [file-key (keyword (str (name k) "-file"))
        file-name (env file-key)]
    (if file-name
      (assoc env k (str/trim (slurp file-name)))
      env)))

(defn- env-with-secrets
  "Environment map, with additional secrets loaded from files.

  If a key \"K\" in `opt-specs` is not present, and a key \"K-file\"
  is present, load the secret from that file and put it in the env map
  under K."
  []
  (->> opt-specs
       keys
       (reduce (fn [env k]
                 (if (contains? env k)
                   env
                   (load-secret-from-file env k)))
               env)))

en dan verderop in main:

  (let [[config errs] (envopts/opts (env-with-secrets) opt-specs)]  

[joodie]

graag aanpassen

[mdemare]

In jouw voorstel gooi je de validatie weg, dwz ofwel SURF_CONEXT_CLIENT_ID of SURF_CONEXT_CLIENT_ID_FILE is verplicht, en als je allebei weglaat moet het proces exiten met een error message. Heb de config code nu verplaatst naar een eigen namespace. Dit is nu dezelfde structuur als in de edumapper, dus als het hier rommelig is, dan daar ook. Volgende week maar even naar kijken.

In principe kan Remco Jak testen met de b1070c6 commit, heb de laatste commit nog niet goed kunnen testen.