8000 TH-1446 Lokalisering av meldinger for Keycloak-passordvalidering. by EbiJon · Pull Request #123 · kartverket/matrikkel-keycloak · GitHub
[go: up one dir, main page]

Skip to content

TH-1446 Lokalisering av meldinger for Keycloak-passordvalidering. - #123

Merged
EbiJon merged 1 commit into
mainfrom
TH-1446
Aug 13, 2024
Merged

EbiJon merged 1 commit into
mainfrom
TH-1446

Conversation

@EbiJon
@EbiJon EbiJon commented Aug 9, 2024
Copy link
Copy Markdown
Contributor

No description provided.

@EbiJon
EbiJon requested review from alexasl and ingroa August 9, 2024 08:30
@alexasl alexasl left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Virker dette? For meg så ser det ut som disse keysene skulle hett noe med invalidPassword :-)

Se i keycloak repoet

Det er også noe med at det er noen parametere til disse strenge, f.eks.

invalidPasswordMinSpecialCharsMessage=Invalid password: must contain at least {0} special characters. - der {0} er antallet spesialtegn krevet

Ang språket i meldingenene (men det kommer kanskje fra spec), så tenker jeg at f.eks. 1 små bokstaver burde være én liten bokstav eller {0} små bokstav(er)

Ta alt over med en klype salt, fordi disse tingene har blitt veldig annerledes siden jeg så på Keycloak sist (da var det bl.a. ikke en javascript app for alle disse account greiene)

@EbiJon
EbiJon merged commit c908574 into main Aug 13, 2024
@EbiJon
EbiJon deleted the TH-1446 branch August 13, 2024 07:56
@lislei
lislei commented Aug 13, 2024
Copy link
Copy Markdown
Contributor

Disse property-filene er ganske løst koblet til distribusjonen av keycloak.
Hvordan vet man at dette fremedeles vil virke ved oppdateringer. Kunne man lagt til noen regressjons-tester ?

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

Successfully merging this pull request may close these issues.

3 participants

0