[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