Comment 1 for bug 1619044

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.config (master)

Reviewed: https://review.openstack.org/358045
Committed: https://git.openstack.org/cgit/openstack/oslo.config/commit/?id=babff882c0794fab81665744bcc1aa155b90eed2
Submitter: Jenkins
Branch: master

commit babff882c0794fab81665744bcc1aa155b90eed2
Author: Dharini Chandrasekar <email address hidden>
Date: Fri Aug 19 18:34:18 2016 +0000

    Fixing HostName and adding support for HostAddress

    When config options in different projects use IPOpt as the opt's
    type, it restricts operators to only IP addresses.
    When the opt is set to HostnameOpt type, currently even an
    incomplete or invalid IP passes as a valid hostname. Also, currently
    HostnameOpt does not make sure that there is a presense of at least one
    non-numeric character in the provided host name.

    According to RFC 1123, (https://tools.ietf.org/html/rfc1123),
    a valid host name can never have the dotted-decimal form #.#.#.#,
    since at least the highest-level component label will be alphabetic.

    This patch fixes the existing Hostname Opt to abide by the stated RFC
    and also adds a new opt type that would enable operators to provide
    either a hostname or an IP and at the same time perform checks on
    both IPOpt type and HostnameOpt type, by setting opt type to
    "HostAddressOpt" type. This would ensure that an invalid IP does
    not pass as a valid hostname and at the same time retains the rules
    required to be followed for the validation of an acceptable hostname.

    Change-Id: I77bdb64b7e6e56ce761d76696bc4448a9bd325eb
    Closes-Bug: #1619044
    Closes-Bug: #1615028