Skip to content

[15.0][FIX] password_security: constraint of password_estimate range#491

Merged
OCA-git-bot merged 1 commit intoOCA:15.0from
onesteinbv:15_fix_password_security1
Jul 2, 2023
Merged

[15.0][FIX] password_security: constraint of password_estimate range#491
OCA-git-bot merged 1 commit intoOCA:15.0from
onesteinbv:15_fix_password_security1

Conversation

@astirpe
Copy link
Copy Markdown
Member

@astirpe astirpe commented Mar 3, 2023

This fixes the computation of constraint of password_estimate range. Without this fix, the constraint never raises an error.
Backported from #482

@astirpe astirpe marked this pull request as ready for review March 3, 2023 08:32
@astirpe astirpe mentioned this pull request Mar 3, 2023
7 tasks
Copy link
Copy Markdown
Member

@amh-mw amh-mw left a comment

Choose a reason for hiding this comment

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

LGTM @OCA/tools-maintainers

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 2, 2023

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 2, 2023
@thomaspaulb
Copy link
Copy Markdown

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 15.0-ocabot-merge-pr-491-by-thomaspaulb-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 5393617 into OCA:15.0 Jul 2, 2023
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 86ec3fb. Thanks a lot for contributing to OCA. ❤️

SiesslPhillip pushed a commit to grueneerde/OCA-server-auth that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-auth (14.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved merged 🎉 stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants