check_site: introduce in_site_or_domain() stub

This does not change any behaviour. It's just a preparatory step to
make things more clear in the following commits.
This commit is contained in:
lemoer 2021-01-06 07:47:07 +01:00
parent 2f8b3af144
commit d571f6054b
12 changed files with 28 additions and 23 deletions

View File

@ -23,6 +23,7 @@ files["package/**/check_site.lua"] = {
read_globals = { read_globals = {
"alternatives", "alternatives",
"extend", "extend",
"in_site_or_domain",
"in_domain", "in_domain",
"in_site", "in_site",
"need", "need",

View File

@ -1,10 +1,10 @@
need_string(in_site({'autoupdater', 'branch'}), false) need_string(in_site({'autoupdater', 'branch'}), false)
need_table({'autoupdater', 'branches'}, function(branch) need_table(in_site_or_domain({'autoupdater', 'branches'}), function(branch)
need_alphanumeric_key(branch) need_alphanumeric_key(branch)
need_string(in_site(extend(branch, {'name'}))) need_string(in_site(extend(branch, {'name'})))
need_string_array_match(extend(branch, {'mirrors'}), '^http://') need_string_array_match(in_site_or_domain(extend(branch, {'mirrors'})), '^http://')
need_number(in_site(extend(branch, {'good_signatures'}))) need_number(in_site(extend(branch, {'good_signatures'})))
need_string_array_match(in_site(extend(branch, {'pubkeys'})), '^%x+$') need_string_array_match(in_site(extend(branch, {'pubkeys'})), '^%x+$')
obsolete(in_site(extend(branch, {'probability'})), 'Use GLUON_PRIORITY in site.mk instead.') obsolete(in_site(extend(branch, {'probability'})), 'Use GLUON_PRIORITY in site.mk instead.')

View File

@ -8,9 +8,9 @@ need_string_match(in_domain({'next_node', 'ip6'}), '^[%x:]+$', false)
for _, config in ipairs({'wifi24', 'wifi5'}) do for _, config in ipairs({'wifi24', 'wifi5'}) do
if need_table({config, 'ap'}, nil, false) then if need_table(in_site_or_domain({config, 'ap'}), nil, false) then
need_boolean({config, 'ap', 'disabled'}, false) need_boolean(in_site_or_domain({config, 'ap', 'disabled'}), false)
if need_boolean({config, 'ap', 'owe_transition_mode'}, false) then if need_boolean(in_site_or_domain({config, 'ap', 'owe_transition_mode'}), false) then
need_string_match(in_domain({config, 'ap', 'ssid'}), '^' .. ('.?'):rep(32) .. '$') need_string_match(in_domain({config, 'ap', 'ssid'}), '^' .. ('.?'):rep(32) .. '$')
need_string_match(in_domain({config, 'ap', 'owe_ssid'}), '^' .. ('.?'):rep(32) .. '$') need_string_match(in_domain({config, 'ap', 'owe_ssid'}), '^' .. ('.?'):rep(32) .. '$')
else else

View File

@ -14,9 +14,9 @@ end
need_string_match(in_domain({'domain_seed'}), '^' .. ('%x'):rep(64) .. '$') need_string_match(in_domain({'domain_seed'}), '^' .. ('%x'):rep(64) .. '$')
need_string({'opkg', 'openwrt'}, false) need_string(in_site_or_domain({'opkg', 'openwrt'}), false)
obsolete({'opkg', 'lede'}, 'Use opkg.openwrt instead.') obsolete({'opkg', 'lede'}, 'Use opkg.openwrt instead.')
need_table({'opkg', 'extra'}, function(extra_repo) need_table(in_site_or_domain({'opkg', 'extra'}), function(extra_repo)
need_alphanumeric_key(extra_repo) need_alphanumeric_key(extra_repo)
need_string(extra_repo) need_string(extra_repo)
end, false) end, false)
@ -24,13 +24,13 @@ end, false)
need_string(in_site({'hostname_prefix'}), false) need_string(in_site({'hostname_prefix'}), false)
need_string(in_site({'timezone'})) need_string(in_site({'timezone'}))
need_string_array({'ntp_servers'}, false) need_string_array(in_site_or_domain({'ntp_servers'}), false)
need_string_match(in_domain({'prefix6'}), '^[%x:]+/64$') need_string_match(in_domain({'prefix6'}), '^[%x:]+/64$')
local supported_rates = {6000, 9000, 12000, 18000, 24000, 36000, 48000, 54000} local supported_rates = {6000, 9000, 12000, 18000, 24000, 36000, 48000, 54000}
for _, config in ipairs({'wifi24', 'wifi5'}) do for _, config in ipairs({'wifi24', 'wifi5'}) do
if need_table({config}, nil, false) then if need_table(in_site_or_domain({config}), nil, false) then
need_string(in_site({'regdom'})) -- regdom is only required when wifi24 or wifi5 is configured need_string(in_site({'regdom'})) -- regdom is only required when wifi24 or wifi5 is configured
need_number({config, 'beacon_interval'}, false) need_number({config, 'beacon_interval'}, false)
@ -62,8 +62,8 @@ end
need_boolean(in_site({'poe_passthrough'}), false) need_boolean(in_site({'poe_passthrough'}), false)
if need_table({'dns'}, nil, false) then if need_table(in_site_or_domain({'dns'}), nil, false) then
need_string_array_match({'dns', 'servers'}, '^[%x:]+$') need_string_array_match(in_site_or_domain({'dns', 'servers'}), '^[%x:]+$')
end end
need_string_array(in_domain({'next_node', 'name'}), false) need_string_array(in_domain({'next_node', 'name'}), false)

View File

@ -138,6 +138,10 @@ function M.in_domain(path)
return path return path
end end
function M.in_site_or_domain(path)
return path
end
function M.this_domain() function M.this_domain()
return domain_code return domain_code
end end

View File

@ -1 +1 @@
need_boolean({'mesh', 'filter_membership_reports'}, false) need_boolean(in_site_or_domain({'mesh', 'filter_membership_reports'}), false)

View File

@ -1,3 +1,3 @@
need_string({'syslog', 'ip'}, true) need_string(in_site_or_domain({'syslog', 'ip'}), true)
need_number_range({'syslog', 'port'}, 1, 65535, false) need_number_range(in_site_or_domain({'syslog', 'port'}), 1, 65535, false)
need_one_of({'syslog', 'proto'}, {'tcp', 'udp'}, false) need_one_of(in_site_or_domain({'syslog', 'proto'}), {'tcp', 'udp'}, false)

View File

@ -1,5 +1,5 @@
-- mesh/vxlan is required in single domain setups (this_domain() is nil) -- mesh/vxlan is required in single domain setups (this_domain() is nil)
need_boolean(in_domain({'mesh', 'vxlan'}), not this_domain()) need_boolean(in_domain({'mesh', 'vxlan'}), not this_domain())
need_number({'mesh', 'batman_adv', 'gw_sel_class'}, false) need_number(in_site_or_domain({'mesh', 'batman_adv', 'gw_sel_class'}), false)
need_one_of({'mesh', 'batman_adv', 'routing_algo'}, {'BATMAN_IV', 'BATMAN_V'}) need_one_of(in_site_or_domain({'mesh', 'batman_adv', 'routing_algo'}), {'BATMAN_IV', 'BATMAN_V'})

View File

@ -1,5 +1,5 @@
need_boolean(in_site({'mesh_vpn', 'enabled'}), false) need_boolean(in_site({'mesh_vpn', 'enabled'}), false)
need_number({'mesh_vpn', 'mtu'}) need_number(in_site_or_domain({'mesh_vpn', 'mtu'}))
need_boolean(in_site({'mesh_vpn', 'pubkey_privacy'}), false) need_boolean(in_site({'mesh_vpn', 'pubkey_privacy'}), false)
need_boolean(in_site({'mesh_vpn', 'bandwidth_limit', 'enabled'}), false) need_boolean(in_site({'mesh_vpn', 'bandwidth_limit', 'enabled'}), false)

View File

@ -1,5 +1,5 @@
local fastd_methods = {'salsa2012+gmac', 'salsa2012+umac', 'null+salsa2012+gmac', 'null+salsa2012+umac', 'null'} local fastd_methods = {'salsa2012+gmac', 'salsa2012+umac', 'null+salsa2012+gmac', 'null+salsa2012+umac', 'null'}
need_array_of({'mesh_vpn', 'fastd', 'methods'}, fastd_methods) need_array_of(in_site_or_domain({'mesh_vpn', 'fastd', 'methods'}), fastd_methods)
need_boolean(in_site({'mesh_vpn', 'fastd', 'configurable'}), false) need_boolean(in_site({'mesh_vpn', 'fastd', 'configurable'}), false)
need_one_of(in_site({'mesh_vpn', 'fastd', 'syslog_level'}), need_one_of(in_site({'mesh_vpn', 'fastd', 'syslog_level'}),
@ -20,4 +20,4 @@ local function check_group(k)
need_table(extend(k, {'groups'}), check_group, false) need_table(extend(k, {'groups'}), check_group, false)
end end
need_table({'mesh_vpn', 'fastd', 'groups'}, check_group) need_table(in_site_or_domain({'mesh_vpn', 'fastd', 'groups'}), check_group)

View File

@ -1,2 +1,2 @@
need_boolean({'wifi', 'mesh', 'sae'}, false) need_boolean(in_site_or_domain({'wifi', 'mesh', 'sae'}), false)
need_string({'wifi', 'mesh', 'sae_passphrase'}, false) need_string(in_site_or_domain({'wifi', 'mesh', 'sae_passphrase'}), false)

View File

@ -1 +1 @@
need_number({'radv_filterd', 'threshold'}, false) need_number(in_site_or_domain({'radv_filterd', 'threshold'}), false)