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

[Bug]: Corsair iCue 4.17.244 requires elevation for install/uninstall but doesn't request it #36230

Open
Clockwork-Muse opened this issue Nov 27, 2021 · 5 comments
Labels
Issue-Bug It either shouldn't be doing this or needs an investigation.

Comments

@Clockwork-Muse
Copy link

Clockwork-Muse commented Nov 27, 2021

Brief description of your issue

Attempting to install Corsair iCue from a non-elevated command prompt fails due to insufficient permissions, and it does not request elevation (unlike, say, git)

Steps to reproduce

winget install --id Corsair.iCUE.4
Found CORSAIR iCUE 4 Software [Corsair.iCUE.4] Version 4.17.244
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://downloads.corsair.com/Files/CUE/iCUESetup_4.17.244_release.msi
  ██████████████████████████████   792 MB /  792 MB
Successfully verified installer hash
Starting package install...
Installer failed with exit code: 1603

Expected behavior

The package requests elevation, if necessary.

Actual behavior

The package fails to install
WinGet-Corsair.iCUE.4.4.17.244-2021-11-26-18-21-47.977.log

Environment

Windows Package Manager v1.1.12986
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19044.1348
Package: Microsoft.DesktopAppInstaller v1.16.12986.0
@Clockwork-Muse Clockwork-Muse added the Issue-Bug It either shouldn't be doing this or needs an investigation. label Nov 27, 2021
@ghost ghost added the Needs-Triage This work item needs to be triaged by a member of the core team. label Nov 27, 2021
@OfficialEsco
Copy link
Contributor

We cannot resolve this at the moment

  1. winget version 1.2+ still does not respect this field (Hopefully its supported in winget version 1.2)
    Add messaging to support package specific elevation requirements winget-cli#1537
    Milestone: v1.2-Client
  2. wingetcreate does not support manifest version 1.1.0 and removes the unknown fields
    Support Windows Package Manager v1.1 schema winget-create#139
  3. YamlCreate does not support manifest version 1.1.0 yet but will once this PR gets merged
    Installer Detection Enhancements #35735

This PR should remain open and should probably get a new tag so we can fix it once those 3 requirements are resolved

@denelon denelon removed the Needs-Triage This work item needs to be triaged by a member of the core team. label Nov 29, 2021
@Trenly Trenly mentioned this issue Nov 29, 2021
5 tasks
@Trenly
Copy link
Contributor

Trenly commented Nov 29, 2021

winget version 1.2+ still does not respect this field (Hopefully its supported in winget version 1.2)

I've added the field and requested a merge on #35735 so that hopefully this package will work once the next winget version is released

@denelon
Copy link
Contributor

denelon commented Jun 9, 2022

I was testing this, and I think I found another bug:

winget-bug

@OfficialEsco
Copy link
Contributor

Did you see that in Windows 10 or Windows 11? I've seen a very similar popup randomly on multiple installers when validating packages on Windows 11, however there you just click Not now and it installs.

@denelon
Copy link
Contributor

denelon commented Jun 9, 2022

It was on Windows 11. We've seen this before in the Sandbox, but we have an idea what might have caused it on the primary OS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Bug It either shouldn't be doing this or needs an investigation.
Projects
None yet
Development

No branches or pull requests

4 participants