[pve-devel] applied: [PATCH common 1/1] JSONSchema: extend pve-configid regex by '-'

Thomas Lamprecht t.lamprecht at proxmox.com
Sat Apr 25 19:26:39 CEST 2020


On 4/9/20 4:10 PM, Dominik Csapak wrote:
> we use this format for all 'delete' options but we have some options
> that have a '-' in the name (e.g. 'sync-defaults-options') that cannot
> be deleted if it is not included
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  src/PVE/JSONSchema.pm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm
> index 01a3cce..1d28f36 100644
> --- a/src/PVE/JSONSchema.pm
> +++ b/src/PVE/JSONSchema.pm
> @@ -169,7 +169,7 @@ register_format('pve-configid', \&pve_verify_configid);
>  sub pve_verify_configid {
>      my ($id, $noerr) = @_;
>  
> -    if ($id !~ m/^[a-z][a-z0-9_]+$/i) {
> +    if ($id !~ m/^[a-z][a-z0-9_-]+$/i) {
>  	return undef if $noerr;
>  	die "invalid configuration ID '$id'\n";
>      }
> 

applied, thanks!




More information about the pve-devel mailing list