feat: kill get_collection's redundant extract_resource call

as transaction_http always calls this for us for X-Last-Modified

Issue STOR-579

Author: pjenvey

syncserver/src/web/handlers.rs

@@ -347,14 +347,8 @@ where
         }
     })?;
 
-    let ts = db
-        .extract_resource(coll.user_id.clone(), Some(coll.collection.clone()), None)
-        .await?;
-
     let mut builder = HttpResponse::build(StatusCode::OK);
-    let resp = builder
-        .insert_header((X_LAST_MODIFIED, ts.as_header()))
-        .insert_header((X_WEAVE_RECORDS, result.items.len().to_string()));
+    let resp = builder.insert_header((X_WEAVE_RECORDS, result.items.len().to_string()));
 
     if let Some(offset) = result.offset {
         resp.insert_header((X_WEAVE_NEXT_OFFSET, offset));

syncserver/src/web/transaction.rs

@@ -139,13 +139,11 @@ impl DbTransactionPool {
 
             let mut resp = action(db).await?;
 
-            if resp.headers().contains_key(X_LAST_MODIFIED) {
-                return Ok(resp);
-            }
-
             // See if we already extracted one and use that if possible
-            if let Ok(ts_header) = header::HeaderValue::from_str(&resource_ts.as_header()) {
-                trace!("📝 Setting X-Last-Modfied {:?}", ts_header);
+            if !resp.headers().contains_key(X_LAST_MODIFIED)
+                && let Ok(ts_header) = HeaderValue::from_str(&resource_ts.as_header())
+            {
+                trace!("📝 Setting X-Last-Modfied {ts_header:?}");
                 resp.headers_mut()
                     .insert(header::HeaderName::from_static(X_LAST_MODIFIED), ts_header);
             }