prometheus-client-mmap 1.0.0-x86_64-linux → 1.0.2-x86_64-linux

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 73360e69c9b83ad6a3748094621dc3f93d5fc9d9ff79e881f320bcbb35d9f798
-  data.tar.gz: 6a9fa48dbb5da204fe5bf5007ec8a24f4be3b3ba5290344c635027b14e997a0d
+  metadata.gz: a91f92d624ef1c32f1c60e646f04d63df1f4bd1c2ff7e599abdbc687964997de
+  data.tar.gz: 2775a1a98a0cf4309293aaf075406ab543259d530db7a59d80ad9698e835326a
 SHA512:
-  metadata.gz: decae167312a1f71047a6a830dc25e6f497f0c537657749c597720f548462ec250c32b44f09f345338b2bde9c3d16d3131f9f8217cb606583f1e2ddd2d93d9f8
-  data.tar.gz: 426245645098e4296757e9eeb23ebc807fef93b954c92b68b8d678f358fc7540f69bc822dc7d171a6244dba5d686c6b9ccc12ea647da33f142075d05de07c09d
+  metadata.gz: 7d9f05cf642375bd3189ebc93343415ed46c4cbf6cd9060640fa5168f2504886e1961b9b6c376a91d2c3e54559c2b34e90db2d4f1ac5eb23eaeeafd1fc7ae14a
+  data.tar.gz: 3f28eb9663d990735d010521fd79c7644b6f5ea362986ecfbc71a0582a02d54e779ec82c61e6291aa0e6f17bd913277d1c3eadf9ae05fc818d560ba902b29b98

data/ext/fast_mmaped_file_rs/src/file_info.rs

@@ -91,6 +91,11 @@ impl FileInfo {
 
         match self.file.read_to_end(buf) {
             Ok(n) if n == self.len => Ok(()),
+            // A worker may expand the file between our `stat` and `read`, no harm done.
+            Ok(n) if n > self.len => {
+                self.len = n;
+                Ok(())
+            }
             Ok(_) => Err(MmapError::io(
                 "read",
                 &self.path,
@@ -106,6 +111,8 @@ mod test {
     use magnus::{eval, RArray, Symbol};
     use rand::{thread_rng, Rng};
     use sha2::{Digest, Sha256};
+    use std::fs;
+    use std::io::Write;
 
     use super::*;
     use crate::testhelper::TestFile;
@@ -165,7 +172,7 @@ mod test {
 
         let mut info = FileInfo {
             file,
-            path,
+            path: path.clone(),
             len: buf.len(),
             multiprocess_mode: Symbol::new("puma"),
             type_: Symbol::new("max"),
@@ -187,4 +194,47 @@ mod test {
 
         assert_eq!(in_hash, out_hash, "content hashes");
     }
+
+    #[test]
+    fn test_read_from_file_resized() {
+        let _cleanup = unsafe { magnus::embed::init() };
+        let ruby = magnus::Ruby::get().unwrap();
+        crate::init(&ruby).unwrap();
+
+        const BUF_LEN: usize = 1 << 14; // 16KiB
+
+        // Create a buffer with random data.
+        let mut buf = vec![0u8; BUF_LEN];
+        thread_rng().fill(buf.as_mut_slice());
+
+        let TestFile {
+            file,
+            path,
+            dir: _dir,
+        } = TestFile::new(&buf);
+
+        let mut info = FileInfo {
+            file,
+            path: path.clone(),
+            len: buf.len(),
+            multiprocess_mode: Symbol::new("puma"),
+            type_: Symbol::new("max"),
+            pid: "worker-0_0".to_string(),
+        };
+
+        let mut resized_file = fs::OpenOptions::new()
+            .write(true)
+            .append(true)
+            .open(path)
+            .unwrap();
+
+        // Write data to file after it has been `stat`ed in the
+        // constructor.
+        resized_file.write_all(&[1; 1024]).unwrap();
+
+        let mut out_buf = Vec::new();
+        info.read_from_file(&mut out_buf).unwrap();
+
+        assert_eq!(BUF_LEN + 1024, info.len, "resized file updated len");
+    }
 }

data/ext/fast_mmaped_file_rs/src/mmap.rs

@@ -536,14 +536,14 @@ impl MmapedFile {
         match file.seek(SeekFrom::Start(len - 1)) {
             Ok(_) => {}
             Err(_) => {
-                return Err(MmapError::WithErrno(format!("Can't lseek {}", len - 1)));
+                return Err(MmapError::with_errno(format!("Can't lseek {}", len - 1)));
             }
         }
 
         match file.write(&[0x0]) {
             Ok(1) => {}
             _ => {
-                return Err(MmapError::WithErrno(format!(
+                return Err(MmapError::with_errno(format!(
                     "Can't extend {}",
                     path.display()
                 )));

data/lib/prometheus/client/version.rb

@@ -1,5 +1,5 @@
 module Prometheus
   module Client
-    VERSION = '1.0.0'.freeze
+    VERSION = '1.0.2'.freeze
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: prometheus-client-mmap
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.2
 platform: x86_64-linux
 authors:
 - Tobias Schmidt
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-11-15 00:00:00.000000000 Z
+date: 2023-12-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rb_sys