pidlock 0.1.1 → 0.1.2

data/lib/pidlock.rb

@@ -21,7 +21,7 @@ class Pidlock
       unless (File.writable?(File.dirname(@filename)))
         @filename = File.join('/', 'tmp', @name)
       end
-      @file = File.open(@filename, 'r+') 
+      @file = File.open(@filename, File::RDWR|File::CREAT, 0600) 
       if (old_pid = @file.gets)
         if (old_process = Sys::ProcTable.ps(old_pid.chomp.to_i))
           raise ProcessRunning if old_process.comm == File.basename(@name, File.extname(@name))

data/spec/lib/pidlock_spec.rb

@@ -9,7 +9,7 @@ describe Pidlock do
     @file.stub(:flock => 0)
     @file.stub(:flush)
     @file.stub(:gets)
-    File.stub(:open).with('/var/run/my.pid', 'r+').and_return(@file)
+    File.stub(:open).with('/var/run/my.pid', File::RDWR|File::CREAT, 0600).and_return(@file)
     File.stub(:writable?).with('/var/run').and_return(true)
     Logger.stub!(:new => logger)
   end
@@ -26,7 +26,7 @@ describe Pidlock do
       Pidlock.new('my.pid').lock
     end
     it "uses a directory under /var/run if given" do
-      File.should_receive(:open).with("/var/run/my/my.pid", 'r+').and_return(@file)
+      File.should_receive(:open).with("/var/run/my/my.pid",  File::RDWR|File::CREAT, 0600).and_return(@file)
       File.should_receive(:writable?).with("/var/run/my").and_return(true)
       @file.should_receive(:write).with(666)
       Pidlock.new('my/my.pid').lock
@@ -61,7 +61,7 @@ describe Pidlock do
 
     it "uses /tmp if /var/run is not writeable" do
       File.should_receive(:writable?).with('/var/run').and_return(false)
-      File.should_receive(:open).with('/tmp/my.pid', 'r+').and_return(@file)
+      File.should_receive(:open).with('/tmp/my.pid', File::RDWR|File::CREAT, 0600).and_return(@file)
       Pidlock.new('my.pid').lock
     end
     it "warns but continue if the file exists but the process name does not" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pidlock
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors: