Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NetFx PackageGroups should use WixVariables for DownloadUrls #6462

Closed
barnson opened this issue May 25, 2021 · 6 comments
Closed

NetFx PackageGroups should use WixVariables for DownloadUrls #6462

barnson opened this issue May 25, 2021 · 6 comments
Labels

Comments

@barnson
Copy link
Member

barnson commented May 25, 2021

  • Describe the scenario and benefits that the feature supports.

Today, PackageGroups in WixNetFxExtension use preprocessor variables for DownloadUrls. They should use WixVariables instead, to make it possible to reuse WixNetFxExtension directly to host the downloads at different URLs (i.e., a CDN/S3/Azure blobs) to avoid Microsoft changing the payloads.

  • Describe how you're accomplishing the feature today (if possible).

Copying/pasting WixNetFxExtension authoring.

  • Describe what you'd like the new feature to do.
<WixVariable Id="NetFx48RedistLink" Value="https://go.microsoft.com/fwlink/?linkid=2088631" Overridable="yes" />
...
<ExePackagePayload Name="!(wix.NetFx48RedistPackageDirectory)ndp48-x86-x64-allos-enu.exe" DownloadUrl="!(wix.NetFx48RedistLink)" ...
@barnson barnson added the triage label May 25, 2021
@rseanhall
Copy link
Contributor

I assume this is an alternative proposal to #6447 for addressing #6438. This would seem to require adding a new PackageGroup every time there's a new package because the hash would change. How would this proposal work with the latest 4.8 update? The URL and the version stayed the same, the signing certificate stayed the same, but the hash changed.

@robmen
Copy link
Member

robmen commented May 26, 2021

This seems like a useful standalone feature. It would make it much easier to self-host the .NETFX download and not have to duplicate all of the installation logic.

@rseanhall
Copy link
Contributor

This feature would not work if NetFx only exposes one Net48Redist package with hash verification. We would have to update the hash every time Microsoft uploads a new package, which then would not match the package at the user's URL.

@barnson
Copy link
Member Author

barnson commented May 26, 2021

It's both.

@BMurri
Copy link

BMurri commented May 26, 2021

This feature would not work if NetFx only exposes one Net48Redist package with hash verification. We would have to update the hash every time Microsoft uploads a new package, which then would not match the package at the user's URL.

As I see what is being attempted here, the default case of using the extension is that the package is hosted somewhere other than Microsoft downloads, and the extension can then be revised every time MSFT touches their package, with a new URL (and hash or certificate, as applicable and if needed).

Since the variable is overridable, we would need to also expose the hash (or whatever) as a WixVar in the same manner, since then the author would be specifying the package to use.

@barnson barnson added this to the v4.0 milestone May 27, 2021
@barnson
Copy link
Member Author

barnson commented Oct 21, 2021

Losing hashes makes this pretty low-value.

@barnson barnson closed this as completed Oct 21, 2021
@rseanhall rseanhall removed this from the v4.0 milestone Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants