Headline
CVE-2023-40585: Merge pull request from GHSA-jwpr-9fwh-m4g7 · metal3-io/ironic-image@f64bb6c
ironic-image is a container image to run OpenStack Ironic as part of Metal³. Prior to version capm3-v1.4.3, if Ironic is not deployed with TLS and it does not have API and Conductor split into separate services, access to the API is not protected by any authentication. Ironic API is also listening in host network. In case the node is not behind a firewall, the API could be accessed by anyone via network without authentication. By default, Ironic API in Metal3 is protected by TLS and basic authentication, so this vulnerability requires operator to configure API without TLS for it to be vulnerable. TLS and authentication however should not be coupled as they are in versions prior to capm3-v1.4.3. A patch exists in versions capm3-v1.4.3 and newer. Some workarounds are available. Either configure TLS for Ironic API (deploy.sh -t ..., IRONIC_TLS_SETUP=true) or split Ironic API and Conductor via configuration change (old implementation, not recommended). With both workarounds, services are configured with httpd front-end, which has proper authentication configuration in place.
Expand Up @@ -8,6 +8,9 @@ export IRONIC_EXPOSE_JSON_RPC=${IRONIC_EXPOSE_JSON_RPC:-false} # shellcheck disable=SC1091 . /bin/configure-ironic.sh
export IRONIC_HTPASSWD=${IRONIC_HTPASSWD:-} export IRONIC_REVERSE_PROXY_SETUP=${IRONIC_REVERSE_PROXY_SETUP:-false}
Ramdisk logs mkdir -p /shared/log/ironic/deploy
Expand All @@ -20,4 +23,16 @@ if [[ “$IRONIC_TLS_SETUP” == “true” ]] && [[ "${RESTART_CONTAINER_CERTIFICATE_UP done & fi
CONFIG=/etc/ironic/ironic.conf
Configure HTTP basic auth for API server HTPASSWD_FILE=/etc/ironic/htpasswd if [[ -n “${IRONIC_HTPASSWD}” ]]; then printf “%s\n” “${IRONIC_HTPASSWD}” >"${HTPASSWD_FILE}" if [[ “$IRONIC_REVERSE_PROXY_SETUP” == “false” ]]; then crudini --set “$CONFIG” DEFAULT auth_strategy http_basic crudini --set “$CONFIG” DEFAULT http_basic_auth_user_file “${HTPASSWD_FILE}” fi fi
exec /usr/bin/ironic