Bug 340707 - [RFE] Wireless Security add "Ask for this password every time"
Summary: [RFE] Wireless Security add "Ask for this password every time"
Status: RESOLVED FIXED
Alias: None
Product: plasma-nm
Classification: Plasma
Component: editor (show other bugs)
Version: 0.9.3.4
Platform: Kubuntu Linux
: NOR wishlist
Target Milestone: ---
Assignee: Lukáš Tinkl
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-11-07 01:37 UTC by zigi
Modified: 2015-11-03 12:41 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
ask, store password (38.40 KB, image/png)
2014-11-07 10:30 UTC, zigi
Details

Note You need to log in before you can comment on or make changes to this bug.
Description zigi 2014-11-07 01:37:53 UTC
Hi,
   please add "Ask for this password every time" functionality to these Security types:

1. LEAP
2. Dynamic WEP (802.1x)
  - TLS
  - LEAP
3. WPA & WPA2 Personal
4. WPA & WPA2 Enterprise
  - TLS
  - LEAP

Thank you

Reproducible: Always
Comment 1 Jan Grulich 2014-11-07 09:10:53 UTC
Does it make sense to have this option? For those types of wireless security is the password still same, right? We have it in WPA & WPA2 Enterprise because this password could be always different.
Comment 2 zigi 2014-11-07 09:23:28 UTC
Yes. You still forget on sysadmins. A regular user does not apply it, but for administrators and implementers of WiFi networks it can be useful.
Comment 3 Jan Grulich 2014-11-07 09:53:18 UTC
I believe that could be useful for administrators. What about having global checkbox instead, something like "Do not store secrets" or something like that, probably on the "General configuration" tab?
Comment 4 zigi 2014-11-07 10:04:13 UTC
It depends on the Connection. For me, keep it for the concrete connection near by the user/password/private key fields. Don't move it to some General tab for all types of connections.
Comment 5 Jan Grulich 2014-11-07 10:13:34 UTC
What could be even better and I would really like to have is the same password field as nm-connection-editor has, just with an option to not store the secrets. It's basically a text field with a small icon on the right side, where you can switch whether you want to store this password only for you or for all users. Personally, I don't like idea having two checkboxes below each password field asking for showing password and not storing it.
Comment 6 zigi 2014-11-07 10:30:37 UTC
Created attachment 89492 [details]
ask, store password

(In reply to Jan Grulich from comment #5)
> What could be even better and I would really like to have is the same
> password field as nm-connection-editor has, just with an option to not store

I don't know exactly what you mean. What is nm-connection-editor - GTK manager?

> the secrets. It's basically a text field with a small icon on the right
> side, where you can switch whether you want to store this password only for
> you or for all users.

Yes, I can imagine this. But more choices will be needed.

> Personally, I don't like idea having two checkboxes
> below each password field asking for showing password and not storing it.

See attached screenshot of OpenVPN plugin - nice, clean and usable solution.
Comment 7 Jan Grulich 2015-11-03 12:41:17 UTC
Git commit 5b8db6cf2363906e25b4e033100b8da7ac3a9aa4 by Jan Grulich.
Committed on 03/11/2015 at 12:40.
Pushed by grulich into branch 'master'.

Add option to select password storage to all password fields

TODO: Make this option accessible using keyboard
REVIEW:125723

M  +7    -13   libs/editor/connectiondetaileditor.cpp
M  +1    -3    libs/editor/settings/bondwidget.cpp
M  +1    -1    libs/editor/settings/bondwidget.h
M  +1    -3    libs/editor/settings/bridgewidget.cpp
M  +1    -1    libs/editor/settings/bridgewidget.h
M  +1    -3    libs/editor/settings/btwidget.cpp
M  +1    -1    libs/editor/settings/btwidget.h
M  +36   -29   libs/editor/settings/cdmawidget.cpp
M  +3    -6    libs/editor/settings/cdmawidget.h
M  +42   -44   libs/editor/settings/gsmwidget.cpp
M  +3    -7    libs/editor/settings/gsmwidget.h
M  +1    -3    libs/editor/settings/infinibandwidget.cpp
M  +1    -1    libs/editor/settings/infinibandwidget.h
M  +1    -3    libs/editor/settings/ipv4widget.cpp
M  +1    -1    libs/editor/settings/ipv4widget.h
M  +1    -3    libs/editor/settings/ipv6widget.cpp
M  +1    -1    libs/editor/settings/ipv6widget.h
M  +46   -12   libs/editor/settings/pppoewidget.cpp
M  +2    -2    libs/editor/settings/pppoewidget.h
M  +1    -3    libs/editor/settings/pppwidget.cpp
M  +2    -2    libs/editor/settings/pppwidget.h
M  +108  -41   libs/editor/settings/security802-1x.cpp
M  +2    -2    libs/editor/settings/security802-1x.h
M  +1    -3    libs/editor/settings/teamwidget.cpp
M  +1    -1    libs/editor/settings/teamwidget.h
M  +0    -96   libs/editor/settings/ui/802-1x.ui
M  +6    -25   libs/editor/settings/ui/cdma.ui
M  +9    -44   libs/editor/settings/ui/gsm.ui
M  +1    -3    libs/editor/settings/vlanwidget.cpp
M  +1    -1    libs/editor/settings/vlanwidget.h
M  +1    -3    libs/editor/settings/wificonnectionwidget.cpp
M  +1    -1    libs/editor/settings/wificonnectionwidget.h
M  +80   -30   libs/editor/settings/wifisecurity.cpp
M  +6    -6    libs/editor/settings/wifisecurity.h
M  +1    -3    libs/editor/settings/wimaxwidget.cpp
M  +1    -1    libs/editor/settings/wimaxwidget.h
M  +1    -3    libs/editor/settings/wiredconnectionwidget.cpp
M  +1    -1    libs/editor/settings/wiredconnectionwidget.h
M  +4    -3    libs/editor/settings/wiredsecurity.cpp
M  +1    -1    libs/editor/settings/wiredsecurity.h
M  +100  -9    libs/editor/widgets/passwordfield.cpp
M  +20   -1    libs/editor/widgets/passwordfield.h
M  +2    -1    libs/editor/widgets/settingwidget.h
M  +1    -21   vpn/l2tp/l2tp.ui
M  +1    -2    vpn/l2tp/l2tpauth.cpp
M  +3    -3    vpn/l2tp/l2tpauth.h
M  +14   -18   vpn/l2tp/l2tpwidget.cpp
M  +1    -1    vpn/l2tp/l2tpwidget.h
M  +1    -2    vpn/openconnect/openconnectauth.cpp
M  +2    -2    vpn/openconnect/openconnectauth.h
M  +1    -2    vpn/openconnect/openconnectwidget.cpp
M  +3    -3    vpn/openconnect/openconnectwidget.h
M  +2    -42   vpn/openswan/openswan.ui
M  +1    -2    vpn/openswan/openswanauth.cpp
M  +3    -3    vpn/openswan/openswanauth.h
M  +29   -59   vpn/openswan/openswanwidget.cpp
M  +3    -7    vpn/openswan/openswanwidget.h
M  +0    -80   vpn/openvpn/openvpn.ui
M  +0    -20   vpn/openvpn/openvpnadvanced.ui
M  +28   -33   vpn/openvpn/openvpnadvancedwidget.cpp
M  +6    -8    vpn/openvpn/openvpnadvancedwidget.h
M  +1    -2    vpn/openvpn/openvpnauth.cpp
M  +3    -3    vpn/openvpn/openvpnauth.h
M  +58   -73   vpn/openvpn/openvpnwidget.cpp
M  +5    -10   vpn/openvpn/openvpnwidget.h
M  +1    -2    vpn/pptp/pptpauth.cpp
M  +3    -3    vpn/pptp/pptpauth.h
M  +0    -20   vpn/pptp/pptpprop.ui
M  +32   -38   vpn/pptp/pptpwidget.cpp
M  +6    -7    vpn/pptp/pptpwidget.h
M  +1    -2    vpn/ssh/sshauth.cpp
M  +2    -2    vpn/ssh/sshauth.h
M  +29   -28   vpn/ssh/sshwidget.cpp
M  +4    -4    vpn/ssh/sshwidget.h
M  +6    -25   vpn/ssh/sshwidget.ui
M  +1    -2    vpn/sstp/sstpauth.cpp
M  +2    -2    vpn/sstp/sstpauth.h
M  +27   -36   vpn/sstp/sstpwidget.cpp
M  +4    -4    vpn/sstp/sstpwidget.h
M  +5    -24   vpn/sstp/sstpwidget.ui
M  +1    -2    vpn/strongswan/strongswanauth.cpp
M  +1    -1    vpn/strongswan/strongswanauth.h
M  +0    -72   vpn/strongswan/strongswanprop.ui
M  +7    -103  vpn/strongswan/strongswanwidget.cpp
M  +4    -10   vpn/strongswan/strongswanwidget.h
M  +0    -40   vpn/vpnc/vpnc.ui
M  +1    -2    vpn/vpnc/vpncauth.cpp
M  +3    -3    vpn/vpnc/vpncauth.h
M  +34   -32   vpn/vpnc/vpncwidget.cpp
M  +3    -3    vpn/vpnc/vpncwidget.h

http://commits.kde.org/plasma-nm/5b8db6cf2363906e25b4e033100b8da7ac3a9aa4