[PATCH pve-firewall v2] api: aliases: make 'cidr' optional when updating

[PATCH pve-firewall v2] api: aliases: make 'cidr' optional when updating

[Arthur Bied-Charreton]

Updating an alias required passing its CIDR, even when only renaming it.
Clients therefore had to fetch the alias first just to send back the
currently configured CIDR.

Make 'cidr' optional in PUT .../firewall/aliases/{name} and keep the
existing CIDR if none is passed.

Signed-off-by: Arthur Bied-Charreton <a.bied-charreton@proxmox.com>
---
changes since v1:
- fix subject line

 src/PVE/API2/Firewall/Aliases.pm | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/PVE/API2/Firewall/Aliases.pm b/src/PVE/API2/Firewall/Aliases.pm
index 4f6960d..9fd1348 100644
--- a/src/PVE/API2/Firewall/Aliases.pm
+++ b/src/PVE/API2/Firewall/Aliases.pm
@@ -215,7 +215,7 @@ sub register_update_alias {
 
     $properties->{name} = $api_properties->{name};
     $properties->{rename} = $api_properties->{rename};
-    $properties->{cidr} = $api_properties->{cidr};
+    $properties->{cidr} = { $api_properties->{cidr}->%*, optional => 1 };
     $properties->{comment} = $api_properties->{comment};
     $properties->{digest} = get_standard_option('pve-config-digest');
 
@@ -251,7 +251,10 @@ sub register_update_alias {
 
                     raise_param_exc({ name => "no such alias" }) if !$aliases->{$name};
 
-                    my $data = { name => $param->{name}, cidr => $param->{cidr} };
+                    my $data = {
+                        name => $param->{name},
+                        cidr => $param->{cidr} // $aliases->{$name}->{cidr},
+                    };
                     $data->{comment} = $param->{comment} if $param->{comment};
 
                     $aliases->{$name} = $data;
-- 
2.47.3