feature/apache-sso #12
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "feature/apache-sso"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Add optional sso authentication via apache and keycloak
@ -0,0 +11,4 @@@initdef initialize(project):Wenn wir ein neues buildfile einbauen, dann muss im root dir noch die referenz drauf gesetzt werden.
Sonst funktioniert das übergreifende versionieren nicht.
@ -0,0 +24,4 @@::apache-sso-crypto-passphrase]:opt-un []))(defn-spec replace-all-matching-prefixes map?So eine funkton sollte eher nach c4k-common ...
Dort dann gerne mit einem Hauch Doku ...
Yess, wo würde ich die Doku am besten anlegen?
@ -0,0 +75,4 @@(->(ns/load-and-adjust-namespace "apache-sso/secret.yaml" name)(replace-all-matching-prefixes "NAME" name)(#(assoc-in % [:stringData :oidc-secret.conf]vielleicht ist das auch eine funktion in c4k-common wert ?
Sowas hat mir auch schon öfter gefehlt.
Mir ist beim Implementieren eben aufgefallen, dass es in common modul die funktion replace-map-value die glaube genau das tut. Hab den test mal um diesen case erweitert
@ -0,0 +64,4 @@(let [{:keys [name]} config](->(ns/load-and-adjust-namespace "apache-sso/deployment.yaml" name)(assoc-in [:metadata :namespace] name)das passiert schon in der zeile drübert :-)
@ -136,1 +145,4 @@(cm/replace-all-matching "BASIC_AUTH_NAME" basic-auth-secret)(cm/replace-all-matching "BASIC_AUTH_USERNAME" (b64/encode basic-auth-user))(cm/replace-all-matching "BASIC_AUTH_PASSWORD" (b64/encode basic-auth-password)))[])))Im einen Fall eine Map und sonst ein [] ??
Gibt es hier einen Test ?
@ -214,0 +234,4 @@name (replace-dots-by-minus unique-name)][(generate-build-secret config auth)(generate-basic-auth-secret config auth)(if (some? ssoauth)if braucht immer das else noch mit dazu, sonst musst du ein when einsetzen. Dürfte so nicht compilieren ....
Schreibst du hier noch einen Test für ?
nice :-)
@ -187,0 +182,4 @@[config websiteconfig?](let [{:keys [unique-name ssoconf]} configname (replace-dots-by-minus unique-name)](if (some? ssoconf) (str name "-sso") name)))let's switch to
(contains? config :ssoconf)- not mutch difference but a clean way to go.@ -206,2 +205,3 @@:burst-rate burst-rate:namespace name}))]))):namespace name}))(when (some? ssoconf)let's switch to
(contains? config :ssoconf)@ -214,0 +213,4 @@(let [{:keys [unique-name ssoconf]} config{:keys [ssoauth]} authname (replace-dots-by-minus unique-name)][(generate-build-secret config auth)If we've (when ...) statements in a sequence, when might lead to a nil.
Means the sequence might look like
[{the build secret}, nil, ...].concat-vec has a filter inside, so the nil elements will vanish ...
@ -214,0 +214,4 @@{:keys [ssoauth]} authname (replace-dots-by-minus unique-name)][(generate-build-secret config auth)(when (do-basic-auth auth)let's switch to
(contains? config :....)@ -214,0 +216,4 @@[(generate-build-secret config auth)(when (do-basic-auth auth)(generate-basic-auth-secret config auth))(when (some? ssoauth)let's switch to
(contains? config :....)- lets use config as the leading info point.WIP:feature/apache-ssoto feature/apache-sso