ruby-nginx 1.0.1 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b53325ca5fc64324eaf5204d8192e9366a9564279e615d2e3797ce700c87556e
-  data.tar.gz: 6f29b0ed5033f6e738cef7b6b66a514be2d6aa7d1eae9c965f86dc8526e60c4a
+  metadata.gz: b9a56d7ceb957f43c95f6e5cebf1d42a7cd46a09f3625b54e54e83f3e76d94a9
+  data.tar.gz: 9bbbbf29c2a137f7c68f77e9a266e9d0acbbedef55b47aaadad6436a2476ad50
 SHA512:
-  metadata.gz: 4160c514cc0e5562dce1ec6c567dfaa234ecc555fc999eba2aa0739133c9df6a2af2332b688ae415f6593abe21e82c42f5c850fe54a2d00e692db31c78a95a31
-  data.tar.gz: 04d918dec2962326dbec3e20c02e5c62c4f8e9b5e9268d6416f69a1975895de5ff5c03c092e445b2aba14efdf420211e0d0b66e3e244f854cce9b339263f0304
+  metadata.gz: 0de69d714a591acd91cd40a2ef3bc3321a3f187bd5c9305206bf4bae4e608c52740a20088c3035f98cda42d9572bf87fd8acbd14bcc8a1c759716f994ad6914d
+  data.tar.gz: f1d52d87b799624478855abdcf5ee23bf09d6545f10d2bcc1cda0de257b824b9ed56e96577fe2a09ec377f276953cc035c0181bd7df139809864ac680583447c

data/lib/ruby/nginx/commands/add_nginx_config.rb

@@ -9,8 +9,30 @@ module Ruby
       class AddNginxConfig
         include Ruby::Nginx::Constants
 
-        def run(name, config)
-          Ruby::Nginx::System::SafeFile.write("#{SERVERS_PATH}/#{name}.conf", config)
+        def initialize(name, config)
+          @name = name
+          @config = config
+        end
+
+        def run
+          return false unless changed?
+
+          Ruby::Nginx::System::SafeFile.write(config_path, @config)
+          true
+        end
+
+        private
+
+        def changed?
+          !Ruby::Nginx::System::SafeFile.exist?(config_path) || (nginx_config != @config)
+        end
+
+        def config_path
+          @config_path ||= "#{SERVERS_PATH}/#{@name}.conf"
+        end
+
+        def nginx_config
+          @nginx_config ||= Ruby::Nginx::System::SafeFile.read(config_path)
         end
       end
     end

data/lib/ruby/nginx/commands/remove_nginx_config.rb

@@ -9,8 +9,25 @@ module Ruby
       class RemoveNginxConfig
         include Ruby::Nginx::Constants
 
-        def run(name)
-          Ruby::Nginx::System::SafeFile.rm("#{SERVERS_PATH}/#{name}.conf")
+        def initialize(name)
+          @name = name
+        end
+
+        def run
+          return false unless exists?
+
+          Ruby::Nginx::System::SafeFile.rm(config_path)
+          true
+        end
+
+        private
+
+        def exists?
+          Ruby::Nginx::System::SafeFile.exist?(config_path)
+        end
+
+        def config_path
+          @config_path ||= "#{SERVERS_PATH}/#{@name}.conf"
         end
       end
     end

data/lib/ruby/nginx/system/nginx.rb

@@ -32,11 +32,11 @@ module Ruby
           end
 
           def add_server_config(name, config)
-            Commands::AddNginxConfig.new.run(name, config)
+            Commands::AddNginxConfig.new(name, config).run
           end
 
           def remove_server_config(name)
-            Commands::RemoveNginxConfig.new.run(name)
+            Commands::RemoveNginxConfig.new(name).run
           end
 
           def validate_config!

data/lib/ruby/nginx/system/safe_file.rb

@@ -10,11 +10,15 @@ module Ruby
           def mkdir(dir_path)
             safe_path = File.expand_path(dir_path)
 
-            FileUtils.mkdir_p(dir_path)
+            FileUtils.mkdir_p(safe_path)
 
             safe_path
           end
 
+          def exist?(file_path)
+            File.exist?(File.expand_path(file_path))
+          end
+
           def touch(file_path)
             safe_path = File.expand_path(file_path)
 
@@ -25,9 +29,7 @@ module Ruby
           end
 
           def read(file_path)
-            safe_path = File.expand_path(file_path)
-
-            File.read(safe_path)
+            File.read(File.expand_path(file_path))
           end
 
           def write(file_path, content)
@@ -39,9 +41,7 @@ module Ruby
           end
 
           def rm(file_path)
-            safe_path = File.expand_path(file_path)
-
-            FileUtils.rm_f(safe_path)
+            FileUtils.rm_f(File.expand_path(file_path))
           end
         end
       end

data/lib/ruby/nginx/templates/nginx.conf.erb

@@ -56,6 +56,7 @@ server {
 
   # Do not use the default proxy_temp_path, this is to avoid directory permissions issues.
   proxy_temp_path <%= options[:temp_path] %> 1 2;
+  client_body_temp_path <%= options[:temp_path] %> 1 2;
 
   # reverse proxy
   location @<%= name %> {

data/lib/ruby/nginx/version.rb

@@ -2,7 +2,7 @@
 
 module Ruby
   module Nginx
-    VERSION = "1.0.1"
+    VERSION = "1.0.3"
 
     Version = Gem::Version
   end

data/lib/ruby/nginx.rb

@@ -14,9 +14,11 @@ module Ruby
       setup!
       conf = config(options, &block)
 
-      System::Nginx.add_server_config(conf.name, conf.generate!)
-      System::Nginx.validate_config!
-      System::Nginx.restart!
+      if System::Nginx.add_server_config(conf.name, conf.generate!)
+        System::Nginx.validate_config!
+        System::Nginx.restart!
+      end
+
       System::Hosts.add(conf.domain, conf.host)
 
       conf
@@ -30,8 +32,7 @@ module Ruby
     def self.remove!(options = {}, &block)
       conf = config(options, &block)
 
-      System::Nginx.remove_server_config(conf.name)
-      System::Nginx.restart!
+      System::Nginx.restart! if System::Nginx.remove_server_config(conf.name)
       System::Hosts.remove(conf.domain)
 
       conf

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby-nginx
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.3
 platform: ruby
 authors:
 - Bert McCutchen
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-03-27 00:00:00.000000000 Z
+date: 2025-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: erb