Bug 464765 - Network Manager OpenVPN gui fields for CA and personal certs are swapped.
Network Manager OpenVPN gui fields for CA and personal certs are swapped.
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: NetworkManager-openvpn (Show other bugs)
9
All Linux
medium Severity medium
: ---
: ---
Assigned To: Dan Williams
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2008-09-30 07:54 EDT by jay dwyer
Modified: 2008-10-01 02:42 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-09-30 08:58:04 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
Patch for correct labeling (1.38 KB, patch)
2008-09-30 08:55 EDT, Christoph Höger
no flags Details | Diff

  None (edit)
Description jay dwyer 2008-09-30 07:54:21 EDT
Description of problem:

New network manager openvpn gui has the CA cert and user cert fields swapped, such that the selector for each and what gets stored are vice-versa for each.


Version-Release number of selected component (if applicable):

NetworkManager-openvpn-0.7.0-15.svn4027.fc9.i386

How reproducible:

update to above version, either open an existing open vpn connection or enter a new one. 

existing one will have the current values swapped ( ie the user cert will be selected in the CA cert field, and vice versa )

adding a new one, when you click to select the CA cert, the file selector actually says in the title 'select your personal cert'


Steps to Reproduce:

see above
  
Actual results:

certificate fields are switched

Expected results:

certificate fields will not be switched. a pay rise might also be nice.

Additional info:

when you switch the selections to be not what the gui labels say, openvpn connections from NetworkManager ARE actually working, which pleases me very much!
Comment 1 Christoph Höger 2008-09-30 08:12:30 EDT
I can confirm that for certificate with password connection type. 

So basically the labels CA_Certificate and User Certificate are swapped. 

Working on it.
Comment 2 Christoph Höger 2008-09-30 08:53:47 EDT
Please apply the following patch to 
/usr/share/gnome-vpn-properties/openvpn/nm-openvpn-dialog.glade

by entering the parent directory and applying the patch via 
'patch -p1 < PATHTOPATCH/switch_certificate_label.patch' 
where PATHTOPATCH is wherever you stored that patch.
Comment 3 Christoph Höger 2008-09-30 08:55:59 EDT
Created attachment 318067 [details]
Patch for correct labeling
Comment 4 Christoph Höger 2008-09-30 08:58:04 EDT
Thank you for your contribution. 
I will close this bug for now - I wont do an update for that task but send the patch upstream. 
Feel free to reopen it, if you find other orthographic mistakes ;)
Comment 5 Dan Williams 2008-09-30 15:55:40 EDT
thanks, fixed in svn4127 upstream and will be backported to F8, F9, and F10
Comment 6 Fedora Update System 2008-09-30 16:29:11 EDT
NetworkManager-openvpn-0.7.0-16.svn4027.fc8 has been submitted as an update for Fedora 8.
http://admin.fedoraproject.org/updates/NetworkManager-openvpn-0.7.0-16.svn4027.fc8
Comment 7 Fedora Update System 2008-09-30 16:30:15 EDT
NetworkManager-openvpn-0.7.0-16.svn4027.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/NetworkManager-openvpn-0.7.0-16.svn4027.fc9
Comment 8 jay dwyer 2008-09-30 19:40:03 EDT
phwoor!

i know it was a simple fix, but i logged this bug before i went to bed, and before i wake up its fixed!

nice work guys!

thanks.
Comment 9 Fedora Update System 2008-10-01 02:41:20 EDT
NetworkManager-openvpn-0.7.0-16.svn4027.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 10 Fedora Update System 2008-10-01 02:41:56 EDT
NetworkManager-openvpn-0.7.0-16.svn4027.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Note You need to log in before you can comment on or make changes to this bug.