proxmox-mirrors/pve-http-server
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-http-server synced 2025-05-29 18:22:33 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

AnyEvent/websocket_proxy: remove 'base64' handling

Browse Source 
novnc does not support this anymore since 2015, and neither does
our xtermjs client. it is also not listed in IANAs list of websocket
protocols [0].

so simply drop it and only send out binary frames and don't decode text frames

0: https://www.iana.org/assignments/websocket/websocket.xml#subprotocol-name

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Dominik Csapak 2021-05-17 15:07:34 +02:00 committed by Fabian Grünbichler
parent f8f97c8d26
commit 015964a341
1 changed files with 3 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/PVE/APIServer/AnyEvent.pm
View File

@ -496,15 +496,6 @@ sub websocket_proxy {
my $max_payload_size = 128*1024;
my $binary;
if ($wsproto eq 'binary') {
$binary = 1;
} elsif ($wsproto eq 'base64') {
$binary = 0;
} else {
die "websocket_proxy: unsupported protocol '$wsproto'\n";
}
if ($param->{port}) {
$remhost = 'localhost';
$remport = $param->{port};
@ -520,13 +511,9 @@ sub websocket_proxy {
my $string;
my $payload;
if ($binary) {
$string = $opcode ? $opcode : "\x82"; # binary frame
$payload = $$data;
} else {
$string = $opcode ? $opcode : "\x81"; # text frame
$payload = encode_base64($$data, '');
}
$string = $opcode ? $opcode : "\x82"; # binary frame
$payload = $$data;
my $payload_len = length($payload);
if ($payload_len <= 125) {
@ -635,8 +622,6 @@ sub websocket_proxy {
$payload ^= $mask;
}
$payload = decode_base64($payload) if !$binary;
if ($opcode == 1 || $opcode == 2) {
$reqstate->{proxyhdl}->push_write($payload) if $reqstate->{proxyhdl};
} elsif ($opcode == 8) {