diff options
| author | Hannes Duerr <h.duerr@proxmox.com> | 2025-02-10 15:19:24 +0100 |
|---|---|---|
| committer | Thomas Lamprecht <t.lamprecht@proxmox.com> | 2025-03-06 10:00:21 +0100 |
| commit | 77671ba327fa0fc6461d70f9e9e64891b58c36ab (patch) | |
| tree | a7828f46243af6fd0df0fe5e679e39ca17ef1673 /src/PVE/Network/SDN.pm | |
| parent | 9829f51a48c6e5e3e48d4705df8a14ea2d114007 (diff) | |
api request helper: use builtin method to check success of requests
Avoid using a regex if we have a builtin function ready to use.
Signed-off-by: Hannes Duerr <h.duerr@proxmox.com>
Tested-by: Stefan Hanreich <s.hanreich@proxmox.com>
[TL: touch up commit message]
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Diffstat (limited to 'src/PVE/Network/SDN.pm')
| -rw-r--r-- | src/PVE/Network/SDN.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/PVE/Network/SDN.pm b/src/PVE/Network/SDN.pm index 0fe7c9f..1186d9e 100644 --- a/src/PVE/Network/SDN.pm +++ b/src/PVE/Network/SDN.pm @@ -273,10 +273,10 @@ sub api_request { $ua->ssl_opts(verify_hostname => 0, SSL_verify_mode => 0x00); my $response = $ua->request($req); - my $code = $response->code; - if ($code !~ /^2(\d+)$/) { + if (!$response->is_success) { my $msg = $response->message || 'unknown'; + my $code = $response->code; die "Invalid response from server: $code $msg\n"; } |