Abort process writer task if reader is dropped

src/middleware.rs

@@ -1,3 +1,4 @@
+use actix_rt::time::Timeout;
 use actix_web::{
     dev::{Service, ServiceRequest, Transform},
     http::StatusCode,
@@ -9,7 +10,6 @@ use std::{
     pin::Pin,
     task::{Context, Poll},
 };
-use actix_rt::time::Timeout;
 use tracing_futures::{Instrument, Instrumented};
 use uuid::Uuid;
 

src/range.rs

@@ -49,9 +49,7 @@ impl Range {
     ) -> impl Stream<Item = Result<Bytes, UploadError>> + Unpin {
         match self {
             Range::RangeStart(start) => once(ready(Ok(bytes.slice(*start as usize..)))),
-            Range::SuffixLength(from_start) => {
+            Range::SuffixLength(from_start) => once(ready(Ok(bytes.slice(..*from_start as usize)))),
-                once(ready(Ok(bytes.slice(..*from_start as usize))))
-            }
             Range::Segment(start, end) => {
                 once(ready(Ok(bytes.slice(*start as usize..*end as usize))))
             }

src/stream.rs

@@ -19,6 +19,7 @@ pub(crate) struct ProcessRead<I> {
     inner: I,
     err_recv: tokio::sync::oneshot::Receiver<std::io::Error>,
     err_closed: bool,
+    handle: actix_rt::task::JoinHandle<()>,
 }
 
 struct BytesFreezer<S>(S);
@@ -43,7 +44,7 @@ impl Process {
 
         let mut child = self.child;
 
-        actix_rt::spawn(async move {
+        let handle = actix_rt::spawn(async move {
             if let Err(e) = stdin.write_all_buf(&mut input).await {
                 let _ = tx.send(e);
                 return;
@@ -67,6 +68,7 @@ impl Process {
             inner: stdout,
             err_recv: rx,
             err_closed: false,
+            handle,
         }))
     }
 
@@ -81,7 +83,7 @@ impl Process {
 
         let mut child = self.child;
 
-        actix_rt::spawn(async move {
+        let handle = actix_rt::spawn(async move {
             if let Err(e) = tokio::io::copy(&mut input_reader, &mut stdin).await {
                 let _ = tx.send(e);
                 return;
@@ -105,6 +107,7 @@ impl Process {
             inner: stdout,
             err_recv: rx,
             err_closed: false,
+            handle,
         }))
     }
 }
@@ -144,6 +147,12 @@ where
     }
 }
 
+impl<I> Drop for ProcessRead<I> {
+    fn drop(&mut self) {
+        self.handle.abort();
+    }
+}
+
 impl<S> Stream for BytesFreezer<S>
 where
     S: Stream<Item = std::io::Result<BytesMut>> + Unpin,