Opened 12 years ago
Closed 11 years ago
#9666 closed enhancement (fixed)
NetworkStatus: indicate if network is protected
Reported by: | dsjonny | Owned by: | mmadia |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Applications/NetworkStatus | Version: | R1/Development |
Keywords: | Cc: | przemub@… | |
Blocked By: | Blocking: | ||
Platform: | All |
Description
It would be fine if the user can see the protection status in the NetworkStatus's WiFi list for example with a "lock" icon. Now in the popup menu we can see only the signal, but in the command line we can find the protection's status/type of the network:
~/Desktop> ifconfig /dev/net/iprowifi4965/0 scan name address signal auth TPLINKHOME d8:5d:4c:86:24:b4 25 WPA2 DR-WiFi cc:5d:4e:47:99:bd 24 WPA2 SZKSZAK_T10Z c8:60:00:67:32:d4 11 WPA2
Using 45487 nightly.
Attachments (3)
Change History (10)
by , 12 years ago
Attachment: | wifilist.png added |
---|
comment:1 by , 12 years ago
Cc: | added |
---|
comment:2 by , 11 years ago
patch: | 0 → 1 |
---|
by , 11 years ago
by , 11 years ago
Attachment: | networkstatus.patch added |
---|
comment:5 by , 11 years ago
Wow, that was fast. Thanks! Just one minor issue for the next time: there are always two blank lines between functions.
I don't have the time to commit it right now, but if no one beats me to it, I'll try to do it this weekend.
comment:6 by , 11 years ago
Owner: | changed from | to
---|---|
Status: | new → in-progress |
I'll apply the patch and do the whitespace changes.
To note, {{{ git am --signoff -i networkstatus.patch }} failed to detect the patch format. So, I did a
git apply networkstatus.patch git commit --author='Przemysław Buczkowski <przemub@przemub.pl>' --signoff
and re-used the original commit message.
comment:7 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | in-progress → closed |
Applied in hrev46341. Thanks for the patch (and the quick review of it)!
Thanks! The patch is not bad, but there are a few issues:
where the AuthenticationName() method would looke like:
}}}