Skip to content

Commit 8bdbf85

Browse files
Zuulopenstack-gerrit
authored andcommitted
Merge "Set ownership of /etc/pki/<console> files for TLS"
2 parents 18d1dc9 + e2853bf commit 8bdbf85

2 files changed

Lines changed: 23 additions & 1 deletion

File tree

lib/nova

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -674,6 +674,22 @@ function configure_console_proxies {
674674
sudo mkdir -p /etc/pki/nova-novnc
675675
deploy_int_CA /etc/pki/nova-novnc/ca-cert.pem
676676
deploy_int_cert /etc/pki/nova-novnc/client-cert.pem /etc/pki/nova-novnc/client-key.pem
677+
# OpenSSL 1.1.0 generates the key file with permissions: 600, by
678+
# default, and the deploy_int* methods use 'sudo cp' to copy the
679+
# files, making them owned by root:root.
680+
# Change ownership of everything under /etc/pki/nova-novnc to
681+
# $STACK_USER:$(id -g ${STACK_USER}) so that $STACK_USER can read
682+
# the key file.
683+
sudo chown -R $STACK_USER:$(id -g ${STACK_USER}) /etc/pki/nova-novnc
684+
# This is needed to enable TLS in the proxy itself, example log:
685+
# WebSocket server settings:
686+
# - Listen on 0.0.0.0:6080
687+
# - Flash security policy server
688+
# - Web server (no directory listings). Web root: /usr/share/novnc
689+
# - SSL/TLS support
690+
# - proxying from 0.0.0.0:6080 to None:None
691+
iniset $conf DEFAULT key "/etc/pki/nova-novnc/client-key.pem"
692+
iniset $conf DEFAULT cert "/etc/pki/nova-novnc/client-cert.pem"
677693
fi
678694
fi
679695

lib/nova_plugins/functions-libvirt

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,9 +155,15 @@ EOF
155155
echo "vnc_tls_x509_verify = 1" | sudo tee -a $QEMU_CONF
156156

157157
sudo mkdir -p /etc/pki/libvirt-vnc
158-
sudo chown libvirt-qemu:libvirt-qemu /etc/pki/libvirt-vnc
159158
deploy_int_CA /etc/pki/libvirt-vnc/ca-cert.pem
160159
deploy_int_cert /etc/pki/libvirt-vnc/server-cert.pem /etc/pki/libvirt-vnc/server-key.pem
160+
# OpenSSL 1.1.0 generates the key file with permissions: 600, by
161+
# default and the deploy_int* methods use 'sudo cp' to copy the
162+
# files, making them owned by root:root.
163+
# Change ownership of everything under /etc/pki/libvirt-vnc to
164+
# libvirt-qemu:libvirt-qemu so that libvirt-qemu can read the key
165+
# file.
166+
sudo chown -R libvirt-qemu:libvirt-qemu /etc/pki/libvirt-vnc
161167
fi
162168
fi
163169

0 commit comments

Comments
 (0)