bind streaming codec to hyper 1.x

8 files changed, 186 insertions, 39 deletions

diff --git a/Cargo.lock b/Cargo.lock
index 77dda64..c6228af 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -91,6 +91,7 @@ dependencies = [
  "hyper-util",
  "imap-codec",
  "imap-flow",
+ "quick-xml",
  "rustls 0.22.2",
  "rustls-pemfile 2.1.1",
  "smtp-message",
@@ -98,6 +99,7 @@ dependencies = [
  "thiserror",
  "tokio",
  "tokio-rustls 0.25.0",
+ "tokio-stream",
  "tokio-util",
  "tracing",
 ]
@@ -1824,12 +1826,12 @@ dependencies = [
 
 [[package]]
 name = "http-body-util"
-version = "0.1.0"
+version = "0.1.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "41cb79eb393015dadd30fc252023adb0b2400a0caee0fa2a077e6e21a551e840"
+checksum = "0475f8b2ac86659c21b64320d5d653f9efe42acd2a4e560073ec61a155a34f1d"
 dependencies = [
  "bytes",
- "futures-util",
+ "futures-core",
  "http 1.1.0",
  "http-body 1.0.0",
  "pin-project-lite 0.2.13",
@@ -3381,9 +3383,9 @@ checksum = "1f3ccbac311fea05f86f61904b462b55fb3df8837a366dfc601a0161d0532f20"
 
 [[package]]
 name = "tokio"
-version = "1.35.1"
+version = "1.36.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c89b4efa943be685f629b149f53829423f8f5531ea21249408e8e2f8671ec104"
+checksum = "61285f6515fa018fb2d1e46eb21223fff441ee8db5d0f1435e8ab4f5cdb80931"
 dependencies = [
  "backtrace",
  "bytes",
diff --git a/Cargo.toml b/Cargo.toml
index a18c41c..d4bc543 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -24,8 +24,9 @@ aero-proto = { version = "0.3.0", path = "aero-proto" }
 aerogramme = { version = "0.3.0", path = "aerogramme" }
 
 # async runtime
-tokio = { version = "1.18", default-features = false, features = ["rt", "rt-multi-thread", "io-util", "net", "time", "macros", "sync", "signal", "fs"] }
+tokio = { version = "1.36", default-features = false, features = ["rt", "rt-multi-thread", "io-util", "net", "time", "macros", "sync", "signal", "fs"] }
 tokio-util = { version = "0.7", features = [ "compat" ]  }
+tokio-stream = { version = "0.1" }
 futures = "0.3"
 
 # debug
@@ -57,8 +58,8 @@ imap-codec = { version = "2.0.0", features = ["bounded-static", "ext_condstore_q
 imap-flow = { git = "https://github.com/duesee/imap-flow.git", branch = "main" }
 
 # http & web
-http = "1.0"
-http-body-util = "0.1"
+http = "1.1"
+http-body-util = "0.1.1"
 hyper = "1.2"
 hyper-rustls = { version = "0.26", features = ["http2"] }
 hyper-util = { version = "0.1", features = ["full"] }
diff --git a/aero-dav/src/calencoder.rs b/aero-dav/src/calencoder.rs
index fff4d47..e00876d 100644
--- a/aero-dav/src/calencoder.rs
+++ b/aero-dav/src/calencoder.rs
@@ -393,7 +393,14 @@ impl QWrite for CompKind {
                 for comp in many_comp.iter() {
                     // Required: recursion in an async fn requires boxing
                     // rustc --explain E0733
-                    Box::pin(comp.qwrite(xml)).await?;
+                    // Cycle detected when computing type of ...
+                    // For more information about this error, try `rustc --explain E0391`.
+                    // https://github.com/rust-lang/rust/issues/78649
+                    #[inline(always)]
+                    fn recurse<'a>(comp: &'a Comp, xml: &'a mut Writer<impl IWrite>) -> futures::future::BoxFuture<'a, Result<(), QError>> {
+                        Box::pin(comp.qwrite(xml))
+                    }
+                    recurse(comp, xml).await?;
                 }
                 Ok(())
             }
@@ -525,7 +532,14 @@ impl QWrite for CompFilterMatch {
         for comp_item in self.comp_filter.iter() {
             // Required: recursion in an async fn requires boxing
             // rustc --explain E0733
-            Box::pin(comp_item.qwrite(xml)).await?;
+            // Cycle detected when computing type of ...
+            // For more information about this error, try `rustc --explain E0391`.
+            // https://github.com/rust-lang/rust/issues/78649
+            #[inline(always)]
+            fn recurse<'a>(comp: &'a CompFilter, xml: &'a mut Writer<impl IWrite>) -> futures::future::BoxFuture<'a, Result<(), QError>> {
+                Box::pin(comp.qwrite(xml))
+            }
+            recurse(comp_item, xml).await?;
         }
         Ok(())
     }
diff --git a/aero-dav/src/error.rs b/aero-dav/src/error.rs
index f1b5cba..570f779 100644
--- a/aero-dav/src/error.rs
+++ b/aero-dav/src/error.rs
@@ -15,6 +15,25 @@ pub enum ParsingError {
     Int(std::num::ParseIntError),
     Eof
 }
+impl std::fmt::Display for ParsingError {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        match self {
+            Self::Recoverable => write!(f, "Recoverable"),
+            Self::MissingChild => write!(f, "Missing child"),
+            Self::MissingAttribute => write!(f, "Missing attribute"),
+            Self::NamespacePrefixAlreadyUsed => write!(f, "Namespace prefix already used"),
+            Self::WrongToken => write!(f, "Wrong token"),
+            Self::TagNotFound => write!(f, "Tag not found"),
+            Self::InvalidValue => write!(f, "Invalid value"),
+            Self::Utf8Error(_) => write!(f, "Utf8 Error"),
+            Self::QuickXml(_) => write!(f, "Quick XML error"),
+            Self::Chrono(_) => write!(f, "Chrono error"),
+            Self::Int(_) => write!(f, "Number parsing error"),
+            Self::Eof => write!(f, "Found EOF while expecting data"),
+        }
+    }
+}
+impl std::error::Error for ParsingError {}
 impl From<AttrError> for ParsingError {
     fn from(value: AttrError) -> Self {
         Self::QuickXml(value.into())
diff --git a/aero-dav/src/realization.rs b/aero-dav/src/realization.rs
index 5781637..a7bbb16 100644
--- a/aero-dav/src/realization.rs
+++ b/aero-dav/src/realization.rs
@@ -11,8 +11,8 @@ impl xml::QRead<Disabled> for Disabled {
     }
 }
 impl xml::QWrite for Disabled {
-    async fn qwrite(&self, _xml: &mut xml::Writer<impl xml::IWrite>) -> Result<(), quick_xml::Error> {
-        unreachable!();
+    fn qwrite(&self, _xml: &mut xml::Writer<impl xml::IWrite>) -> impl futures::Future<Output = Result<(), quick_xml::Error>> + Send {
+        async { unreachable!(); }
     }
 }
 
diff --git a/aero-dav/src/xml.rs b/aero-dav/src/xml.rs
index 1f8a6b1..e078c6f 100644
--- a/aero-dav/src/xml.rs
+++ b/aero-dav/src/xml.rs
@@ -12,19 +12,19 @@ pub const CAL_URN: &[u8] = b"urn:ietf:params:xml:ns:caldav";
 pub const CARD_URN: &[u8] = b"urn:ietf:params:xml:ns:carddav";
 
 // Async traits
-pub trait IWrite = AsyncWrite + Unpin;
+pub trait IWrite = AsyncWrite + Unpin + Send;
 pub trait IRead = AsyncBufRead + Unpin;
 
 // Serialization/Deserialization traits
 pub trait QWrite {
-    fn qwrite(&self, xml: &mut Writer<impl IWrite>) -> impl Future<Output = Result<(), quick_xml::Error>>; 
+    fn qwrite(&self, xml: &mut Writer<impl IWrite>) -> impl Future<Output = Result<(), quick_xml::Error>> + Send;
 }
 pub trait QRead<T> {
     fn qread(xml: &mut Reader<impl IRead>) -> impl Future<Output = Result<T, ParsingError>>;
 }
 
 // The representation of an XML node in Rust
-pub trait Node<T> = QRead<T> + QWrite + std::fmt::Debug + PartialEq;
+pub trait Node<T> = QRead<T> + QWrite + std::fmt::Debug + PartialEq + Sync;
 
 // ---------------
 
diff --git a/aero-proto/Cargo.toml b/aero-proto/Cargo.toml
index df8c696..e9f28d1 100644
--- a/aero-proto/Cargo.toml
+++ b/aero-proto/Cargo.toml
@@ -22,6 +22,7 @@ futures.workspace = true
 tokio.workspace = true
 tokio-util.workspace = true
 tokio-rustls.workspace = true
+tokio-stream.workspace = true
 rustls.workspace = true
 rustls-pemfile.workspace = true
 imap-codec.workspace = true
@@ -33,3 +34,4 @@ duplexify.workspace = true
 smtp-message.workspace = true
 smtp-server.workspace = true
 tracing.workspace = true
+quick-xml.workspace = true
diff --git a/aero-proto/src/dav.rs b/aero-proto/src/dav.rs
index 42fde2c..3981b61 100644
--- a/aero-proto/src/dav.rs
+++ b/aero-proto/src/dav.rs
@@ -6,6 +6,8 @@ use base64::Engine;
 use hyper::service::service_fn;
 use hyper::{Request, Response, body::Bytes};
 use hyper::server::conn::http1 as http;
+use hyper::rt::{Read, Write};
+use hyper::body::Incoming;
 use hyper_util::rt::TokioIo;
 use http_body_util::Full;
 use futures::stream::{FuturesUnordered, StreamExt};
@@ -13,13 +15,16 @@ use tokio::net::TcpListener;
 use tokio::sync::watch;
 use tokio_rustls::TlsAcceptor;
 use tokio::net::TcpStream;
-use hyper::rt::{Read, Write};
 use tokio::io::{AsyncRead, AsyncWrite};
+use tokio::io::AsyncWriteExt;
 use rustls_pemfile::{certs, private_key};
 
 use aero_user::config::{DavConfig, DavUnsecureConfig};
 use aero_user::login::ArcLoginProvider;
 use aero_collections::user::User;
+use aero_dav::types::{PropFind, Multistatus, PropValue, ResponseDescription};
+use aero_dav::realization::{Core, Calendar};
+use aero_dav::xml as dav;
 
 pub struct Server {
     bind_addr: SocketAddr,
@@ -94,13 +99,10 @@ impl Server {
                 //abitrarily bound
                 //@FIXME replace with a handler supporting http2 and TLS
 
-
                 match http::Builder::new().serve_connection(stream, service_fn(|req: Request<hyper::body::Incoming>| {
                     let login = login.clone();
                     tracing::info!("{:?} {:?}", req.method(), req.uri());
-                    async move {
-                        auth(login, req).await
-                    }
+                    auth(login, req)
                 })).await {
                     Err(e) => tracing::warn!(err=?e, "connection failed"),
                     Ok(()) => tracing::trace!("connection terminated with success"),
@@ -127,11 +129,13 @@ impl Server {
     }
 }
 
+use http_body_util::BodyExt;
+
 //@FIXME We should not support only BasicAuth
 async fn auth(
     login: ArcLoginProvider,
-    req: Request<impl hyper::body::Body>, 
-) -> Result<Response<Full<Bytes>>> {
+    req: Request<Incoming>, 
+) -> Result<Response<BoxBody<Bytes, std::io::Error>>> {
 
     tracing::info!("headers: {:?}", req.headers());
     let auth_val = match req.headers().get(hyper::header::AUTHORIZATION) {
@@ -141,7 +145,7 @@ async fn auth(
             return Ok(Response::builder()
                 .status(401)
                 .header("WWW-Authenticate", "Basic realm=\"Aerogramme\"")
-                .body(Full::new(Bytes::from("Missing Authorization field")))?)
+                .body(text_body("Missing Authorization field"))?)
         },
     };
 
@@ -151,7 +155,7 @@ async fn auth(
             tracing::info!("Unsupported authorization field");
             return Ok(Response::builder()
                 .status(400)
-                .body(Full::new(Bytes::from("Unsupported Authorization field")))?)
+                .body(text_body("Unsupported Authorization field"))?)
         },
     };
 
@@ -176,7 +180,7 @@ async fn auth(
             return Ok(Response::builder()
                 .status(401)
                 .header("WWW-Authenticate", "Basic realm=\"Aerogramme\"")
-                .body(Full::new(Bytes::from("Wrong credentials")))?)
+                .body(text_body("Wrong credentials"))?)
         },
     };
 
@@ -187,26 +191,131 @@ async fn auth(
     router(user, req).await 
 }
 
-async fn router(user: std::sync::Arc<User>, req: Request<impl hyper::body::Body>) -> Result<Response<Full<Bytes>>> {
-    let path_segments: Vec<_> = req.uri().path().split("/").filter(|s| *s != "").collect();
-    tracing::info!("router");
-    match path_segments.as_slice() {
-        [] => tracing::info!("root"),
-        [ username, ..] if *username != user.username => return Ok(Response::builder()
+async fn router(user: std::sync::Arc<User>, req: Request<Incoming>) -> Result<Response<BoxBody<Bytes, std::io::Error>>> {
+    let path = req.uri().path().to_string();
+    let path_segments: Vec<_> = path.split("/").filter(|s| *s != "").collect();
+    let method = req.method().as_str().to_uppercase();
+
+    match (method.as_str(), path_segments.as_slice()) {
+        ("PROPFIND", []) => propfind_root(user, req).await,
+        (_, [ username, ..]) if *username != user.username => return Ok(Response::builder()
             .status(403)
-            .body(Full::new(Bytes::from("Accessing other user ressources is not allowed")))?),
-        [ _ ] => tracing::info!("user home"),
-        [ _, "calendar" ] => tracing::info!("user calendars"),
-        [ _, "calendar", colname ] => tracing::info!(name=colname, "selected calendar"),
-        [ _, "calendar", colname, member ] => tracing::info!(name=colname, obj=member, "selected event"),
+            .body(text_body("Accessing other user ressources is not allowed"))?),
+        ("PROPFIND", [ _ ]) => propfind_home(user, &req).await,
+        ("PROPFIND", [ _, "calendar" ]) => propfind_all_calendars(user, &req).await,
+        ("PROPFIND", [ _, "calendar", colname ]) => propfind_this_calendar(user, &req, colname).await,
+        ("PROPFIND", [ _, "calendar", colname, event ]) => propfind_event(user, req, colname, event).await,
         _ => return Ok(Response::builder()
-            .status(404)
-            .body(Full::new(Bytes::from("Resource not found")))?),
+            .status(501)
+            .body(text_body("Not implemented"))?),
     }
-    Ok(Response::new(Full::new(Bytes::from("Hello World!"))))
 }
 
+/// <D:propfind xmlns:D='DAV:' xmlns:A='http://apple.com/ns/ical/'>
+/// <D:prop><D:getcontenttype/><D:resourcetype/><D:displayname/><A:calendar-color/>
+/// </D:prop></D:propfind>
+
+async fn propfind_root(user: std::sync::Arc<User>, req: Request<Incoming>) -> Result<Response<BoxBody<Bytes, std::io::Error>>> { 
+    tracing::info!("root");
+
+    let r = deserialize::<PropFind<Core>>(req).await?;
+    println!("r: {:?}", r);
+    serialize(Multistatus::<Core, PropValue<Core>> {
+        responses: vec![],
+        responsedescription: Some(ResponseDescription("hello world".to_string())),
+    })
+}
+
+async fn propfind_home(user: std::sync::Arc<User>, req: &Request<impl hyper::body::Body>) -> Result<Response<BoxBody<Bytes, std::io::Error>>> {
+    tracing::info!("user home");
+    Ok(Response::new(text_body("Hello World!")))
+}
+
+async fn propfind_all_calendars(user: std::sync::Arc<User>, req: &Request<impl hyper::body::Body>) -> Result<Response<BoxBody<Bytes, std::io::Error>>> {
+    tracing::info!("calendar");
+    Ok(Response::new(text_body("Hello World!")))
+}
+
+async fn propfind_this_calendar(
+    user: std::sync::Arc<User>, 
+    req: &Request<Incoming>,
+    colname: &str
+) -> Result<Response<BoxBody<Bytes, std::io::Error>>> {
+    tracing::info!(name=colname, "selected calendar");
+    Ok(Response::new(text_body("Hello World!")))
+}
+
+async fn propfind_event(
+    user: std::sync::Arc<User>, 
+    req: Request<Incoming>,
+    colname: &str,
+    event: &str,
+) -> Result<Response<BoxBody<Bytes, std::io::Error>>> {
+    tracing::info!(name=colname, obj=event, "selected event");
+    Ok(Response::new(text_body("Hello World!")))
+}
+
+
 #[allow(dead_code)]
 async fn collections(_user: std::sync::Arc<User>, _req: Request<impl hyper::body::Body>) -> Result<Response<Full<Bytes>>> {
     unimplemented!();
 }
+
+
+use futures::stream::TryStreamExt;
+use http_body_util::{BodyStream, Empty};
+use http_body_util::StreamBody;
+use http_body_util::combinators::BoxBody;
+use hyper::body::Frame;
+use tokio_util::sync::PollSender;
+use std::io::{Error, ErrorKind};
+use futures::sink::SinkExt;
+use tokio_util::io::{SinkWriter, CopyToBytes};
+
+
+fn text_body(txt: &'static str) -> BoxBody<Bytes, std::io::Error> {
+    BoxBody::new(Full::new(Bytes::from(txt)).map_err(|e| match e {}))
+}
+
+fn serialize<T: dav::QWrite + Send + 'static>(elem: T) -> Result<Response<BoxBody<Bytes, std::io::Error>>> {
+    let (tx, rx) = tokio::sync::mpsc::channel::<Bytes>(1);
+
+    // Build the writer
+    tokio::task::spawn(async move {
+        let sink = PollSender::new(tx).sink_map_err(|_| Error::from(ErrorKind::BrokenPipe));
+        let mut writer = SinkWriter::new(CopyToBytes::new(sink));
+        let q = quick_xml::writer::Writer::new_with_indent(&mut writer, b' ', 4);
+        let ns_to_apply = vec![ ("xmlns:D".into(), "DAV:".into()) ];
+        let mut qwriter = dav::Writer { q, ns_to_apply };
+        match elem.qwrite(&mut qwriter).await {
+            Ok(_) => tracing::debug!("fully serialized object"),
+            Err(e) => tracing::error!(err=?e, "failed to serialize object"),
+        }
+    });
+
+
+    // Build the reader
+    let recv = tokio_stream::wrappers::ReceiverStream::new(rx);
+    let stream = StreamBody::new(recv.map(|v| Ok(Frame::data(v))));
+    let boxed_body = BoxBody::new(stream);
+
+    let response = Response::builder()
+        .status(hyper::StatusCode::OK)
+        .body(boxed_body)?;
+
+    Ok(response)
+}
+
+
+/// Deserialize a request body to an XML request
+async fn deserialize<T: dav::Node<T>>(req: Request<Incoming>) -> Result<T> {
+    let stream_of_frames = BodyStream::new(req.into_body());
+    let stream_of_bytes = stream_of_frames
+        .try_filter_map(|frame| async move { Ok(frame.into_data().ok()) })
+        .map_err(|err| std::io::Error::new(std::io::ErrorKind::Other, err));
+    let async_read = tokio_util::io::StreamReader::new(stream_of_bytes);
+    let async_read = std::pin::pin!(async_read);
+    let mut rdr = dav::Reader::new(quick_xml::reader::NsReader::from_reader(async_read)).await?;
+    let parsed = rdr.find::<T>().await?;
+    Ok(parsed)
+}