[Commotion-admin] [luci-commotion] added validation checks on the back end of the name input value as well ... (#152)
Seamus Tuohy
notifications at github.com
Tue Jan 21 16:16:11 UTC 2014
...as for the network value in case a user has no-script turned on and therefore bypasses the front-end checks.
This pull request addresses issue https://github.com/opentechinstitute/luci-commotion/issues/144
To test.
1) On an existing node try to use the following formats for a mesh name with noscript turned off and then again with it turned on.
* 0 chars
* more than 15 chars
* a name containing the following chars $\"[]?+/
* a name starting with a space, !, or # char.
If it ever sets an incorrect name reject this pull request and I will fix it. Otherwise MERGE ME!
You can merge this Pull Request by running:
git pull https://github.com/opentechinstitute/luci-commotion Issue-144
Or you can view, comment on it, or merge it online at:
https://github.com/opentechinstitute/luci-commotion/pull/152
-- Commit Summary --
* added validation checks on the back end of the name input value as well as for the network value in case a user has no-script turned on and therefore bypasses the front-end checks.
-- File Changes --
M luasrc/model/cbi/commotion/basic_mn.lua (26)
-- Patch Links --
https://github.com/opentechinstitute/luci-commotion/pull/152.patch
https://github.com/opentechinstitute/luci-commotion/pull/152.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/opentechinstitute/luci-commotion/pull/152
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.chambana.net/pipermail/commotion-admin/attachments/20140121/1b897242/attachment-0001.html>
More information about the Commotion-admin
mailing list