minor bug in Linux vm_config.sh?

[TonyWildish-BH]

Describe the bug
By inspection, there appears to be a minor bug in templates/workspace_services/guacamole/user_resources/guacamole-azure-linuxvm/terraform/vm_config.sh.

The end of the script has a "cleanup" which reboots the machine, but the script has code after that to prevent the screen timeout. That code is never going to be executed.

[...]
## Cleanup
echo "init_vm.sh: Cleanup"
sudo shutdown -r now

# Prevent screen timeout 
echo "init_vm.sh: Preventing Timeout"
sudo touch /home/"${VM_USER}"/.config/xfce4/xfconf/xfce-perchannel-xml/xfce4-screensaver.xml
sudo chmod 664 /home/"${VM_USER}"/.config/xfce4/xfconf/xfce-perchannel-xml/xfce4-screensaver.xml
sudo chown "${VM_USER}":"${VM_USER}" /home/"${VM_USER}"/.config/xfce4/xfconf/xfce-perchannel-xml/xfce4-screensaver.xml
sudo tee /home/"${VM_USER}"/.config/xfce4/xfconf/xfce-perchannel-xml/xfce4-screensaver.xml << END
<?xml version="1.0" encoding="UTF-8"?>
<channel name="xfce4-screensaver" version="1.0">
  <property name="saver" type="empty">
    <property name="mode" type="int" value="0"/>
    <property name="enabled" type="bool" value="false"/>
  </property>
  <property name="lock" type="empty">
    <property name="enabled" type="bool" value="false"/>
  </property>
</ channel>
END

[tim-allen-ck]

Hi @TonyWildish-BH that code does still get executed, I've checked the logs. There's a PR open to change how the system prevents the timeout.