This prevents the exporter HTTP port from being omitted from the
firewall rule DisplayName.
I've tested this in the following four scenarios, and web.listen-address
is set correctly for each:
* Nothing set: 0.0.0.0:9182
* LISTEN_PORT=9184: 0.0.0.0:9184
* LISTEN_ADDR=127.0.0.1: 127.0.0.1:9182
* LISTEN_PORT=9184 LISTEN_ADDR=127.0.0.1: 127.0.0.1:9184
Signed-off-by: Ben Reedy <breed808@breed808.com>
This change updates WiX to the latest major upstream version, which is a
prerequisite for ARM64 MSI builds.
Signed-off-by: Ben Reedy <breed808@breed808.com>
Resolves#911 which was introduced by 45e9357a.
This is due to the exporter only using the default port if no LISTEN_ADDR
**and** no LISTEN_PORT is defined.
Signed-off-by: Ben Reedy <breed808@breed808.com>
Explicit setting of listening port in the service definition causes port
setting in configuration file to be ignored.
Exporter already defines a default port (9812) if one is not specified,
so no impact from this change is anticipated.
Signed-off-by: Ben Reedy <breed808@breed808.com>
With afterInstallExecute, the old installation is only removed after the new one is finished, which has led to some users seeing the new version failing to start, leading to the install rolling back. afterInstallInitialize, in contrast, uninstalls before the new installation.
Signed-off-by: Calle Pettersson <carlpett@users.noreply.github.com>
