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

Kuntalaisen salasanan parempi validointi #6252

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Gekkio
Copy link
Contributor

@Gekkio Gekkio commented Jan 15, 2025

Screenshot from 2025-01-22 10-41-05

Rikkoo kuntien buildit, koska pitää kuvata salasanavaatimukset. Esimerkki:

    @Bean
    fun espooPasswordSpecification(): PasswordSpecification =
        DefaultPasswordSpecification(
            PasswordConstraints.UNCONSTRAINED.copy(
                minLength = 15,
                minLowers = 1,
                minUppers = 1,
                minDigits = 1,
                minSymbols = 1,
            )
        )

@Gekkio Gekkio added the enhancement Uusi toiminnallisuus tai parannus label Jan 15, 2025
@Gekkio Gekkio force-pushed the citizen-password-validation branch 7 times, most recently from 69dff02 to 7c958c6 Compare January 22, 2025 08:14
@Gekkio Gekkio marked this pull request as ready for review January 22, 2025 08:44
@Wnt Wnt changed the base branch from master to citizen-weak-credentials January 22, 2025 11:36
Wnt
Wnt previously approved these changes Jan 22, 2025
@Wnt Wnt dismissed their stale review January 22, 2025 12:23

vahinkolaukaus

@Gekkio Gekkio force-pushed the citizen-weak-credentials branch from 732eb7a to bf587f6 Compare January 23, 2025 14:20
@Gekkio Gekkio force-pushed the citizen-password-validation branch from 7c958c6 to f54b631 Compare January 23, 2025 14:21
Base automatically changed from citizen-weak-credentials to master January 23, 2025 14:40
@Gekkio Gekkio force-pushed the citizen-password-validation branch 2 times, most recently from fac74b5 to 03344c2 Compare January 27, 2025 06:57
@Gekkio Gekkio added breaking Vaatii toimia päivityksen yhteydessä and removed enhancement Uusi toiminnallisuus tai parannus labels Jan 27, 2025
@Gekkio Gekkio force-pushed the citizen-password-validation branch from 03344c2 to 4afaa9d Compare January 27, 2025 08:58
@Gekkio Gekkio force-pushed the citizen-password-validation branch from 4afaa9d to f562d4d Compare January 27, 2025 09:50
@Gekkio
Copy link
Contributor Author

Gekkio commented Jan 27, 2025

Mob review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Vaatii toimia päivityksen yhteydessä
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants