Comment 1 for bug 2035982

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ansible-playbooks (master)

Reviewed: https://review.opendev.org/c/starlingx/ansible-playbooks/+/895206
Committed: https://opendev.org/starlingx/ansible-playbooks/commit/049b9863047892cd67e5fdb93d98ebb2b3d0773c
Submitter: "Zuul (22348)"
Branch: master

commit 049b9863047892cd67e5fdb93d98ebb2b3d0773c
Author: Tae Park <email address hidden>
Date: Thu Sep 14 14:21:07 2023 -0400

    Extra condition added to password validation

    The presence of quotation marks (single or double) causes issues with
    certain commands run within the bootstrap. As such, adding extra
    validation during "Validate admin password" task in bootstrap so that
    allowed password patterns that include single quotes and double qoutes
    are rejected. Error message for the task also indicates as such. In
    particular, this aims to stop issues caused by syntax issues created
    from passwords being handled in plaintext.

    Test Plan:

    PASS: admin passwords that include single or double quote characters
    should be rejected in boostrap
    PASS: error message for "Fail if provided admin password does not meet
    required complexity" task should indicate those characters are not
    allowed.
    PASS: passwords without any quotation marks, and passing all other
    existing rules should pass the bootstrap without any issues

    Partial-bug: 2035982

    Change-Id: I59435417172f661799dce37fee02489bb2229351
    Signed-off-by: Tae Park <email address hidden>