propfind does not need to be async

crates/caldav/src/calendar/methods/report/calendar_multiget.rs

@@ -90,11 +90,11 @@ pub async fn handle_calendar_multiget<C: CalendarStore + ?Sized>(
     let mut responses = Vec::new();
     for object in objects {
         let path = format!("{}/{}", req.path(), object.get_uid());
-        responses.push(
-            CalendarObjectResource::from(object)
-                .propfind(&path, props.clone(), req.resource_map())
-                .await?,
-        );
+        responses.push(CalendarObjectResource::from(object).propfind(
+            &path,
+            props.clone(),
+            req.resource_map(),
+        )?);
     }
 
     let not_found_responses = not_found

crates/caldav/src/calendar/methods/report/calendar_query.rs

@@ -236,11 +236,11 @@ pub async fn handle_calendar_query<C: CalendarStore + ?Sized>(
             vec![principal, cid, object.get_uid()],
         )
         .unwrap();
-        responses.push(
-            CalendarObjectResource::from(object)
-                .propfind(&path, props.clone(), req.resource_map())
-                .await?,
-        );
+        responses.push(CalendarObjectResource::from(object).propfind(
+            &path,
+            props.clone(),
+            req.resource_map(),
+        )?);
     }
 
     Ok(MultistatusElement {

crates/caldav/src/calendar/methods/report/sync_collection.rs

@@ -75,11 +75,11 @@ pub async fn handle_sync_collection<C: CalendarStore + ?Sized>(
             vec![principal, cid, &object.get_uid()],
         )
         .unwrap();
-        responses.push(
-            CalendarObjectResource::from(object)
-                .propfind(&path, props.clone(), req.resource_map())
-                .await?,
-        );
+        responses.push(CalendarObjectResource::from(object).propfind(
+            &path,
+            props.clone(),
+            req.resource_map(),
+        )?);
     }
 
     for object_uid in deleted_objects {

crates/dav/src/methods/propfind.rs

@@ -76,18 +76,12 @@ pub async fn route_propfind<R: ResourceService>(
     let mut member_responses = Vec::new();
     if depth != Depth::Zero {
         for (path, member) in resource_service.get_members(req.resource_map()).await? {
-            member_responses.push(
-                member
-                    .propfind(&path, props.clone(), req.resource_map())
-                    .await?,
-            );
+            member_responses.push(member.propfind(&path, props.clone(), req.resource_map())?);
         }
     }
 
     let resource = resource_service.get_resource(user.id).await?;
-    let response = resource
-        .propfind(req.path(), props, req.resource_map())
-        .await?;
+    let response = resource.propfind(req.path(), props, req.resource_map())?;
 
     Ok(MultistatusElement {
         responses: vec![response],

crates/dav/src/resource.rs

@@ -52,8 +52,7 @@ pub trait Resource: Clone {
             .to_owned())
     }
 
-    #[allow(async_fn_in_trait)]
-    async fn propfind(
+    fn propfind(
         &self,
         path: &str,
         mut props: Vec<&str>,