fix: symlink issues

Author: veeso

CHANGELOG.md

@@ -1,12 +1,21 @@
 # Changelog
 
 - [Changelog](#changelog)
+  - [0.1.3](#013)
   - [0.1.2](#012)
   - [0.1.1](#011)
   - [0.1.0](#010)
 
 ---
 
+## 0.1.3
+
+Released on 25/10/2024
+
+- set `modified` on `on_written`
+- symlink: write `target` as file content
+- symlink: fixed file mode
+
 ## 0.1.2
 
 Released on 23/10/2024

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "remotefs-memory"
-version = "0.1.2"
+version = "0.1.3"
 authors = ["Christian Visintin <[email protected]>"]
 edition = "2021"
 license = "MIT"

README.md

@@ -7,7 +7,7 @@
 <p align="center">~ A remotefs implementation for testing and simulation ~</p>
 
 <p align="center">Developed by <a href="https://veeso.github.io/" target="_blank">@veeso</a></p>
-<p align="center">Current version: 0.1.2</p>
+<p align="center">Current version: 0.1.3</p>
 
 <p align="center">
   <a href="https://opensource.org/licenses/MIT"

src/inode.rs

@@ -43,17 +43,17 @@ impl Inode {
     }
 
     /// Create a new [`Inode`] with type **Symlink** with the given metadata and target.
-    pub fn symlink(uid: u32, gid: u32, mode: UnixPex, target: PathBuf) -> Self {
+    pub fn symlink(uid: u32, gid: u32, target: PathBuf) -> Self {
         Self {
             metadata: Metadata::default()
                 .uid(uid)
                 .gid(gid)
                 .file_type(FileType::Symlink)
                 .created(SystemTime::now())
                 .accessed(SystemTime::now())
-                .mode(mode)
-                .symlink(target),
-            content: None,
+                .mode(UnixPex::from(0o777))
+                .symlink(target.clone()),
+            content: Some(target.to_string_lossy().as_bytes().to_vec()),
         }
     }
 

src/lib.rs

@@ -56,6 +56,7 @@ mod test;
 
 use std::io::{Cursor, Read, Write};
 use std::path::{Path, PathBuf};
+use std::time::SystemTime;
 
 pub use orange_trees::{node, Node, Tree};
 use remotefs::fs::stream::{StreamWriter, WriteAndSeek};
@@ -436,12 +437,7 @@ impl RemoteFs for MemoryFs {
             .unwrap_or_else(|| Path::new("/"))
             .to_path_buf();
 
-        let symlink = Inode::symlink(
-            (self.get_uid)(),
-            (self.get_gid)(),
-            UnixPex::from(0o755),
-            target.to_path_buf(),
-        );
+        let symlink = Inode::symlink((self.get_uid)(), (self.get_gid)(), target.to_path_buf());
 
         let parent = self
             .tree
@@ -677,6 +673,8 @@ impl RemoteFs for MemoryFs {
             }
             WriteMode::Create => handle.data.get_ref().len() as u64,
         };
+        value.metadata.modified = Some(SystemTime::now());
+
         debug!("{:?} written {:?}", handle.path, value);
         node.set_value(value);