Bug 1061826
| Summary: | RFE: use username:pass in ovirt:// uris (such as ovirt://user:pass@rhev.example.org/) | ||
|---|---|---|---|
| Product: | Red Hat Enterprise Linux 7 | Reporter: | David Jaša <djasa> |
| Component: | virt-viewer | Assignee: | Jonathon Jongsma <jjongsma> |
| Status: | CLOSED ERRATA | QA Contact: | Virtualization Bugs <virt-bugs> |
| Severity: | low | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 7.0 | CC: | berrange, cfergeau, codong, dblechte, djasa, dyuan, jjongsma, marcandre.lureau, mzhan, rbalakri, tzheng |
| Target Milestone: | rc | Keywords: | FutureFeature |
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | virt-viewer-0.6.0-5.el7 | Doc Type: | Enhancement |
| Doc Text: |
Feature: Allow user to pass username through an ovirt:// URI specified on the command line instead of having to type it in the authentication dialog which shows up later
|
Story Points: | --- |
| Clone Of: | Environment: | ||
| Last Closed: | 2015-03-05 13:38:47 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
David Jaša
2014-02-05 17:23:58 UTC
NB, if the username contains a '@' that has to be escaped as '%40' since '@' is a reserved character used as the separator. David, the URI is parsed with '%40' instead of @. Could you verify with a ovirt instance? (In reply to Marc-Andre Lureau from comment #2) > David, the URI is parsed with '%40' instead of @. Could you verify with a > ovirt instance? The uri is indeed parsed but the username is not passed further. Tested on RHEL7: libgovirt-0.1.0-3.el7.x86_64 virt-viewer-0.5.7-7.el7.x86_64 spice-gtk3-0.20-8.el7.x86_64 (In reply to David Jaša from comment #3) > (In reply to Marc-Andre Lureau from comment #2) > > David, the URI is parsed with '%40' instead of @. Could you verify with a > > ovirt instance? > > The uri is indeed parsed but the username is not passed further. Ok, could you change the bug description then, or open a different bug? I guess none of user:password is used atm. thanks Summary updated. Description can not be updated unfortunately... (In reply to David Jaša from comment #5) > Summary updated. Description can not be updated unfortunately... by parsing, I guess you mean "use", afaik parsing is fine > → RFE: user username:pass in ovirt:// uris (such as ovirt://user:pass.org/)
So you want to use password from CLI as well? It's convenient for testing but otherwise it's more of a security risk
(In reply to David Jaša from comment #7) > > → RFE: user username:pass in ovirt:// uris (such as ovirt://user:pass.org/) > > So you want to use password from CLI as well? It's convenient for testing > but otherwise it's more of a security risk sure, but it's already possible to pass that, it's just not being used. So somehow there isn't much we can do about it. Or we can keep ignoring the pass, or error out if it's provided. assigning to Jonathon, moving to post? https://www.redhat.com/archives/virt-tools-list/2014-July/msg00073.html Test with virt-viewer-0.6.0-4.el7.x86_64: Steps: 1. prepare a guest on rhevm(rhevm.example.com), and add user "admin" to virtual machine permissions 2. on client: # wget http://rhevm.example.com/ca.crt # remote-viewer --ovirt-ca-file=ca.crt ovirt://rhevm.example.com/$vm 3. A login dialog comes out with blank for username and password, and input right username and password, will connect the guest. 4. Reconnect the guest with: # remote-viewer --ovirt-ca-file=ca.crt ovirt://admin%40internal.com/$vm Result: Step 4, dialog comes out with blank username and password. VERIFY with virt-viewer-0.6.0-5.el7.x86_64.rpm Steps are same with above. Result: dialog comes out, the username is filed with "admin@internal" and blank for password, and can connect the guest after input right password. As the result, username is passed form cmdline to login dialog. Move to VERIFIED If this Feature should be included in the 7.1 Release Notes, could you please change the Doc Type from Enhancement to "Release Note"? Note that the Release Notes are intended to list the most prominent and customer-relevant new features rather than every single enhancement. Cheers, Jirka No need to include this in the release notes. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://rhn.redhat.com/errata/RHBA-2015-0295.html |