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

Package Request: MSI Afterburner #1990

Closed
Neikon opened this issue Jun 19, 2020 · 11 comments
Closed

Package Request: MSI Afterburner #1990

Neikon opened this issue Jun 19, 2020 · 11 comments
Labels
Help-Wanted This is a good candidate work item from the community.
Milestone

Comments

@Neikon
Copy link

Neikon commented Jun 19, 2020

#Package Requested
MSI Afterburner a tool for OC and monitoring PC stats

Link: https://es.msi.com/page/afterburner

@Neikon Neikon added the Help-Wanted This is a good candidate work item from the community. label Jun 19, 2020
@ghost ghost added the Needs: Triage label Jun 19, 2020
@jedieaston
Copy link
Contributor

This is waiting on microsoft/winget-cli#140, I think. Since Afterburner is distributed as a exe in a zip file, winget can't install it yet.

@denelon denelon added the Blocking-Issue Manifest validation is blocked by a known issue. label Jul 16, 2020
@ghost ghost added the No-Recent-Activity No activity has occurred on this work item for seven days. label Jan 6, 2021
@ghost
Copy link

ghost commented Jan 6, 2021

This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 4 days. It will be closed if no further activity occurs within 3 days of this comment.

@denelon denelon added the .zip label Apr 1, 2021
@Macleykun
Copy link
Contributor

Small problem, even if .zip support would be added. The link has some hot-linking protection:
https://download.msi.com/uti_exe/vga/MSIAfterburnerSetup.zip?__token__=exp=1645450949~acl=/*~hmac=6085551a15037028645e6481cecadf35351bf7d52fc05be9290f0ffd6c1bb64f

This is one i currently got. But i'm sure it'll expire in a few hours/days. The only way to actually add support is to use a 3rd party and check the hashes from both installers.
I've made a PR #47081 and i've checked that the latest 4.6.4 is indeed the exact same from both sources.

@ghost ghost removed the No-Recent-Activity No activity has occurred on this work item for seven days. label Feb 19, 2022
@soredake
Copy link
Contributor

@Macleykun
Copy link
Contributor

@Macleykun how choco package deals with this url https://github.com/HunterZ/choco/blob/master/msiafterburner/tools/chocolateyInstall.ps1#L2

Heya! Thanks for taking the time to look into this :) <3
I'm not entirely sure if winget allows using powershell commands to formulate the download url. I see that the token is merely the current day which is great news! But i have to ask one of the mods here if it's possible. If so i would love to give this another try!

If ya have any questions ya can always reach me on Discord Macley#6969

@ItzLevvie ItzLevvie mentioned this issue Jul 24, 2022
5 tasks
@soredake
Copy link
Contributor

soredake commented Aug 3, 2022

Example download url from chocolatey install https://download.msi.com/uti_exe/vga/MSIAfterburnerSetup.zip?__token__=exp=1658905109~acl=/*~hmac=c9252b6f24d8a58f87fe3f4561bc2c94627e00efda3b89892bb301b99ff176d0

@Macleykun
Copy link
Contributor

Example download url from chocolatey install https://download.msi.com/uti_exe/vga/MSIAfterburnerSetup.zip?__token__=exp=1658905109~acl=/*~hmac=c9252b6f24d8a58f87fe3f4561bc2c94627e00efda3b89892bb301b99ff176d0

If you visit this link it won't download from msi. There's a better way but i need confirmation from mods and zip support.

guru3d shows here: https://www.guru3d.com/files-details/msi-afterburner-beta-download.html
That we can download it, however the download starts after 5 seconds and i doubt winget would work with that.
Downloading it and looking at the downloaded url shows this:

https://ftp.nluug.nl/pub/games/PC/guru3d/afterburner/[Guru3D.com]-MSIAfterburner.zip

This is what guru3d refers to when downloading so in my eyes it's valid but not static (so you have to change the version and hash each time an upgrade happens).

That's the only way unless you manage to get msi to offer the download in a normal way.

@Macleykun
Copy link
Contributor

@Macleykun how choco package deals with this url https://github.com/HunterZ/choco/blob/master/msiafterburner/tools/chocolateyInstall.ps1#L2

also the way this works is probably not gonna be implemented in winget.

The way they get the token of today is by asking the api (funny this is set public) what the token is if you give the current date.
That token then grants you withoud manual intervention the download link.

Unless winget adds some sort of pre-scripting, we can't use msi directly for this.

Both solutions again would always send latest and need to automatically be updated, otherwise we get an incorrect version + hash.

@OfficialEsco
Copy link
Contributor

Ahh i didn't realize you wanted a reply since you've described the requirements.
If we're going to use the chocolatey method we will have to wait for ps1/script InstallerType support (if it gets added)

+1 for adding the ability to execute a .ps1 or .bat script after extracting a ZIP file. Ideally, this should be a file that may not be included in the original package. This should allow us to work with most types of packages distributed as ZIP files.

-1 for running semi-arbitrary scripts during a privileged install. Security? What was that?

Originally posted by IanDarwin in microsoft/winget-cli#140 (comment)

then proper zip support as you said.

@wingetbot would update this regularly so @wingetbot would need update ARP Entries support, in theory this should only trigger a hash missmatch once per version

So, this is a looooooooong way from being added.

The best way would be for Guru3D to release the installer without zipping it.
Or MSI or Guru3D to maintain it them self in the Microsoft Store repo.

@Macleykun
Copy link
Contributor

Ahh i didn't realize you wanted a reply since you've described the requirements. If we're going to use the chocolatey method we will have to wait for ps1/script InstallerType support (if it gets added)

+1 for adding the ability to execute a .ps1 or .bat script after extracting a ZIP file. Ideally, this should be a file that may not be included in the original package. This should allow us to work with most types of packages distributed as ZIP files.

-1 for running semi-arbitrary scripts during a privileged install. Security? What was that?
Originally posted by IanDarwin in microsoft/winget-cli#140 (comment)

then proper zip support as you said.

@wingetbot would update this regularly so @wingetbot would need update ARP Entries support, in theory this should only trigger a hash missmatch once per version

So, this is a looooooooong way from being added.

The best way would be for Guru3D to release the installer without zipping it. Or MSI or Guru3D to maintain it them self in the Microsoft Store repo.

Ya wanna ask :p ?

@denelon denelon removed Blocking-Issue Manifest validation is blocked by a known issue. .zip labels Feb 22, 2023
@mdanish-kh
Copy link
Contributor

Close with reason: Package added in #97402;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help-Wanted This is a good candidate work item from the community.
Projects
None yet
Development

No branches or pull requests

7 participants