Revert "luci-proto-openconnect: fix server url validation"

This reverts commit f1312cadc6675fb29d02425775dd8a2a533d7e7c.

The commit contained unrelated changes.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>

diff --git a/modules/luci-base/htdocs/luci-static/resources/ui.js b/modules/luci-base/htdocs/luci-static/resources/ui.js
index 9433659f7d83c452744f2201535b791eec40be84..51b6c0902de56536aa514d36a1a957138ddf6a68 100644 (file)
--- a/modules/luci-base/htdocs/luci-static/resources/ui.js
+++ b/modules/luci-base/htdocs/luci-static/resources/ui.js
@@ -820,7 +820,7 @@ var UISelect = UIElement.extend(/** @lends LuCI.ui.Select.prototype */ {
                                                'type': this.options.multiple ? 'checkbox' : 'radio',
                                                'class': this.options.multiple ? 'cbi-input-checkbox' : 'cbi-input-radio',
                                                'value': keys[i],
-                                               'checked': ((!i && !this.values.length) || this.values.indexOf(keys[i]) > -1) ? '' : null,
+                                               'checked': (this.values.indexOf(keys[i]) > -1) ? '' : null,
                                                'disabled': this.options.disabled ? '' : null
                                        }),
                                        E('label', { 'for': this.options.id ? 'widget.%s.%d'.format(this.options.id, i) : null }),
@@ -832,9 +832,6 @@ var UISelect = UIElement.extend(/** @lends LuCI.ui.Select.prototype */ {
                                ]));
 
                                frameEl.appendChild(brEl.cloneNode());
-
-                               if (!frameEl.querySelector('> span > input[checked]')
-                                       frameEl.querySelector('> span > input').checked = true;
                        }
                }
 

diff --git a/protocols/luci-proto-openconnect/htdocs/luci-static/resources/protocol/openconnect.js b/protocols/luci-proto-openconnect/htdocs/luci-static/resources/protocol/openconnect.js
index e38afdc201c17eaeb7f90c0b6a5a202bd1e66ce7..91ad65cb342cf5dd230171512334f63567d93c9c 100644 (file)
--- a/protocols/luci-proto-openconnect/htdocs/luci-static/resources/protocol/openconnect.js
+++ b/protocols/luci-proto-openconnect/htdocs/luci-static/resources/protocol/openconnect.js
@@ -2,7 +2,6 @@
 'require rpc';
 'require form';
 'require network';
-'require validation';
 
 var callGetCertificateFiles = rpc.declare({
        object: 'luci.openconnect',
@@ -101,41 +100,7 @@ return network.registerProtocol('openconnect', {
                o.value('pulse', 'Pulse Connect Secure SSL VPN');
 
                o = s.taboption('general', form.Value, 'server', _('VPN Server'));
-               o.validate = function(section_id, value) {
-                       var m = String(value).match(/^(?:(\w+):\/\/|)(?:\[([0-9a-f:.]{2,45})\]|([^\/:]+))(?::([0-9]{1,5}))?(?:\/.*)?$/i);
-
-                       if (!m)
-                               return _('Invalid server URL');
-
-                       if (m[1] != null) {
-                               if (!m[1].match(/^(?:http|https|socks|socks4|socks5)$/i))
-                                       return _('Unsupported protocol');
-                       }
-
-                       if (m[2] != null) {
-                               if (!validation.parseIPv6(m[2]))
-                                       return _('Invalid IPv6 address');
-                       }
-
-                       if (m[3] != null) {
-                               if (!validation.parseIPv4(m[3])) {
-                                       if (!(m[3].length <= 253 &&
-                                             (m[3].match(/^[a-zA-Z0-9_]+$/) != null ||
-                                              (m[3].match(/^[a-zA-Z0-9_][a-zA-Z0-9_\-.]*[a-zA-Z0-9]$/) &&
-                                               m[3].match(/[^0-9.]/)))))
-                                               return _('Invalid hostname or IPv4 address');
-                               }
-                       }
-
-                       if (m[4] != null) {
-                               var p = +m[4];
-
-                               if (p < 0 || p > 65535)
-                                       return _('Invalid port');
-                       }
-
-                       return true;
-               };
+               o.datatype = 'host(0)';
 
                o = s.taboption('general', form.Value, 'port', _('VPN Server port'));
                o.placeholder = '443';
@@ -151,7 +116,7 @@ return network.registerProtocol('openconnect', {
 
                o = s.taboption('general', form.Value, 'password2', _('Password2'));
                o.password = true;
-
+       
                o = s.taboption('general', form.Value, 'proxy', _('Proxy Server'));
                o.optional = true;
 
@@ -195,7 +160,7 @@ return network.registerProtocol('openconnect', {
                o.optional = true;
                o.placeholder = 1406;
                o.datatype = 'range(68, 9200)';
-
+               
                o = s.taboption('advanced', form.Value, 'reconnect_timeout', _('Reconnect Timeout'));
                o.optional = true;
                o.placeholder = 300;