Bug 845236
Summary: | Installer could ask for elevated privileges (UAC) (because of usbclerk) | ||
---|---|---|---|
Product: | Red Hat Enterprise Virtualization Manager | Reporter: | Tomas Jamrisko <tjamrisk> |
Component: | usb-clerk-win | Assignee: | Arnon Gilboa <agilboa> |
Status: | CLOSED WONTFIX | QA Contact: | Desktop QE <desktop-qa-list> |
Severity: | high | Docs Contact: | |
Priority: | high | ||
Version: | 3.1.0 | CC: | acathrow, cfergeau, dblechte, iheim, lyarwood, marcandre.lureau, marcobillpeter, mkrcmari, pvine, Rhev-m-bugs, uril |
Target Milestone: | --- | ||
Target Release: | 3.3.0 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | spice | ||
Fixed In Version: | mingw-usbclerk-0.0.1.1-2.el6_4 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-07-16 15:23:12 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | Spice | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | 984137 | ||
Bug Blocks: |
Description
Tomas Jamrisko
2012-08-02 11:58:35 UTC
The installer is UAC. In theory, the .cab installation should be silent. I wasn't aware of this dialog it has been added recently, I suppose it will be shown only once during first installation (do we want that dialog at every upgrade?). I am not sure how we can ask the user to elevate privileges during installation, and that would perhaps also be annoying for users who might get confused and think they can't do the installation. Even if the user can't give admin privileges, the installation should still complete anyway. Well, so it's reasonable to assume, that the installation can no longer be completely silent (as usb sharing from windows clients seems to be a rather important new feature). Anyway, it seems that virt-viewer installer is using NSIS, isn't it possible to make usbclerk a post-install option and use UAC plugin to gain admin access? (In reply to comment #2) > Well, so it's reasonable to assume, that the installation can no longer be > completely silent (as usb sharing from windows clients seems to be a rather > important new feature). Until now, it was silently not installing USB clerk if run as regular user. I don't think the dialog is the best way to tell the user that information. The question last week was rather if we should ship the USB clerk installer seperately. (I was away last week and didn't participate to the discussion that lead to the current behaviour) > Anyway, it seems that virt-viewer installer is using NSIS, isn't it possible > to make usbclerk a post-install option and use UAC plugin to gain admin > access? As long as the installation can be performed silently by regular user. If you want my honest opinion, I think the cab is really *the wrong way* of doing all of this, and we should go with a seperate _visible_ installer. It seems quite easy to me to replace the current cab mechanism, by an activex version check giving a link to the installer directly in the page. Imho, the situation that happens today is that none of the Spice developper really can make this change happen, and none of the rhevm developper seem to care enough either to make this change happen or even participate to the discussion to improve that situation. The cab is today ~16mb I listed several times *all* the issues the cab system have, and this bugs shows again how limited we are with the current cab approach. That cab should just go away, honestly. (In reply to comment #3) > (In reply to comment #2) > > Well, so it's reasonable to assume, that the installation can no longer be > > completely silent (as usb sharing from windows clients seems to be a rather > > important new feature). > > Until now, it was silently not installing USB clerk if run as regular user. > I don't think the dialog is the best way to tell the user that information. Without usbclerk, usbredir fails. We wanted users to know that. You do have a good point about the silent install being broken by this. (In reply to comment #4) > Without usbclerk, usbredir fails. We wanted users to know that. > You do have a good point about the silent install being broken by this. It would be more reliable to have a hint during the execution of the client, for example, if the usbclerk is not running. mingw-virt-viewer will no longer ship the activex, moving to appropriate component. (mingw-virt-viewer will be provided as admin MSI which makes this bug obsolete there) I realize it's not so much an ActiveX/CAB bug.. so I think we should move this bug to depend on 984137 instead (USB clerk MSI) close as WONTFIX. The usb installation problem will be addressed in machine level usb-clerk MSI installation (984137). And the usb-clerk installation will be removed from the user level installation, so the this bug is irrelevant for 3.3. |