]> git.proxmox.com Git - pve-client.git/blobdiff - PVE/APIClient/Commands/lxc.pm
Use get_standard_option for vmid in enter
[pve-client.git] / PVE / APIClient / Commands / lxc.pm
index 9da6a1f2e16bd1642be3c1f189edcfeeb2e598a0..4e76f70fce4bfc696f1ec7b6f19e46b42497bae2 100644 (file)
@@ -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) = @_;
@@ -71,15 +72,13 @@ my $create_websockt_frame = sub {
 my $parse_web_socket_frame = sub  {
     my ($wsbuf_ref) = @_;
 
-    my $wsbuf = $$wsbuf_ref;
-
     my $payload;
     my $req_close = 0;
 
-    while (my $len = length($wsbuf)) {
+    while (my $len = length($$wsbuf_ref)) {
        last if $len < 2;
 
-       my $hdr = unpack('C', substr($wsbuf, 0, 1));
+       my $hdr = unpack('C', substr($$wsbuf_ref, 0, 1));
        my $opcode = $hdr & 0b00001111;
        my $fin = $hdr & 0b10000000;
 
@@ -88,7 +87,7 @@ my $parse_web_socket_frame = sub  {
        my $rsv = $hdr & 0b01110000;
        die "received websocket frame with RSV flags\n" if $rsv;
 
-       my $payload_len = unpack 'C', substr($wsbuf, 1, 1);
+       my $payload_len = unpack 'C', substr($$wsbuf_ref, 1, 1);
 
        my $masked = $payload_len & 0b10000000;
        die "received masked websocket frame from server\n" if $masked;
@@ -97,11 +96,11 @@ my $parse_web_socket_frame = sub  {
        $payload_len = $payload_len & 0b01111111;
        if ($payload_len == 126) {
            last if $len < 4;
-           $payload_len = unpack('n', substr($wsbuf, $offset, 2));
+           $payload_len = unpack('n', substr($$wsbuf_ref, $offset, 2));
            $offset += 2;
        } elsif ($payload_len == 127) {
            last if $len < 10;
-           $payload_len = unpack('Q>', substr($wsbuf, $offset, 8));
+           $payload_len = unpack('Q>', substr($$wsbuf_ref, $offset, 8));
            $offset += 8;
        }
 
@@ -110,7 +109,7 @@ my $parse_web_socket_frame = sub  {
 
        last if $len < ($offset + $payload_len);
 
-       my $data = substr($wsbuf, 0, $offset + $payload_len, ''); # now consume data
+       my $data = substr($$wsbuf_ref, 0, $offset + $payload_len, ''); # now consume data
 
        my $frame_data = substr($data, $offset, $payload_len);
 
@@ -130,30 +129,34 @@ my $parse_web_socket_frame = sub  {
     return ($payload, $req_close);
 };
 
-my $client_exit = sub {
-    my ($select, $web_socket, $old_termios) = @_;
-
-    foreach my $fh ($select->handles) {
-       $select->remove($fh);
-
-       if ($fh == $web_socket) {
-           if ($fh->connected) {
-
-               # close connection
-               # Opcode, mask, statuscode
-               my $msg = "\x88" . pack('N', 0) . pack('n', 0);
-               $fh->syswrite($msg);
-               close($fh);
-           }
+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;
     }
 
-    #
-    # Reset the terminal parameters.
-    #
-    print "\e[24H\r\n";
-    PVE::PTY::tcsetattr(*STDIN, $old_termios);
+    return $len;
+};
+
+# Takes an escape character with an optional '^' prefix and returns an escape
+# character code.
+my $escapekey_to_char = sub {
+    my ($def) = @_;
+    if ($def =~ /^\^?([a-zA-Z])$/) {
+       return 1 + ord(lc($1)) - ord('a');
+    }
+    die "bad escape key definition: $def\n";
 };
 
 __PACKAGE__->register_method ({
@@ -165,10 +168,7 @@ __PACKAGE__->register_method ({
        additionalProperties => 0,
        properties => {
            remote => get_standard_option('pveclient-remote-name'),
-           vmid => {
-               description => "The container ID",
-               type => 'string',
-           },
+           vmid => get_standard_option('pve-vmid')
        },
     },
     returns => { type => 'null'},
@@ -178,6 +178,9 @@ __PACKAGE__->register_method ({
        my $config = PVE::APIClient::Config->load();
        my $conn = PVE::APIClient::Config->remote_conn($config, $param->{remote});
 
+       # FIXME: This should come from $config
+       my $escape_char = $escapekey_to_char->('a');
+
        # Get the real node from the resources endpoint
        my $resource_list = $conn->get("api2/json/cluster/resources", { type => 'vm'});
        my ($resource) = grep { $_->{type} eq "lxc" && $_->{vmid} eq $param->{vmid}} @$resource_list;
@@ -203,13 +206,15 @@ __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 = '';
 
        my $wb_socket_read_available_bytes = sub {
            my $nr = $web_socket->sysread($wsbuf, $max_payload_size, length($wsbuf));
-           die "web socket read error - $!\n" if $nr < 0;
+           if (!defined($nr) && !($! == EINTR || $! == EAGAIN)) {
+               die "web socket read error - $!\n";
+           }
            return $nr;
        };
 
@@ -240,122 +245,179 @@ __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);
        my $raw_termios = {%$old_termios};
-       PVE::PTY::cfmakeraw($raw_termios);
-       PVE::PTY::tcsetattr(*STDIN, $raw_termios);
 
-       # And set it to non-blocking so we can every char with IO::Select.
-       STDIN->blocking(0);
+       my $read_select = IO::Select->new;
+       my $write_select = IO::Select->new;
 
-       my $select = IO::Select->new;
+       my $output_buffer = ''; # write buffer for STDOUT
+       my $websock_buffer = ''; # write buffer for $web_socket
 
-       $web_socket->blocking(0);
-       $select->add($web_socket);
-       $select->add(fileno(STDIN));
+       eval {
+           $SIG{TERM} = $SIG{INT} = $SIG{KILL} = sub { die "received interrupt\n"; };
 
-       my @messages;
-       my $ctrl_a_pressed_before = 0;
-       my $next_ping = time() + 3;
+           PVE::PTY::cfmakeraw($raw_termios);
+           PVE::PTY::tcsetattr(*STDIN, $raw_termios);
 
-       eval {
-           while (1) {
-               # Ping server every 3 seconds.
-               my $now = time();
-               if ($now >= $next_ping) {
-                   push(@messages, $create_websockt_frame->("2"));
-                   $next_ping = $now + 3;
-               }
+           # And set it to non-blocking so we can every char with IO::Select.
+           STDIN->blocking(0);
+           STDOUT->blocking(0);
+           $web_socket->blocking(0);
+           $read_select->add($web_socket);
+           my $input_fh = \*STDIN;
+           $read_select->add($input_fh);
 
-               # Write
-               foreach my $fh ($select->can_write(0.5)) {
-                   if ($fh == $web_socket and my $msg = shift @messages) {
-                       $fh->syswrite($msg, length($msg));
-                   }
-               }
+           my $output_fh = \*STDOUT;
 
-               # Read
-               foreach my $fh ($select->can_read(0.5)) {
+           my $in_escape_sequence;
 
-                   # From Web Socket
-                   if ($fh == $web_socket) {
-                       # Read from WebSocket
-                       my $nr = $wb_socket_read_available_bytes->();
-                       my ($payload, $req_close) = $parse_web_socket_frame->(\$wsbuf);
+           my $winch_received = 0;
+           $SIG{WINCH} = sub { $winch_received = 1; };
 
-                       if ($payload ne "OK") {
-                           syswrite(\*STDOUT, $payload, length($payload));
-                       }
-                   }
+           my $check_terminal_size = sub {
+               my ($ncols, $nrows) = PVE::PTY::tcgetsize(*STDIN);
+               if ($ncols != $columns or $nrows != $rows) {
+                   $columns = $ncols;
+                   $rows = $nrows;
+                   $websock_buffer .= $create_websockt_frame->("1:$columns:$rows:");
+                   $write_select->add($web_socket);
+               }
+               $winch_received = 0;
+           };
 
-                   # From STDIN
-                   elsif ($fh == fileno(STDIN)) {
+           my $max_buffer_len = 256*1024;
 
-                       # Read from STDIN
-                       my $nr = read(\*STDIN, my $buff, 4096);
-                       if (!$nr) {
-                           next;
-                       }
+           my $drain_buffer = sub {
+               my ($fh, $buffer_ref) = @_;
 
-                       my $char = ord($buff);
+               my $len = length($$buffer_ref);
+               my $nr = syswrite($fh, $$buffer_ref);
+               if (!defined($nr)) {
+                   return if $! == EINTR || $! == EAGAIN;
+                   die "drain buffer - write error - $!\n";
+               }
+               return $nr if !$nr;
+               substr($$buffer_ref, 0, $nr, '');
+               $len = length($$buffer_ref);
+               $write_select->remove($fh) if !$len;
+           };
 
-                       if ($ctrl_a_pressed_before == 1 && $char == hex("0x71")) {
-                           $client_exit->($select, $web_socket, $old_termios);
-                           return;
+           while (1) {
+               while(my ($readable, $writable) = IO::Select->select($read_select, $write_select, undef, 3)) {
+                   $check_terminal_size->() if $winch_received;
+
+                   foreach my $fh (@$writable) {
+                       if ($fh == $output_fh) {
+                           $drain_buffer->($output_fh, \$output_buffer);
+                           $read_select->add($web_socket) if length($output_buffer) <= $max_buffer_len;
+                       } elsif ($fh == $web_socket) {
+                           $drain_buffer->($web_socket, \$websock_buffer);
                        }
+                   }
 
-                       if ($char == hex("0x01")) {
-                           if ($ctrl_a_pressed_before == 0) {
-                               $ctrl_a_pressed_before = 1;
+                   foreach my $fh (@$readable) {
+
+                       if ($fh == $web_socket) {
+                           # Read from WebSocket
+
+                           my $nr = $wb_socket_read_available_bytes->();
+                           if (!defined($nr)) {
+                               # wait
+                           } elsif ($nr == 0) {
+                               return; # EOF
+                           } else {
+                               my ($payload, $req_close) = $parse_web_socket_frame->(\$wsbuf);
+                               if (defined($payload) && length($payload)) {
+                                   $output_buffer .= $payload;
+                                   $write_select->add($output_fh);
+                                   if (length($output_buffer) > $max_buffer_len) {
+                                       $read_select->remove($web_socket);
+                                   }
+                               }
+                               return if $req_close;
                            }
-                       }
-                       else {
-                           $ctrl_a_pressed_before = 0;
-                       }
 
-                       push(@messages, $create_websockt_frame->("0:" . $nr . ":" . $buff));
+                       } elsif ($fh == $input_fh) {
+                           # Read from STDIN
+
+                           my $nr = sysread($input_fh, my $buff, 4096);
+                           return if !$nr; # EOF or error
+
+                           my $char = ord($buff);
+
+                           # Handle escape sequences:
+                           if ($in_escape_sequence) {
+                               $in_escape_sequence = 0;
+                               if ($char == 0x71) {
+                                   # (escape, 'q')
+                                   return;
+                               } elsif ($char == $escape_char) {
+                                   # (escape, escape)
+                                   # Pass this one through as a single escapekey
+                               } else {
+                                   # Unknown escape sequence
+                                   # We could generate a bell or something...
+                                   # but for now just skip it
+                                   next;
+                               }
+                           } elsif ($char == $escape_char) {
+                               $in_escape_sequence = 1;
+                               next;
+                           }
+
+                           # Pass the key through:
+                           $websock_buffer .= $create_websockt_frame->("0:" . $nr . ":" . $buff);
+                           $write_select->add($web_socket);
+                       }
                    }
                }
+               $check_terminal_size->() if $winch_received;
+
+               # got timeout
+               $websock_buffer .= $create_websockt_frame->("2"); # ping server to keep connection alive
+               $write_select->add($web_socket);
            }
        };
-       print "ERROR: " . $@ . ".\n" if $@;
+       my $err = $@;
 
-       $client_exit->($select, $web_socket, $old_termios);
+       eval {  # cleanup
 
-       return undef
-    }});
+           # switch back to blocking mode (else later shell commands will fail).
+           STDIN->blocking(1);
 
-__PACKAGE__->register_method ({
-    name => 'list',
-    path => 'list',
-    method => 'GET',
-    description => "List containers.",
-    parameters => {
-       additionalProperties => 0,
-       properties => {
-           remote => get_standard_option('pveclient-remote-name'),
-       },
-    },
-    returns => { type => 'null'},
-    code => sub {
-       my ($param) = @_;
+           if ($web_socket->connected) {
+               # close connection
+               $websock_buffer .= "\x88" . pack('N', 0) . pack('n', 0); # Opcode, mask, statuscode
+               $full_write->($web_socket, $websock_buffer);
+               $websock_buffer = '';
+               close($web_socket);
+           }
 
-       die "implement me";
+           # Reset the terminal parameters.
+           $output_buffer .= "\e[24H\r\n";
+           $full_write->(\*STDOUT, $output_buffer);
+           $output_buffer = '';
 
-    }});
+           PVE::PTY::tcsetattr(*STDIN, $old_termios);
+       };
+       warn $@ if $@; # show cleanup errors
+
+       print STDERR "\nERROR: $err" if $err;
 
+       return undef;
+    }});
 
 our $cmddef = {
     enter => [ __PACKAGE__, 'enter', ['remote', 'vmid']],
-    list => [ __PACKAGE__, 'list', ['remote']],
 };
 
 1;