Password with SN attribute value is accepted. This is the incorrect behavior. PasswordCheckSyntax attribute is expected to reject passwords contain SN, CN and UID attribute values.
Metadata Update from @sramling: - Custom field component adjusted to CI test - suites - Custom field origin adjusted to QE - Custom field reviewstatus adjusted to review - Custom field rhbz adjusted to https://bugzilla.redhat.com/show_bug.cgi?id=1465600 - Custom field type adjusted to task - Custom field version adjusted to 1.3.6
Metadata Update from @sramling: - Issue assigned to sramling
Adding pytest script
<img alt="0001-Password-accepts-sn-attr.patch" src="/389-ds-base/issue/raw/files/38ccf56b0efaa7f9f38bdaa532e036f7f3fd92acb6e3e7a34eb09f7ca07b870e-0001-Password-accepts-sn-attr.patch" />
The python part seems okay. One thing only, could you please change the link to https://pagure.io/389-ds-base/issue/49342 ?
Besides that, LGTM. I'd wait for developers to check it because they may have an additional view on performed steps.
Adding new patch
<img alt="0001-Password-accepts-sn-attr.patch" src="/389-ds-base/issue/raw/files/c1c52646b4c79a1ffc1f16acd1a18ee43bbb863c9f8ca18433aa118866d654bd-0001-Password-accepts-sn-attr.patch" />
Metadata Update from @mreynolds: - Issue set to the milestone: 1.3.7.0
Metadata Update from @mreynolds: - Issue close_status updated to: fixed - Issue status updated to: Closed (was: Open)
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/2401
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: fixed)
Login to comment on this ticket.