X-Git-Url: https://git.proxmox.com/?p=pve-client.git;a=blobdiff_plain;f=PVE%2FAPIClient%2FCommands%2Flxc.pm;h=c092028a722ed7d4959dff6aea7a670ec10443d4;hp=81dfd3f01c9437ba2742cf0a4b48bdd7891ccd87;hb=e4c016351a2e807c569c4706f778298529b16761;hpb=e9d848651dd50d169a532d4428f47b0754e5ac9d diff --git a/PVE/APIClient/Commands/lxc.pm b/PVE/APIClient/Commands/lxc.pm index 81dfd3f..c092028 100644 --- a/PVE/APIClient/Commands/lxc.pm +++ b/PVE/APIClient/Commands/lxc.pm @@ -2,6 +2,7 @@ package PVE::APIClient::Commands::lxc; use strict; use warnings; +use Errno qw(EINTR EAGAIN); use JSON; use URI::Escape; use IO::Select; @@ -19,7 +20,7 @@ use base qw(PVE::CLIHandler); use PVE::APIClient::Config; my $CRLF = "\x0D\x0A"; -my $max_payload_size = 65536; +my $max_payload_size = 128*1024; my $build_web_socket_request = sub { my ($host, $path, $ticket, $termproxy) = @_; @@ -128,6 +129,26 @@ my $parse_web_socket_frame = sub { return ($payload, $req_close); }; +my $full_write = sub { + my ($fh, $data) = @_; + + my $len = length($data); + my $todo = $len; + my $offset = 0; + while(1) { + my $nr = syswrite($fh, $data, $todo, $offset); + if (!defined($nr)) { + next if $! == EINTR || $! == EAGAIN; + die "console write error - $!\n" + } + $offset += $nr; + $todo -= $nr; + last if $todo <= 0; + } + + return $len; +}; + __PACKAGE__->register_method ({ name => 'enter', path => 'enter', @@ -175,7 +196,7 @@ __PACKAGE__->register_method ({ my ($request, $wskey) = $build_web_socket_request->( $conn->{host}, "/$api_path/vncwebsocket", $conn->{ticket}, $termproxy); - $web_socket->syswrite($request); + $full_write->($web_socket, $request); my $wsbuf = ''; @@ -212,12 +233,12 @@ __PACKAGE__->register_method ({ # send auth again... my $frame = $create_websockt_frame->($termproxy->{user} . ":" . $termproxy->{ticket} . "\n"); - $web_socket->syswrite($frame); + $full_write->($web_socket, $frame); # Send resize command my ($columns, $rows) = PVE::PTY::tcgetsize(*STDIN); $frame = $create_websockt_frame->("1:$columns:$rows:"); - $web_socket->syswrite($frame); + $full_write->($web_socket, $frame); # Set STDIN to "raw -echo" mode my $old_termios = PVE::PTY::tcgetattr(*STDIN); @@ -241,9 +262,25 @@ __PACKAGE__->register_method ({ my $ctrl_a_pressed_before = 0; + my $winch_received = 0; + $SIG{WINCH} = sub { $winch_received = 1; }; + + my $check_terminal_size = sub { + my ($ncols, $nrows) = PVE::PTY::tcgetsize(*STDIN); + if ($ncols != $columns or $nrows != $rows) { + $columns = $ncols; + $rows = $nrows; + $frame = $create_websockt_frame->("1:$columns:$rows:"); + $full_write->($web_socket, $frame); + } + $winch_received = 0; + }; + while (1) { - while(my @ready = $select->can_read(3)) { - foreach my $fh (@ready) { + while(my ($readable) = IO::Select->select($select, undef, undef, 3)) { + $check_terminal_size->() if $winch_received; + + foreach my $fh (@$readable) { if ($fh == $web_socket) { # Read from WebSocket @@ -256,7 +293,7 @@ __PACKAGE__->register_method ({ } else { my ($payload, $req_close) = $parse_web_socket_frame->(\$wsbuf); if ($payload) { - syswrite(\*STDOUT, $payload); + $full_write->(\*STDOUT, $payload); } return if $req_close; } @@ -275,12 +312,14 @@ __PACKAGE__->register_method ({ $ctrl_a_pressed_before = ($char == hex("0x01") && $ctrl_a_pressed_before == 0) ? 1 : 0; my $frame = $create_websockt_frame->("0:" . $nr . ":" . $buff); - syswrite($web_socket, $frame); + $full_write->($web_socket, $frame); } } } + $check_terminal_size->() if $winch_received; + # got timeout - syswrite($web_socket, $create_websockt_frame->("2")); # ping server to keep connection alive + $full_write->($web_socket, $create_websockt_frame->("2")); # ping server to keep connection alive } }; my $err = $@; @@ -293,12 +332,12 @@ __PACKAGE__->register_method ({ if ($web_socket->connected) { # close connection my $msg = "\x88" . pack('N', 0) . pack('n', 0); # Opcode, mask, statuscode - $web_socket->syswrite($msg); + $full_write->($web_socket, $msg); close($web_socket); } # Reset the terminal parameters. - syswrite(\*STDOUT, "\e[24H\r\n"); + $full_write->(\*STDOUT, "\e[24H\r\n"); PVE::PTY::tcsetattr(*STDIN, $old_termios); }; warn $@ if $@; # show cleanup errors