feat(core/services-gdrive): return stat metadata result

core/src/layers/complete.rs

@@ -177,8 +177,19 @@ impl<A: Access> CompleteAccessor<A> {
             return Ok(RpStat::new(Metadata::new(EntryMode::DIR)));
         }
 
-        // Forward to inner if create_dir is supported.
-        if path.ends_with('/') && capability.create_dir {
+        if path.ends_with('/')
+            && (capability.create_dir || // Forward to inner if create_dir is supported.
+            capability.stat_has_cache_control || // Forward to inner if inner returns metadata.
+            capability.stat_has_content_disposition ||
+            capability.stat_has_content_length ||
+            capability.stat_has_content_md5 ||
+            capability.stat_has_content_range ||
+            capability.stat_has_content_type ||
+            capability.stat_has_etag ||
+            capability.stat_has_last_modified ||
+            capability.stat_has_version ||
+            capability.stat_has_user_metadata)
+        {
             let meta = self.inner.stat(path, args).await?.into_metadata();
 
             if meta.is_file() {
@@ -188,7 +199,7 @@ impl<A: Access> CompleteAccessor<A> {
                 ));
             }
 
-            return Ok(RpStat::new(Metadata::new(EntryMode::DIR)));
+            return Ok(RpStat::new(meta));
         }
 
         // Otherwise, we can simulate stat dir via `list`.

core/src/services/gdrive/backend.rs

@@ -53,10 +53,14 @@ impl Access for GdriveBackend {
             .set_root(&self.core.root)
             .set_native_capability(Capability {
                 stat: true,
+                stat_has_content_length: true,
+                stat_has_content_type: true,
+                stat_has_last_modified: true,
 
                 read: true,
 
                 list: true,
+                list_has_content_type: true,
 
                 write: true,
 
@@ -91,11 +95,12 @@ impl Access for GdriveBackend {
         let gdrive_file: GdriveFile =
             serde_json::from_reader(bs.reader()).map_err(new_json_deserialize_error)?;
 
-        if gdrive_file.mime_type == "application/vnd.google-apps.folder" {
-            return Ok(RpStat::new(Metadata::new(EntryMode::DIR)));
+        let file_type = if gdrive_file.mime_type == "application/vnd.google-apps.folder" {
+            EntryMode::DIR
+        } else {
+            EntryMode::FILE
         };
-
-        let mut meta = Metadata::new(EntryMode::FILE);
+        let mut meta = Metadata::new(file_type).with_content_type(gdrive_file.mime_type);
         if let Some(v) = gdrive_file.size {
             meta = meta.with_content_length(v.parse::<u64>().map_err(|e| {
                 Error::new(ErrorKind::Unexpected, "parse content length").set_source(e)

core/src/services/gdrive/lister.rs

@@ -58,7 +58,7 @@ impl oio::PageList for GdriveLister {
             _ => return Err(parse_error(resp)),
         };
 
-        // Gdrive returns empty content when this dir is not exist.
+        // Google Drive returns empty content when this dir does not exist.
         if bytes.is_empty() {
             ctx.done = true;
             return Ok(());
@@ -94,8 +94,12 @@ impl oio::PageList for GdriveLister {
             let path = format!("{}{}", &self.path, file.name);
             let normalized_path = build_rel_path(root, &path);
 
-            // Update path cache with list result.
-            self.core.path_cache.insert(&path, &file.id).await;
+            // Update path cache when path doesn't exist.
+            // When Google Drive converts a format, for example, Microsoft PowerPoint,
+            // Google Drive keeps two entries with the same ID.
+            if let Ok(None) = self.core.path_cache.get(&path).await {
+                self.core.path_cache.insert(&path, &file.id).await;
+            }
 
             let entry = oio::Entry::new(&normalized_path, Metadata::new(file_type));
             ctx.entries.push_back(entry);