Re: Parametrization minimum password lenght

On Thu, 19 Dec 2024 at 09:57, Nathan Bossart <nathandbossart(at)gmail(dot)com> wrote:
> On Thu, Dec 19, 2024 at 09:36:17AM -0600, Nathan Bossart wrote:
>> On Thu, Dec 19, 2024 at 07:25:30AM +0000, Bertrand Drouvot wrote:
>>> - errmsg("password is too short")));
>>> + errmsg("password is too short: %d (< %d)", pwdlen, min_password_length)));
>>
>> I think we should explicitly point to the parameter and note its current
>> value.
>
> Like so.

It seems that the v11 cannot apply on master since 7f97b4734, and rebase is needed.

$ git am ~/v11-0001-Add-passwordcheck.min_password_length.patch
Applying: Add passwordcheck.min_password_length.
error: patch failed: contrib/passwordcheck/passwordcheck.c:26
error: contrib/passwordcheck/passwordcheck.c: patch does not apply
Patch failed at 0001 Add passwordcheck.min_password_length.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

The comment of prev_check_password_hook has been changed which case the apply failed.

--
Regrads,
Japin Li