[pve-devel] [PATCH manager] pvesh: decode streamed responses properly

Christoph Heiss c.heiss at proxmox.com
Tue Mar 28 15:01:39 CEST 2023


This allows to use `pvesh` on endpoints like /nodes/{node}/journal,
which return streamed (and possibly gzip'd) responses.

Currently, e.g. `pvesh get /nodes/localhost/journal --lastentries 10`
fails with:

  gzip: stdout: Broken pipe
  got hash object, but result schema specified array!

(.. and similar messages for other output formats.)

This is due the API call returning a "download" object, which contains
(among some other things) a file handle to read the response from.

With this patch, the response from such endpoints is now correctly
read/displayed. Only handles combinations of `Content-Encoding` ==
'gzip' and either 'text/plain' or 'application/json' as `Content-Type`.
As far as I could see (aka. grep for it), only these combinations are
used currently.

Tested this with all four output formats 'text', 'json', 'json-pretty'
and 'yaml', as well as "cross-node" in a local test cluster.

Signed-off-by: Christoph Heiss <c.heiss at proxmox.com>
---
 PVE/CLI/pvesh.pm | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/PVE/CLI/pvesh.pm b/PVE/CLI/pvesh.pm
index 9acf292a..85cdaf8e 100755
--- a/PVE/CLI/pvesh.pm
+++ b/PVE/CLI/pvesh.pm
@@ -15,6 +15,7 @@ use PVE::CLIHandler;
 use PVE::API2Tools;
 use PVE::API2;
 use JSON;
+use IO::Uncompress::Gunzip qw(gunzip);
 
 use base qw(PVE::CLIHandler);
 
@@ -281,6 +282,28 @@ my $cond_add_standard_output_properties = sub {
     return PVE::RESTHandler::add_standard_output_properties($props, $keys);
 };
 
+my $handle_streamed_response = sub {
+    my ($download) = @_;
+    my ($fh, $encoding, $type) = $download->@{'fh', 'content-encoding', 'content-type'};
+
+    local $/;
+    my $data = <$fh>;
+
+    if (defined($encoding)) {
+	die "unknown 'content-encoding' $encoding\n" if $encoding ne 'gzip';
+	my $out;
+	gunzip(\$data => \$out);
+	$data = $out;
+    }
+
+    if (defined($type) && not $type =~ qw!^text/plain!) {
+	die "unknown 'content-type' $type\n" if not $type =~ qw!^application/json!;
+	$data = decode_json($data)->{data};
+    }
+
+    return $data;
+};
+
 sub call_api_method {
     my ($cmd, $param) = @_;
 
@@ -310,6 +333,9 @@ sub call_api_method {
 	}
 
 	$data = $handler->handle($info, $param);
+
+	$data = &$handle_streamed_response($data->{download})
+	    if ref($data) eq 'HASH' && defined($data->{download});
     }
 
     return if $opt_nooutput || $stdopts->{quiet};
-- 
2.39.2






More information about the pve-devel mailing list