active_fedora-noid 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c19bc92adee3e3f65792c9c8a3d06fff8c3e334f
-  data.tar.gz: b9ddf9a82a9989b270867472363897801d88c108
+  metadata.gz: 7bd89b0820e29801ab139ee09443af8923c1555c
+  data.tar.gz: 471b61afe7180057c9ee56e10b6cf37a1c2ab601
 SHA512:
-  metadata.gz: 9b2b10510a8ae40a4cc4d4518d577dd26f5642903460cbdbc5f1ab87fa4a66347adb8492dc28cec50ac3d4ca6a8424437d98bbeadf9ef207d11270289c6ca57b
-  data.tar.gz: ea939be13d5d4582ed33052b4abd70e249ee48f46dd559308c2ab705cef14671935c64fc477a160b6a2450b9de6d0fc0bdf2f013ce97a731f7472eded2b68366
+  metadata.gz: 339cf5a5e562170ad083e43735479717bda1d549bf8ac4fc3f18dc3f6965064d6bb060e35dbb68e9ecef27755a872104fec65dcecd7d89f6f31a996cae28da64
+  data.tar.gz: 0c0905dd30112fbf3cccbf4ff1547361ba3c8bf1d9dca360584108a859b10f44196ec0efe502c89ab766d5035060731dedf16c59bd67ad2495b598ba707593ff

data/lib/active_fedora/noid/synchronized_minter.rb

@@ -40,16 +40,19 @@ module ActiveFedora
 
       def next_id
         id = ''
-        ::File.open(statefile, ::File::RDWR|::File::CREAT, 0644) do |f|
+        ::File.open(statefile, 'a+b', 0644) do |f|
           f.flock(::File::LOCK_EX)
+          # Files opened in append mode seek to end of file
+          f.rewind
           state = state_for(f)
           minter = ::Noid::Minter.new(state)
+
           id = minter.mint
-          f.rewind
+
+          # Wipe prior contents so the new state can be written from the beginning of the file
+          f.truncate(0)
           new_state = Marshal.dump(minter.dump)
           f.write(new_state)
-          f.flush
-          f.truncate(f.pos)
         end
         id
       end

data/lib/active_fedora/noid/version.rb

@@ -1,5 +1,5 @@
 module ActiveFedora
   module Noid
-    VERSION = '1.0.1'
+    VERSION = '1.0.2'
   end
 end

data/lib/tasks/noid_tasks.rake

@@ -9,19 +9,18 @@ namespace :active_fedora do
       statefile = ENV.fetch('AFNOID_STATEFILE', ActiveFedora::Noid.config.statefile)
       raise "File not found: #{statefile}\nAborting" unless File.exist?(statefile)
       puts "Migrating #{statefile} from YAML to Marshal serialization..."
-      File.open(statefile, File::RDWR | File::CREAT, 0644) do |f|
+      File.open(statefile, 'a+b', 0644) do |f|
         f.flock(File::LOCK_EX)
+        f.rewind
         begin
           yaml_state = YAML.load(f)
         rescue Psych::SyntaxError
           raise "File not valid YAML: #{statefile}\nAborting."
         end
         minter = Noid::Minter.new(yaml_state)
-        f.rewind
+        f.truncate(0)
         new_state = Marshal.dump(minter.dump)
         f.write(new_state)
-        f.flush
-        f.truncate(f.pos)
       end
       puts "Done!"
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_fedora-noid
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - Michael J. Giarlo
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-06 00:00:00.000000000 Z
+date: 2015-08-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active-fedora
@@ -130,7 +130,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: Noid identifier services for ActiveFedora-based applications
@@ -140,4 +140,3 @@ test_files:
 - spec/unit/noid_spec.rb
 - spec/unit/service_spec.rb
 - spec/unit/synchronized_minter_spec.rb
-has_rdoc: