After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 411039 - Manual connection to WPA2 enterprise network always prompts for password
Manual connection to WPA2 enterprise network always prompts for password
Status: RESOLVED FIXED
Product: NetworkManager
Classification: Platform
Component: nm-applet
0.6.6
Other Linux
: Normal normal
: ---
Assigned To: Lorenzo Colitti
Dan Williams
Depends on:
Blocks:
 
 
Reported: 2007-02-23 00:29 UTC by Lorenzo Colitti
Modified: 2007-02-23 17:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch v1 (1.96 KB, patch)
2007-02-23 01:04 UTC, Lorenzo Colitti
none Details | Review
patch checked in (2.04 KB, patch)
2007-02-23 17:29 UTC, Lorenzo Colitti
none Details | Review

Description Lorenzo Colitti 2007-02-23 00:29:55 UTC
I regularly use a wireless network with WPA2 enterprise security. When I start NM from scratch or come out of sleep, it connects with no problems. But if I try to connect manually, it always asks for a password.

This is because when NM asks nm-applet for a network key, nm-applet looks for the password in the keyring. However, 802.1x passwords are not stored in the keyring, so the lookup fails and nm-applet prompts the user.
Comment 1 Lorenzo Colitti 2007-02-23 01:04:43 UTC
Created attachment 83141 [details] [review]
patch v1

Workaround: if the applet is asked for a key when 802.1x is in use, don't look in the keyring, but just return the security info object from gconf if there is one.

The proper fix would be to store all this information in the keyring, but since 0.6.x is a maintenance branch I assume most of this code is going to go away anyway and a low-impact workaround is preferable.

(Note that I have SVN access, so I can check this in if necessary.)
Comment 2 Lorenzo Colitti 2007-02-23 01:07:57 UTC
P.S. The change to the if statement was made because checking if(new_key) is redundant in the else branch of if(!new_key)
Comment 3 Dan Williams 2007-02-23 15:36:18 UTC
Looks good, feel free to commit and close.
Comment 4 Lorenzo Colitti 2007-02-23 17:29:32 UTC
Created attachment 83178 [details] [review]
patch checked in

Checked in.

This was the patch I actually checked in (the previous one was using escaped_network after it was had been freed, although that was not visible in the diff).