proxmox-mirrors/proxmox
1
0
Fork 1
mirror of https://git.proxmox.com/git/proxmox synced 2025-08-04 11:58:04 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

proxmox-http/websocket: remove subprotocol handling

Browse Source 
we do not support websocket subprotocols, but for compatibility with
current clients (novnc, pve-xtermjs) we have to reply with the one requested,
else this is a protocol error and browsers will error out

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Dominik Csapak 2021-05-17 15:11:01 +02:00 committed by Fabian Grünbichler
parent 232d87531e
commit 0786b4b98c
1 changed files with 10 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
proxmox-http/src/websocket/mod.rs
View File

@ -671,13 +671,6 @@ impl WebSocket {
.ok_or_else(|| format_err!("missing websocket key"))?
.to_str()?;
let ws_proto = headers
.get(SEC_WEBSOCKET_PROTOCOL)
.ok_or_else(|| format_err!("missing websocket key"))?
.to_str()?;
let text = ws_proto == "text";
if protocols != "websocket" {
bail!("invalid protocol name");
}
@ -693,13 +686,19 @@ impl WebSocket {
sha1.update(data.as_bytes());
let response_key = base64::encode(sha1.finish());
let response = Response::builder()
let mut response = Response::builder()
.status(StatusCode::SWITCHING_PROTOCOLS)
.header(UPGRADE, HeaderValue::from_static("websocket"))
.header(CONNECTION, HeaderValue::from_static("Upgrade"))
.header(SEC_WEBSOCKET_ACCEPT, response_key)
.header(SEC_WEBSOCKET_PROTOCOL, ws_proto)
.body(Body::empty())?;
.header(SEC_WEBSOCKET_ACCEPT, response_key);
// We currently do not support any subprotocols and we always send binary frames,
// but for backwards compatibilty we need to reply the requested protocols
if let Some(ws_proto) = headers.get(SEC_WEBSOCKET_PROTOCOL) {
response = response.header(SEC_WEBSOCKET_PROTOCOL, ws_proto)
}
let response = response.body(Body::empty())?;
Ok((Self, response))
}