diff options
author | metamuffin <metamuffin@disroot.org> | 2024-01-29 13:11:37 +0100 |
---|---|---|
committer | metamuffin <metamuffin@disroot.org> | 2024-01-29 13:11:37 +0100 |
commit | 424286e569fa48314447464070e9f4e8cfd8dc94 (patch) | |
tree | 9d37d80f96d13c9c4e23d9a833ff4c60579b60eb | |
parent | 6321843c73b2f0469e6044f5391d835d68f18816 (diff) | |
download | jellything-424286e569fa48314447464070e9f4e8cfd8dc94.tar jellything-424286e569fa48314447464070e9f4e8cfd8dc94.tar.bz2 jellything-424286e569fa48314447464070e9f4e8cfd8dc94.tar.zst |
fix problem with mpv crashing on range responses
-rw-r--r-- | server/src/routes/stream.rs | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/server/src/routes/stream.rs b/server/src/routes/stream.rs index 71a01df..4c9906d 100644 --- a/server/src/routes/stream.rs +++ b/server/src/routes/stream.rs @@ -169,11 +169,13 @@ impl<'r> Responder<'r, 'static> for StreamResponse { let mut b = Response::build(); b.status(Status::Ok); b.header(Header::new("access-control-allow-origin", "*")); - if let Some(range) = self.range { - b.status(Status::PartialContent); - b.header(Header::new("content-range", range.to_cr_hv())); - } if self.advertise_range { + //* it is very important here to not reply with content range if we didnt advertise. + //* mpv requests range but will crash if we dont pretend to not support it. + if let Some(range) = self.range { + b.status(Status::PartialContent); + b.header(Header::new("content-range", range.to_cr_hv())); + } b.header(Header::new("accept-ranges", "bytes")); } b.header(Header::new("content-type", self.content_type)) |