Include apphost_version parameter in download URLs for both GUI and CLI apps
authorvitek-karas <vitek.karas@microsoft.com>
Sun, 15 Mar 2020 00:01:19 +0000 (17:01 -0700)
committervitek-karas <vitek.karas@microsoft.com>
Sun, 15 Mar 2020 00:01:19 +0000 (17:01 -0700)
commitf8976a5121b0728679279308d44af1965a9e5ca8
treeb3f857eb5b5770976b02741892c2f335a96efd05
parente4da10ce70e67ddb77564a415ea6e796f4a189f8
Include apphost_version parameter in download URLs for both GUI and CLI apps

So far the code only included the `&apphost_version=<version>` URL parameter in the URL used by the error dialog in GUI apps. The error written to stderr/trace would not have that parameter in its URL.

This change include this parameter in all cases. This also means that comhost/ijwhost and so on will also include the `apphost_version` parameter in their error codes and it will be the version of the comhost/ijwhost and so on. This is not 100% correct, but for the purposed of the website this should be enough (for now we don't see a reason to differentiate between the various hosting scenarios).

Adds a new test to cover this case.
src/installer/corehost/cli/apphost/apphost.windows.cpp
src/installer/corehost/cli/fxr_resolver.cpp
src/installer/test/HostActivation.Tests/PortableAppActivation.cs