#20096: Wireless configuration may update the WiFi key to router's login
password
-------------------------+-------------------------------------------------
Reporter: gvalkov | Owner: developers
Type: defect | Status: new
Priority: high | Milestone: Chaos Calmer (trunk)
Component: packages | Version: Trunk
Resolution: | Keywords: saved login password LuCI wireless
| key administration
-------------------------+-------------------------------------------------
Comment (by gvalkov):
Hello Jow and anonymous!
AFter following your ideas and doing research, I found two workarounds:
1. Add a dummy hidden password box in template network_netlist.htm
{{{
/usr/lib/lua/luci/view/cbi/network_netlist.htm
modules/luci-base/luasrc/view/cbi/network_netlist.htm
line 62:
+ <input style="display:none" type="password" />
<input style="width:6em"
type="text"<%=attr("name", cbid .. ".newnet")%>
onfocus="document.getElementById('<%=cbid%>_new').checked=true" />
}}}
2. Change the password box to text:
{{{
/usr/lib/lua/luci/model/cbi/admin_network/wifi.lua
modules/luci-mod-admin-full/luasrc/model/cbi/admin_network/wifi.lua
line 800:
- wpakey.password = true
+ wpakey.password = false
}}}
Unfortunately I wasn't able to manipulate HTML directly from wifi.lua, so
adding autocomplete="off" to the password would require more experience.
Alas it's my first day messing with Lua and LuCI and I have a lot to
learn. :) The form is one level above wifi.lua and while it might be
possible to act there, it needs more research.
Jow, do you think workaround 1 is suitable for a commit? As for number 2,
I always wanted to have the WPA key visible by default, so I'm going to
use it on my own builds. ;)
--
Ticket URL: <https://dev.openwrt.org/ticket/20096#comment:3>
OpenWrt <http://openwrt.org>
Opensource Wireless Router Technology
_______________________________________________
openwrt-tickets mailing list
[email protected]
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-tickets
