saviour 0.4.7 → 0.4.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5936e61e689f63ba357bd92fc4b57e6d97cb48e8
-  data.tar.gz: cad8d0487163dca1e329e4b0bfd902162d5eb234
+  metadata.gz: 709681c9a25bc91c834cdf0d2ad1027fbd64a458
+  data.tar.gz: 21ba25beeb98a5aa4ff061f96143896ec7006d5a
 SHA512:
-  metadata.gz: ef0ef25467254b34eb00d0103567d469f43c8b72ebec94448b88543f9330999a829c0453f44aa8b1aabc47f220bc79a4e0f26577051c61bcd870d63fdc9773a4
-  data.tar.gz: 70ec47375eafbf65a8b26fdc89c728aadaceed58e134a0967751592755da1b54bfbf0356743dc8656e8c5858982dcc1e0bf382b4ee17141e527088848687f4d7
+  metadata.gz: 4e2f3746a2455e442ae5920f65244eb8ba596ca6a77864e30d4bb24a388ee14fb2bf17c806112071f7c80310bcaee3fb05077484fb1bd97fd83ca6d2a2b95f0d
+  data.tar.gz: db8b60ee46c478b21faa237ed688c5a53a8526e022df2962cd8ad162948322ae3aa6c6850af5876d7c5fe4dcb01fe5f442559137a84e8f713bd5927a901b72fa

data/lib/saviour/local_storage.rb

@@ -7,21 +7,22 @@ module Saviour
     def initialize(opts = {})
       @local_prefix = opts[:local_prefix]
       @public_url_prefix = opts[:public_url_prefix]
+      @permissions = opts.fetch(:permissions, 0644)
     end
 
     def write(contents, path)
       ensure_dir!(path)
 
-      ::File.open(real_path(path), "w") do |f|
-        f.binmode
-        f.write(contents)
-      end
+      ::File.write real_path(path), contents, mode: "wb"
+
+      ensure_file_permissions!(path)
     end
 
     def write_from_file(file, path)
       ensure_dir!(path)
 
       FileUtils.cp file.path, real_path(path)
+      ensure_file_permissions!(path)
     end
 
     def read_to_file(path, dest_file)
@@ -65,6 +66,10 @@ module Saviour
 
     private
 
+    def ensure_file_permissions!(path)
+      ::File.chmod @permissions, real_path(path)
+    end
+
     def ensure_dir!(path)
       dir = ::File.dirname(real_path(path))
       FileUtils.mkdir_p(dir) unless ::File.directory?(dir)

data/lib/saviour/version.rb

@@ -1,3 +1,3 @@
 module Saviour
-  VERSION = "0.4.7"
+  VERSION = "0.4.8"
 end

data/spec/models/local_storage_spec.rb

@@ -36,6 +36,28 @@ describe Saviour::LocalStorage do
         expect(File.read(path)).to eq contents
       end
     end
+
+    describe "permissions" do
+      it "are 0644 by default" do
+        file_destination = File.join(@tmpdir, destination_path)
+        subject.write "Some contents", destination_path
+        expect(File.file?(file_destination)).to be_truthy
+
+        expect(File.stat(file_destination).mode.to_s(8)).to match /0644$/
+      end
+
+      context do
+        subject { Saviour::LocalStorage.new(local_prefix: @tmpdir, permissions: 0600) }
+
+        it "can be changed by config" do
+          file_destination = File.join(@tmpdir, destination_path)
+          subject.write "Some contents", destination_path
+          expect(File.file?(file_destination)).to be_truthy
+
+          expect(File.stat(file_destination).mode.to_s(8)).to match /0600$/
+        end
+      end
+    end
   end
 
   describe "#read" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: saviour
 version: !ruby/object:Gem::Version
-  version: 0.4.7
+  version: 0.4.8
 platform: ruby
 authors:
 - Roger Campos
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-01-09 00:00:00.000000000 Z
+date: 2018-01-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord