From d07b7084c429c31fba8277a1d9e7624f812c6e2e Mon Sep 17 00:00:00 2001 From: Wolfgang Bumiller Date: Wed, 27 May 2015 13:16:35 +0200 Subject: [PATCH] spice-proxy format now uses new address format spice-proxy used dns-name before which matches a combination of letters, digits and dots, which happens to include ipv4 addresses but not ipv6 ones. Since 'dns-name' sounds like it's only for names, I'm adding an address format which is either a dns-name or an ip (including ipv6). --- src/PVE/JSONSchema.pm | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm index e1485ff..36a2de9 100644 --- a/src/PVE/JSONSchema.pm +++ b/src/PVE/JSONSchema.pm @@ -271,9 +271,23 @@ sub pve_verify_iface { return $id; } +# general addresses by name or IP +register_format('address', \&pve_verify_address); +sub pve_verify_address { + my ($addr, $noerr) = @_; + + if (!(pve_verify_ip($addr, 1) || + pve_verify_dns_name($addr, 1))) + { + return undef if $noerr; + die "value does not look like a valid address: $addr\n"; + } + return $addr; +} + register_standard_option('spice-proxy', { description => "SPICE proxy server. This can be used by the client to specify the proxy server. All nodes in a cluster runs 'spiceproxy', so it is up to the client to choose one. By default, we return the node where the VM is currently running. As resonable setting is to use same node you use to connect to the API (This is window.location.hostname for the JS GUI).", - type => 'string', format => 'dns-name', + type => 'string', format => 'address', }); register_standard_option('remote-viewer-config', { -- 2.39.2