Bug 332057

Summary: Remember log in not working for openconnect plugin
Product: [Plasma] plasma-nm Reporter: Philip Keiter <philip.keiter>
Component: appletAssignee: Jan Grulich <jgrulich>
Status: RESOLVED DUPLICATE    
Severity: normal CC: dschridde+kde, dwmw2, jgrulich, lamarque, lukas, philip.keiter, silver.salonen
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Mint (Ubuntu based)   
OS: Linux   
Latest Commit: Version Fixed In:

Description Philip Keiter 2014-03-12 12:07:23 UTC 
When connecting to a cisco vpn, network manager prompts for login and has checkbox to remember it. But it always presents blank fields on next connect.

Reproducible: Always

Steps to Reproduce:
1. Install NetworkManager OpenConnect plugin.
2. Open NetworkManager Settings (edit connections).
3. Add an OpenConnect vpn connection.
4. Enter the Gateway URI and click OK.
5. Log in and check the box to remember.
6. After successful connection, disconnect.
7. Reconnect.
Actual Results:  
Prompted for log in with blank fields.

Expected Results:  
No prompt, or at least prefilled prompt.
Comment 1 Philip Keiter 2014-03-12 14:42:12 UTC 
Oops, there is no checkbox to remember. So I guess this is a feature request not a bug?
Comment 2 Silver Salonen 2014-07-02 17:35:17 UTC 
It's still a bug because before NetworkManager prompts for OpenConnect's login, it requests access to KWallet too - so it seems that by the use-case, this login should be remembered. But after entering KWallet's password, it's just not used anyway.
Comment 3 Silver Salonen 2014-07-02 18:02:51 UTC 
I use:
OpenSUSE 13.1
KDE 4.11.10+4.13.2
NetworkManager 0.9.8.8
NetworkManager-kde4-libs 0.9.0.10
plasmoid-networkmanagement 0.9.0.10
NetworkManager-openconnect 0.9.8.4
NetworkManager-openconnect-kde4 0.9.0.10
Comment 4 Jan Grulich 2014-07-03 11:43:59 UTC 
@Silver Salonen You have the old NM applet, this behaviour could be different in the new plasma-nm applet. 

The Openconnect VPN plugin needs more work than remembering passwords from the password dialog, but unfortunately I haven't had time so far to do something with it. Also the rest of VPN plugins don't store passwords from the password dialog, but at least it's possible to store them in the connection editor.
Comment 5 Silver Salonen 2014-07-03 19:36:22 UTC 
Now I have these installed:
NetworkManager 0.9.8.8
NetworkManager-openconnect 0.9.8.4
NetworkManager-openconnect-lang 0.9.8.4
libNetworkManagerQt0 0.9.8.1
plasma-nm 0.9.3.3
plasma-nm-openconnect 0.9.3.3
typelib-1_0-NetworkManager-1_0 0.9.8.8

And indeed, at least opening KWallet is not requested any more. So now it's really a feature request :)
Comment 6 Silver Salonen 2014-07-07 22:07:26 UTC 
Sorry, just got asked for KWallet password when starting openconnect connection.
Comment 7 Lamarque V. Souza 2014-08-02 17:41:26 UTC 
(In reply to Jan Grulich from comment #4)
> @Silver Salonen You have the old NM applet, this behaviour could be
> different in the new plasma-nm applet. 
> 
> The Openconnect VPN plugin needs more work than remembering passwords from
> the password dialog, but unfortunately I haven't had time so far to do
> something with it. Also the rest of VPN plugins don't store passwords from
> the password dialog, but at least it's possible to store them in the
> connection editor.

I already tried to implement support for saving secrets from password dialog before. It works in the sense that NetworkManager saves secrets. However, NetworkManager does not save login names or any other configuration, it only saves secrets. I hacked a patch to force NetworkManager to save all network configuration (login names and secrets), but NetworkManager ignores the new login name and fails the connection. If you try activating the connection right after that then it works (no password dialog and connection is activated). We need support from NetworkManager to save other connection settings besides secrets from the password dialog or this will never work as expected.
Comment 8 Dennis Schridde 2014-08-17 11:31:53 UTC 
Is this a duplicate of bug #309931?
Comment 9 Lamarque V. Souza 2014-11-04 12:55:02 UTC 
Yes, it's a duplicate.

*** This bug has been marked as a duplicate of bug 309931 ***